Skip to content

Hide leaderboard when leaderboard is hidden

Placeholder Adrien Pavão requested to merge fix-hidden-leaderboard into develop

Summary

The leaderboard was still visible even with the "Leaderboard is hidden" option enabled. This PR fix this behavior. Only the display is changed here, so the leaderboard may be accessible using API or other tricks.

Issues this PR resolves

A checklist for hand testing

  • Check that you can see the leaderboard with a non-admin account
  • Hide the leaderboard
  • Check that it is hidden for the non-admin

Checklist

  • Code review by me
  • Hand tested by me
  • I'm proud of my work
  • Code review by reviewer
  • Hand tested by reviewer
  • CircleCi tests are passing
  • Ready to merge

Merge request reports

Loading