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

Ft priority team members #158174599 #36

Open
wants to merge 113 commits into
base: develop
Choose a base branch
from

Conversation

archbishop12
Copy link
Contributor

Status

READY

Migrations

YES

Description

User should be able to set up a priority list for users to be notified one after the other after a job goes down.

Related PRs

None

Todos

  • Tests
  • Views
  • Models
  • Profile UI

Deploy Notes

On deployment run migrations ./manage.py migrate

Run background processes ./manage.py ensuretriggers and ./manage.py sendalerts

Steps to Test or Reproduce

* Clone the repo, run migrations. First run ```./ manage.py makemogrations``` then ```./ manage.py migrate```
* Run the application ```./ manage.py runserver```.
* Open the app in the browser, log in and invite team members.
* Go to the account settings and invite team members.
* Assign the different users checks and priority.
* If any of the checks assigned to the team members, they should be notified by the prioirity given

Impacted Areas in Application

Edited

  • hc/accounts/views.py - Added the assign_checks that handles the logic for assigning and un assigning checks
  • hc/templates/accounts/profile.html - Added view assigns link and assign check modal.
  • hc/front/tests/test_add_profile.py - Added tests for the assigning checks
  • hc/static/js/profile.js - Added jquery to handle the check boxes.

##Screen shots
screen shot 2018-07-23 at 17 22 49
screen shot 2018-07-23 at 17 22 41

Wagubi Brian and others added 30 commits June 22, 2018 12:21
@magicmarie magicmarie temporarily deployed to health-checks-aces-stagi-pr-36 July 23, 2018 14:24 Inactive
@magicmarie magicmarie temporarily deployed to health-checks-aces-stagi-pr-36 July 23, 2018 16:47 Inactive
@EdwinKato EdwinKato temporarily deployed to health-checks-aces-stagi-pr-36 July 23, 2018 16:53 Inactive
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.

3 participants