One of the changes that was made was not retrieving all resources for the dropdown filters.
This had an unintended consequence when displaying the selected filters as it's possible for the selected filter to not be in the list of allFilters.
There's two main scenarios this happens (the prerequisite is having more filter options than we display which is 250):
* When filtering down the dropdown filters which means it might not be in the allFilters options. This was fixed by keeping track of all the searched options to be able to display.
* A user uses a query string or bookmarked a URL and goes to it directly. If the option isn't in the initial load, we won't have the actual option. In this case, I changed it to at least just display the ID instead of not showing up at all. Otherwise I'd have to load these selected options separately from the initial load which doesn't seem worth for this edge case.
This commit adds the ability to search across multiple tables with joins which unlocks a better search experience in the case of tables that source their data from multiple tables.
This commit has a few notable changes:
1. A simple attempt at establishing a pattern for using SELECT statements. This will be useful in the future for joins/subquery support.
2. Expanded usage of searching to be able to pass instead of just a string, an object that lets a consumer choose which columns to search on.
3. Decouple our session recording filters from being loaded from the route model hook which is a departure from our convention. This has the benefit of actually letting the page first load and having the filters load async so the page doesn't wait for the filters to finish. This is also much faster to query for each individual filter option from SQLite rather than returning all data to filter in application code.
* fix: 🐛 properly show username for oidc auth method in user menu
* test: 💍 fix admin ui tests
* test: 💍 fix desktop ui tests
* refactor: 💡 admin to not store username in authenticated object
* refactor: 💡 use getter in app controller
* test: 💍 POC for how we could update our tests to use auth account
* refactor: 💡 remove unused service
* test: 💍 fix all tests in ui/admin
* test: 💍 revert one change
* refactor: 💡 update ui/desktop w/ feedback + fix tests
* refactor: 💡 add in error handling
* refactor: 💡 cleanup unused service
* test: 💍 fix e2e test selector
✅ Closes: https://hashicorp.atlassian.net/browse/ICU-17231
* chore: 🤖 add ember-a11y-ignore tool
* chore: 🤖 fix handling existing setRunOptions imports
* chore: 🤖 move codemod to support folder
* chore: 🤖 fix edge case for handling property key names
* chore: 🤖 handle case where multiple of same violation in test
This fixes a bug where multiple cases of the same violation would end up
with multiple entries
* chore: 🤖 ignore axe rules that are currently failing in admin
* chore: 🤖 ignore axe rules that are currently failing in desktop
* docs: ✏️ add a comment explaining the codemod
* chore: 🤖 include test.each cases for a11y ignores
* fix: 🐛 handle test.each test cases for a11y ignore codemod
Lisbet Alvarez
Zhihe Li
Dylan HyunandZhihe Li
Priya Patel
hashicc
Dharini
cameronperera