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

User registration, with admin/user pannel #6

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

Himasnhu-AT
Copy link

@Himasnhu-AT Himasnhu-AT commented Feb 10, 2024

User registration, with admin/user panel.

Resolves #5. @invinayak007

Dev Docs and User Docs

Features:

  • Sign in as user or admin.
  • ADMIN can create subscriptions, modify them, and send alerts to subscribed users. (The alert feature isn't working right now, will be done while integrating it with UCI.
  • USER can set preferences and send alerts.

Future tasks:

  • Improve UI
  • Add integration test, and increase test coverage.

This is raised early so that other features can be implemented alongside so that the product can be released early.

Test Written:

  • 8 test cases to cover API requests
  • test cases to write: cover more edge cases, integration testing.

AUTH WORK FLOW:

image

@invinayak007
Copy link
Collaborator

User registration, with admin/user panel.

Resolves #5. @invinayak007

Dev Docs and User Docs

Features:

  • Sign in as user or admin.
  • ADMIN can create subscriptions, modify them, and send alerts to subscribed users. (The alert feature isn't working right now, will be done while integrating it with UCI.
  • USER can set preferences and send alerts.

Future tasks:

  • Improve UI
  • Add integration test, and increase test coverage.

This is raised early so that other features can be implemented alongside so that the product can be released early.

Test Written:

  • 8 test cases to cover API requests
  • test cases to write: cover more edge cases, integration testing.

AUTH WORK FLOW:

image

Send alert is not an operation from admin or user. After keeping the preferences, application will check the topic of a particular message and it will send it to UCI to deliver to recepients

@Himasnhu-AT
Copy link
Author

hey @invinayak007 , as described on Discord, the send alert functionality is just to test whether this functionality is working or not. Later on we can remove it and call dynamically using api's

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.

#4: User registration for the topics with preferences
2 participants