New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Replaces sortString with sortBy #1055

Merged

JorWo merged 1 commit into uhawaii-system-its-ti-iam:main from JorWo:dev-jordanw4-1843

Feb 3, 2025

Contributor

JorWo commented Feb 3, 2025

Ticket Link

List of squashed commits

Removes stale getGrouping code in GroupingsRestController
Adds missing test coverage for getGrouping in GroupingsRestController

Test Checklist

Exhibits Clear Code Structure:
Project Unit Tests Passed:
Project Jasmine Tests Passed:
Executes Expected Functionality:
Tested For Incorrect/Unexpected Inputs:


          Replaces sortString with sortBy

ffa32ff

Contributor Author

JorWo commented Feb 3, 2025 •

edited

Loading

Quick fix in response to changes to API from this ticket.

JorWo requested a review from mhodgesatuh

February 3, 2025 18:28

mhodgesatuh approved these changes

View reviewed changes

JorWo merged commit 12e0411 into uhawaii-system-its-ti-iam:main

14 checks passed

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

Labels

None yet