-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature/arnav710/add delete admin flow #152
Conversation
…e + select/deselect all functionality
- Added an options interface to define each option in the dropdown - Made changes to select all so that the select all opertaion could be performed in multiple tabs at the same time
…//github.com/TritonSE/LAK-Goods-Transport-Application into page/Arnav710/driver-registeration-dashboard
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a comment here on the page structuring:
So in Next, files under pages will be automatically routed to. However, we'd rather have the structure of these pages to be /admin, /admin/add, /admin/delete from a routing perspective. To do this, we can move all the files under a folder in pages named admin, and rename admin.tsx to index.tsx, add-admin.tsx to add.tsx, and delete-admin.tsx to delete.tsx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same comments about file stucture for pages "/admin/add", "/admin/delete", "/admin/index.tsx", etc.
Adminstrative Info
Closes #104
Changes
Added add and delete admin pages, and created component for pop-up box that appears once add or delete buttons clicked.
Confirmation of Change