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

Back-end: Change current functionality to support "time" property of notification #313

Closed
OrAbramovich opened this issue Apr 22, 2018 · 3 comments
Assignees
Labels

Comments

@OrAbramovich
Copy link
Collaborator

No description provided.

@OrAbramovich OrAbramovich added this to the Mid Semester POC milestone Apr 22, 2018
@OrAbramovich OrAbramovich self-assigned this Apr 22, 2018
@OrAbramovich
Copy link
Collaborator Author

Estimated time: 1.5 hour

OrAbramovich added a commit that referenced this issue Apr 23, 2018
…cation

Back-end: Changed current functionality to support "time" property of notification
DB: Back-end: Added time to notification schema

The following commit adds a new property to the notification structure - the time notification was created. We would like to hold this data in order to display it to the user in front-end. The commit includes new & modified tests relevant to the change.

#311, #313, #318
@OrAbramovich
Copy link
Collaborator Author

Actual time: 0.75 hour

@OrAbramovich
Copy link
Collaborator Author

Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant