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

feat-604: Added input to change cohort in Admin Panel #605

Merged
merged 4 commits into from
Sep 9, 2024

Conversation

ItsFlash10
Copy link
Contributor

@ItsFlash10 ItsFlash10 commented Aug 30, 2024

PR Fixes:

  • Admin would be able to change the cohort for any uploaded track.

@hkirat @devsargam

Resolves #604

Untitled.design.video.mp4

Checklist before requesting a review

  • I have performed a self-review of my code
  • I assure there is no similar/duplicate pull request regarding same issue

@ItsFlash10
Copy link
Contributor Author

PS: UI can be improved as well but I've catered the functionality atm as it's important as highlighted a couple of days ago for the filters.

@devsargam
Copy link
Collaborator

Testing locally

@devsargam
Copy link
Collaborator

This updates the backend but can we get the filter functionality working too?

Copy link
Collaborator

@devsargam devsargam left a comment

Choose a reason for hiding this comment

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

Make the filter option working as well

@ItsFlash10
Copy link
Contributor Author

Make the filter option working as well

Done

@ItsFlash10 ItsFlash10 requested a review from devsargam September 4, 2024 16:54
@devsargam
Copy link
Collaborator

Testing locally tomorrow

@devsargam
Copy link
Collaborator

works perfectly

@devsargam devsargam merged commit 082bdff into code100x:main Sep 9, 2024
1 check failed
@devsargam
Copy link
Collaborator

Thank u @ItsFlash10. You will receive the bounty later

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.

feature: Should be able to assign cohort from the admin dashboard
2 participants