Skip to content
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

Trashcan button in Manage Grouping should be in its own column #950

Merged
merged 1 commit into from
Jul 5, 2024
Merged

Trashcan button in Manage Grouping should be in its own column #950

merged 1 commit into from
Jul 5, 2024

Conversation

hokwaichan
Copy link
Contributor

Ticket Link

1698

List of squashed commits

  • Fixed trashcan icon

Test Checklist

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

@hokwaichan hokwaichan changed the title Fixed trashcan icon Trashcan button in Manage Grouping should be in its own column Jul 2, 2024
@hokwaichan hokwaichan requested review from mhodgesatuh and JorWo July 2, 2024 03:09
@JorWo
Copy link
Contributor

JorWo commented Jul 2, 2024 •
edited
Loading

owners.html (Owners tab) will need this change too.

@JorWo JorWo mentioned this pull request Jul 2, 2024
Closed
5 tasks
JorWo
JorWo approved these changes Jul 5, 2024
View reviewed changes
Copy link
Contributor

@JorWo JorWo left a comment

Choose a reason for hiding this comment

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

Thanks for making this fix!

@JorWo JorWo merged commit e986e01 into uhawaii-system-its-ti-iam:main Jul 5, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JorWo JorWo JorWo approved these changes

@mhodgesatuh mhodgesatuh Awaiting requested review from mhodgesatuh

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hokwaichan @JorWo