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.

Sign up for GitHub

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

Jump to bottom

fix(summary): Incorrect go to url #733

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

fix(summary): Incorrect go to url #733

wants to merge 9 commits into from

Conversation

Jonathanio123
Copy link
Collaborator

@Jonathanio123 Jonathanio123 commented Dec 11, 2024 •
edited
Loading

  • New feature
  • Bug fix
  • High impact

Description of work:

AB#57452

  • Completely reworks calculation of task owner report metrics
  • Fixed go to urls
  • Minor report layout change, url at the top
  • Order entries by date
  • Limit to 10 max items
  • Possible to manually set filter for who can receive reports, mostly for testing

Testing:

  • Can be tested
  • Automatic tests created / updated
  • Local tests are passing

Checklist:

  • Considered automated tests
  • Considered updating specification / documentation
  • Considered work items
  • Considered security
  • Performed developer testing
  • Checklist finalized / ready for review

@Jonathanio123 Jonathanio123 marked this pull request as ready for review December 11, 2024 12:33
Mustafa-Hersi
Mustafa-Hersi approved these changes Dec 11, 2024
View reviewed changes
Copy link
Collaborator

@Mustafa-Hersi Mustafa-Hersi left a comment

Choose a reason for hiding this comment

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

Phenomenal work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Mustafa-Hersi Mustafa-Hersi Mustafa-Hersi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Jonathanio123 @Mustafa-Hersi