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 156577855 team access awareness #23

Merged
merged 9 commits into from
May 18, 2018

Conversation

dondrzzy
Copy link

@dondrzzy dondrzzy commented May 17, 2018

What does this PR do?

This PR ensures that users get to know that they can access an expanded dashboard that allows teams to collaboratively create and manage alerts based on the landing page UI/UX rework

Description of Task to be completed?

When a user wants to know what HealthChecks offers on accessing the landing page, they should be aware they should be aware that they can create teams and access/manage checks as a team.

How should this be manually tested?

  1. Clone the application and install the requirements in the requirements.txt
    2.Run the application and access the landing page
    3: There should be a section that informs the user of the features that health checks has to offer.

What are the relevant pivotal tracker stories?

#156577855

Checklist: I completed these to help reviewers

  • My IDE is configured to follow the PEP8 Conventions

    No? Unsure? -> configure your IDE, format the code and add the changes with git add . && git commit --amend

  • I have added tests to cover my changes. (If you refactored
    existing code that was well tested you do not have to add tests)

    No? -> write tests and add them to this commit git add . && git commit --amend

  • All new and existing tests passed.

    No? -> figure out why and add the fix to your commit. It is your responsibility to make sure your code works.

  • My pull request is based on the latest changes of the develop branch.

    No? Unsure? -> execute command git pull --rebase origin develop

@peterwade153
Copy link

Change the branch you want to merge to from master to develop

@aenkya
Copy link
Collaborator

aenkya commented May 18, 2018

Your PR checklist needs fixing
@dondrzzy

@dondrzzy dondrzzy changed the base branch from master to develop May 18, 2018 08:10
Copy link
Collaborator

@aenkya aenkya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You seem to get rid of a file my_checks.png that is still required in the landing page. Either update the links to the image or reintroduce the file

@coveralls
Copy link

Pull Request Test Coverage Report for Build 273

  • 79 of 105 (75.24%) changed or added relevant lines in 11 files are covered.
  • 66 unchanged lines in 4 files lost coverage.
  • Overall coverage decreased (-0.2%) to 78.305%

Changes Missing Coverage Covered Lines Changed/Added Lines %
hc/front/models.py 9 10 90.0%
hc/front/forms.py 21 22 95.45%
hc/front/views.py 18 19 94.74%
hc/celery.py 0 11 0.0%
hc/tasks.py 0 12 0.0%
Files with Coverage Reduction New Missed Lines %
hc/settings.py 3 98.39%
hc/front/forms.py 4 81.72%
hc/api/management/commands/ensuretriggers.py 7 100.0%
hc/front/views.py 52 79.12%
Totals Coverage Status
Change from base Build 259: -0.2%
Covered Lines: 1682
Relevant Lines: 2148

💛 - Coveralls

2 similar comments
@coveralls
Copy link

Pull Request Test Coverage Report for Build 273

  • 79 of 105 (75.24%) changed or added relevant lines in 11 files are covered.
  • 66 unchanged lines in 4 files lost coverage.
  • Overall coverage decreased (-0.2%) to 78.305%

Changes Missing Coverage Covered Lines Changed/Added Lines %
hc/front/models.py 9 10 90.0%
hc/front/forms.py 21 22 95.45%
hc/front/views.py 18 19 94.74%
hc/celery.py 0 11 0.0%
hc/tasks.py 0 12 0.0%
Files with Coverage Reduction New Missed Lines %
hc/settings.py 3 98.39%
hc/front/forms.py 4 81.72%
hc/api/management/commands/ensuretriggers.py 7 100.0%
hc/front/views.py 52 79.12%
Totals Coverage Status
Change from base Build 259: -0.2%
Covered Lines: 1682
Relevant Lines: 2148

💛 - Coveralls

@coveralls
Copy link

Pull Request Test Coverage Report for Build 273

  • 79 of 105 (75.24%) changed or added relevant lines in 11 files are covered.
  • 66 unchanged lines in 4 files lost coverage.
  • Overall coverage decreased (-0.2%) to 78.305%

Changes Missing Coverage Covered Lines Changed/Added Lines %
hc/front/models.py 9 10 90.0%
hc/front/forms.py 21 22 95.45%
hc/front/views.py 18 19 94.74%
hc/celery.py 0 11 0.0%
hc/tasks.py 0 12 0.0%
Files with Coverage Reduction New Missed Lines %
hc/settings.py 3 98.39%
hc/front/forms.py 4 81.72%
hc/api/management/commands/ensuretriggers.py 7 100.0%
hc/front/views.py 52 79.12%
Totals Coverage Status
Change from base Build 259: -0.2%
Covered Lines: 1682
Relevant Lines: 2148

💛 - Coveralls

@coveralls
Copy link

coveralls commented May 18, 2018

Pull Request Test Coverage Report for Build 296

  • 127 of 159 (79.87%) changed or added relevant lines in 13 files are covered.
  • 66 unchanged lines in 4 files lost coverage.
  • Overall coverage increased (+0.2%) to 78.718%

Changes Missing Coverage Covered Lines Changed/Added Lines %
hc/front/models.py 9 10 90.0%
hc/front/forms.py 30 31 96.77%
hc/api/models.py 7 10 70.0%
hc/front/views.py 43 47 91.49%
hc/celery.py 0 11 0.0%
hc/tasks.py 0 12 0.0%
Files with Coverage Reduction New Missed Lines %
hc/settings.py 3 98.39%
hc/front/forms.py 4 83.17%
hc/api/management/commands/ensuretriggers.py 7 100.0%
hc/front/views.py 52 79.65%
Totals Coverage Status
Change from base Build 275: 0.2%
Covered Lines: 1731
Relevant Lines: 2199

💛 - Coveralls

@dondrzzy dondrzzy force-pushed the ft-team-access-awareness-#156577855 branch from a8734bc to 16f0fdb Compare May 18, 2018 09:57
@aenkya aenkya merged commit 046ddf1 into develop May 18, 2018
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

5 participants