Skip to content
Snippets Groups Projects

Update admin health page with the application readiness endpoints

Merged Jose Ivan Vargas Lopez requested to merge update-admin-health-page into master
All threads resolved!

What does this MR do?

Adds the application readiness endpoint links to the monitoring -> health check view inside the admin section

Screenshots (if relevant)

Before After
Screen_Shot_2017-05-15_at_10.46.29_AM Screen_Shot_2017-05-15_at_1.05.35_PM

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

closes #31359 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • @jivanvl A couple tiny comments. Otherwise, this LGTM :star:

  • Thanks @brycepj should I correct the double quotes and assign to an endboss?

  • added 1 commit

    • 1f299612 - Removed old endpoints from the monitoring view

    Compare with previous version

  • Jose Ivan Vargas Lopez resolved all discussions

    resolved all discussions

  • username-removed-408230 approved this merge request

    approved this merge request

  • @brycepj Thank you for reviewing, @filipa handing this over to you :thumbsup:

  • Thank you @jivanvl code LGTM! :thumbsup:

  • Filipa Lacerda enabled an automatic merge when the pipeline for 1f299612 succeeds

    enabled an automatic merge when the pipeline for 1f299612 succeeds

  • @jivanvl can you please merge master? Tests keep failing :sob:

  • assigned to @jivanvl

  • added 72 commits

    • 1f299612...43befaf2 - 70 commits from branch master
    • e402000b - update admin health page with the application readiness endpoints
    • 94a6c0de - Removed old endpoints from the monitoring view

    Compare with previous version

  • @filipa builds passed :tada:

  • Filipa Lacerda approved this merge request

    approved this merge request

  • Filipa Lacerda mentioned in commit 55f6b1ed

    mentioned in commit 55f6b1ed

  • Please register or sign in to reply
    Loading