Skip to content

removing deprecated report page and disabling report-service UI buttons#238

Merged
Zakaria-Kofiro merged 1 commit intomasterfrom
zkofiro/remove-reporting
Apr 24, 2023
Merged

removing deprecated report page and disabling report-service UI buttons#238
Zakaria-Kofiro merged 1 commit intomasterfrom
zkofiro/remove-reporting

Conversation

@Zakaria-Kofiro
Copy link
Collaborator

@Zakaria-Kofiro Zakaria-Kofiro commented Apr 11, 2023

Removing unused report page from Tank and disabling non-functional job report UI buttons

Small PR for removing references to v1/report-service including a deprecated /report page and disabling the three job report buttons (Summary Timing Data, Periodic, and CSV Timing Data) that are referencing v1/report-service and currently return error pages:

Removing Unused Report Page:

https://qa-tank.perf.a.intuit.com/report

Landing Page:

Report Options:

Disabling Report UI buttons:

Current Job Queue:

Current Agent Tracker:

New Job Queue:
Screenshot 2023-04-20 at 6 14 55 PM

New Agent Tracker:
Screenshot 2023-04-20 at 6 15 10 PM

Please make sure these check boxes are checked before submitting

  • ** Squashed Commits **
  • ** All Tests Passed ** - mvn clean test -P default

** PR review process **

  • Requires one +1 from a reviewer
  • Repository owners will merge your PR once it is approved.

@shawn-h-park shawn-h-park self-requested a review April 24, 2023 17:30
…e UI buttons

removing instead of commenting out

Update SSL test to working URL
@Zakaria-Kofiro Zakaria-Kofiro merged commit b58496a into master Apr 24, 2023
@Zakaria-Kofiro Zakaria-Kofiro deleted the zkofiro/remove-reporting branch April 24, 2023 20:28
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.

2 participants