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

Sidenav and path changes for activity feature #1103

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

brrkrmn
Copy link
Collaborator

@brrkrmn brrkrmn commented Feb 15, 2024

Summary

This PR addresses the changes mentioned in #1099.

Closes #1099

Changes

  • Create activity button is visible to 'educator' and 'staff' role users
  • Creators get a warning when they try to visit /activities/create
  • Explore Activities button in the sidenav redirect the users to /activities

Screenshots

For Educators:
Screenshot 2024-02-15 at 10 21 43 PM

For Creators:
Screenshot 2024-02-15 at 10 22 08 PM

Copy link

pre-commit checks failed ❌❌❌
make sure you have pre-commit set up locally (see the README.md for instructions).
If you have set up pre-commit locally, you may need to undo this commit and
run "pre-commit" locally to see what the errors are.

@brrkrmn brrkrmn force-pushed the activity-sidenav-path branch from 329d355 to 3b07b23 Compare February 29, 2024 11:22
@brrkrmn brrkrmn force-pushed the activity-sidenav-path branch from 3b07b23 to e2a7675 Compare February 29, 2024 11:29
Copy link

All pre-commit checks passed ✅✅✅

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.

(activity): Sidenav and Paths
1 participant