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.

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

[#12596] Part of #12596 table doesn't sort by team #12635

Closed
wants to merge 1 commit into from

Conversation

wanqiqiu
Copy link

Fixes part of #12596

Outline of Solution
I made changes to the SortBy.TEAM_NAME section in both extension-confirm-modal.component.ts and instructor-session-individual-extension-page.component.ts to resolve the issue of the table not sorting by the team.

@nusoss-bot
Copy link

Folks, This PR seems to be stalling (no activities for the past 7 days). 🐌 😢
Hope someone can get it to move forward again soon...

1 similar comment
@nusoss-bot
Copy link

Folks, This PR seems to be stalling (no activities for the past 7 days). 🐌 😢
Hope someone can get it to move forward again soon...

@nusoss-bot
Copy link

Folks, This PR seems to be stalling (no activities for the past 9 days). 🐌 😢
Hope someone can get it to move forward again soon...

@cedricongjh
Copy link
Contributor

hey @wanqiqiu we'll proceed to close the PR, do reopen it if you're still working on it, there seems to be an issue with the E2E tests, and that needs to be fixed before we can proceed with reviewing the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants