Skip to content

Conversation

@elzody
Copy link
Contributor

@elzody elzody commented Jan 13, 2026

Summary

When you try to limit an app by a group, the dropdown is initially not filled with any groups. They only populate once you start typing the name of a group, which triggers a network request. This pull request provides the necessary initial state to the app list so that the groups are populated by default and you can quickly select one without searching.

TODO

  • Write a proper test to make sure the dropdown is filled with some groups

Checklist

@elzody elzody self-assigned this Jan 13, 2026
@elzody elzody requested a review from a team as a code owner January 13, 2026 19:56
@elzody elzody added the 2. developing Work in progress label Jan 13, 2026
@elzody elzody requested review from ArtificialOwl, icewind1991 and salmart-dev and removed request for a team January 13, 2026 19:56
@github-project-automation github-project-automation bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Office team Jan 13, 2026
@elzody elzody requested a review from leftybournes January 13, 2026 19:56
@elzody elzody moved this from 🧭 Planning evaluation (don't pick) to 🏗️ In progress in 📝 Office team Jan 13, 2026
@elzody elzody changed the title Fix/limit app to group initial state fix(app-settings): limit app to group initial state Jan 13, 2026
@elzody elzody added the bug label Jan 13, 2026
@elzody elzody force-pushed the fix/limit-app-to-group-initial-state branch 2 times, most recently from 002dc8a to 7bd91a3 Compare January 22, 2026 19:05
@elzody elzody requested a review from a team as a code owner January 23, 2026 21:53
@elzody elzody requested review from nfebe, sorbaugh and susnux and removed request for a team January 23, 2026 21:53
@elzody elzody force-pushed the fix/limit-app-to-group-initial-state branch from 48486dc to daf55db Compare January 23, 2026 22:01
Signed-off-by: Elizabeth Danzberger <[email protected]>
@elzody elzody force-pushed the fix/limit-app-to-group-initial-state branch from daf55db to 90deae4 Compare January 26, 2026 17:47
@elzody elzody moved this from 🏗️ In progress to 👀 In review in 📝 Office team Jan 28, 2026
@elzody elzody added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Jan 28, 2026
@elzody
Copy link
Contributor Author

elzody commented Jan 28, 2026

Should be ready for a proper review now. Added a Cypress test for coverage.

@elzody
Copy link
Contributor Author

elzody commented Jan 28, 2026

/backport to stable31

@elzody
Copy link
Contributor Author

elzody commented Jan 28, 2026

/backport to stable33

@elzody
Copy link
Contributor Author

elzody commented Jan 28, 2026

/backport to stable32

Copy link
Contributor

@susnux susnux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no frontend code needed?

@elzody
Copy link
Contributor Author

elzody commented Jan 30, 2026

no frontend code needed?

No. The front end already expects the initial state to be there, so we just need to provide it in the backend. Because it wasn't before, it was defaulting to an empty dropdown and only populated it once you started searching.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

5 participants