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

ASC-37 #30

Merged
merged 3 commits into from
Jul 22, 2024
Merged

ASC-37 #30

merged 3 commits into from
Jul 22, 2024

Conversation

thea2506
Copy link
Contributor

@thea2506 thea2506 commented Jul 5, 2024

In this PR, I have:

  • Created a new page at /admin
  • Added Search queries, Table queries, Sorting by order, category queries
  • Added authentication: log out, URL restrictions for un-authorized user in /admin

Notes:

  • Add member functionality not yet implemented
  • The pagination is modified into a load more button
  • Visit /admin for testing (remember to log in at admin/login)

@thea2506 thea2506 requested a review from PaulHo0501 July 5, 2024 02:33
@thea2506 thea2506 self-assigned this Jul 5, 2024
Copy link

github-actions bot commented Jul 5, 2024

Visit the preview URL for this PR (updated for commit 9275c3c):

https://alzheimercalgary-v2--pr30-asc-37-orzlqmh5.web.app

(expires Fri, 12 Jul 2024 02:35:23 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 49842b2747b50197f445c596a105e7df038b0757

Copy link
Collaborator

@PaulHo0501 PaulHo0501 left a comment

Choose a reason for hiding this comment

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

LGTM!

@PaulHo0501 PaulHo0501 merged commit d432583 into main Jul 22, 2024
2 checks passed
@PaulHo0501 PaulHo0501 deleted the ASC-37 branch July 22, 2024 17:08
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.

2 participants