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

Admin panel #230

Closed
wants to merge 181 commits into from
Closed

Admin panel #230

wants to merge 181 commits into from

Conversation

bsilkyn
Copy link
Contributor

@bsilkyn bsilkyn commented Apr 3, 2024

The whole admin panel, a place where admin actions can be executed in a ui, should be added.

EwoutV and others added 30 commits March 14, 2024 21:46
Deploy first version of the frontend
# Conflicts:
#	frontend/src/assets/lang/en.json
#	frontend/src/components/projects/ProjectCard.vue
#	frontend/src/router/router.ts
#	frontend/src/views/projects/ProjectView.vue
@bsilkyn
Copy link
Contributor Author

bsilkyn commented Apr 11, 2024

ik heb aan AdminView.vue, UsersView.vue, users.service.ts, router.ts, AdminHeader.vue, AdminLayout.vue, AdminSidebar.vue in frontend en user_view.py in backend gewerkt

@bsilkyn bsilkyn marked this pull request as ready for review April 11, 2024 13:02
@bsilkyn bsilkyn requested review from tyboro2002 and EwoutV April 11, 2024 13:04
@EwoutV
Copy link
Contributor

EwoutV commented Apr 11, 2024

ik heb aan AdminView.vue, UsersView.vue, users.service.ts, router.ts, AdminHeader.vue, AdminLayout.vue, AdminSidebar.vue in frontend en user_view.py in backend gewerkt

Do we risk other work being overwritten by outdated files? I think it's better to make another branch starting from the development branch, including only the changes you listed here and then create a new PR just to be sure we don't lose any important work.

@EwoutV EwoutV closed this Apr 12, 2024
@bsilkyn bsilkyn deleted the admin-panel branch April 15, 2024 12:22
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.

7 participants