Skip to content

ROU-12609: Rename DropdownServerSideItem OnSelect params and add optional parent dropdown id#1114

Merged
gnbm merged 2 commits intodevfrom
ROU-12609
Mar 2, 2026
Merged

ROU-12609: Rename DropdownServerSideItem OnSelect params and add optional parent dropdown id#1114
gnbm merged 2 commits intodevfrom
ROU-12609

Conversation

@gnbm
Copy link
Copy Markdown
Collaborator

@gnbm gnbm commented Mar 1, 2026

This PR is for updating the DropdownServerSideItem OnSelect event contract to improve naming clarity and support reusable logic.

What was happening

  • DropdownServerSideItem OnSelect event used less explicit parameter names.
  • Consumer logic had no direct parent DropdownServerSide identifier from the callback payload.

What was done

  • Renamed OnSelect parameter dropdownIddropdownServerSideItemId
  • Renamed OnSelect parameter dropdownItemIditemOptionId
  • Added new optional OnSelect parameter dropdownServerSideId
  • Updated callback invocation to pass parent id when available.
  • Validated with lint/build.

Test Steps

  1. Got to a test screen with a DropdownServerSide
  2. Select an option
  3. Check that the three parameters are returned in the OnSelect event with the expected value

Screenshots

image

Checklist

  • tested locally
  • documented the code
  • clean all warnings and errors of eslint
  • requires changes in OutSystems (if so, provide a module with changes)
  • requires new sample page in OutSystems (if so, provide a module with changes)

@gnbm gnbm requested a review from a team as a code owner March 1, 2026 17:15
@gnbm gnbm added enhancement New feature or request chore labels Mar 1, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Mar 1, 2026

@gnbm gnbm merged commit 818da09 into dev Mar 2, 2026
20 checks passed
@gnbm gnbm deleted the ROU-12609 branch March 2, 2026 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants