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

Remove Notification Settings from Profile #35

Open
bhawesh96 opened this issue Dec 18, 2018 · 5 comments
Open

Remove Notification Settings from Profile #35

bhawesh96 opened this issue Dec 18, 2018 · 5 comments
Labels
difficulty/low This issue has low difficulty discussion needed

Comments

@bhawesh96
Copy link
Member

Notification Settings in the Profile section seem not important according to the feedback. We can get rid of it.
Yours views? @Arushi1912

@bhawesh96 bhawesh96 added the difficulty/low This issue has low difficulty label Dec 18, 2018
@Arushi1912
Copy link
Member

I think the users haven't had enough time to interact with the portal. As far as I remember, it was offline for most of the time. Maybe we should wait until 2-3 months of usage?
Bcz according to me, notification settings is relevant. Some people might not want an SMS/email, some people might. Some might need it for approval only. Or, all? (This is optional)

@Anutrix
Copy link

Anutrix commented Aug 28, 2019

Status?

@rsarky
Copy link
Member

rsarky commented Aug 28, 2019

I think we are going to keep the notifications for now. What do you think @bhawesh96 ?

@bhawesh96
Copy link
Member Author

bhawesh96 commented Aug 28, 2019

Either we write the backend to check notification settings' status and send notifications accordingly or we remove this feature entirely from the frontend and database structure.
I personally found this not so useful as notification at every stage might look like a SPAM while notification at the end is a MUST anyway.

@rsarky
Copy link
Member

rsarky commented Aug 28, 2019

Either we write the backend to check notification settings' status and send notifications accordingly

Better to give the user an option to disable the notifications if he wants I guess. Let's go with this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty/low This issue has low difficulty discussion needed
Projects
None yet
Development

No branches or pull requests

4 participants