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

Add get user notification preference #57

Merged
merged 1 commit into from
Sep 11, 2019
Merged

Add get user notification preference #57

merged 1 commit into from
Sep 11, 2019

Conversation

adafia
Copy link
Contributor

@adafia adafia commented Sep 11, 2019

What does this PR do?

This Pull Request adds an endpoint to give users the opportunity of getting their notification preferences

Description of Task to be completed?

This Pull Request adds an endpoint to give users the opportunity of getting their notification preferences

How should this be manually tested?

Database Setup (Postgres - Sequelize )

install dependencies and development dependencies

> npm install

create dataabase to postgresql

> createdb authorsHavenDb

remember to change the password in config.js file for development

create your config file

> create a .env file in your root directory and follow the `example.env`
file in setting up yours

add table to database

> sequelize db:migrate

The above command will migrate all table to your database

remove all tables from the database

> sequelize db:migrate:undo

The above command will revert any migration.

Any background context you want to provide?

Prior to this addition, users could only retrieve unseen notifications and couldn't get their notification preferences

What are the relevant pivotal tracker stories?

N/A

Screenshots (if appropriate)

N/A

Questions:

N/A

 - add user get all notifications
 - add user get notification settings

[Finishes]
@manzif manzif merged commit 446049e into develop Sep 11, 2019
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.

None yet

4 participants