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

Staff role/badge - fix test coverage #18

Closed
yukiiii04 opened this issue Jan 29, 2025 · 1 comment
Closed

Staff role/badge - fix test coverage #18

yukiiii04 opened this issue Jan 29, 2025 · 1 comment
Assignees
Milestone

Comments

@yukiiii04
Copy link
Contributor

yukiiii04 commented Jan 29, 2025

Effort: Medium, maybe 3 days since I don't know how to work with test files and write tests
Dependencies: Finishing the previous work on this feature
Acceptance criteria: Needs to maintain coverage and ensure key code chunks are covered

@yukiiii04 yukiiii04 self-assigned this Jan 29, 2025
@yukiiii04 yukiiii04 modified the milestones: Sprint 1, Sprint 2 Jan 30, 2025
@yukiiii04
Copy link
Contributor Author

Feature abandoned, issue closed. Closes #2.

When looking through the codebase & adding a new group role called "Instructors", I realized that we whenever a new group is created, the toggle badge display feature is also carried over. This makes our proposed feature to be unnecessary & only a slight frontend change.

New proposed feature: polls that's restricted to instructors when posting. Opens #32 !

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

No branches or pull requests

1 participant