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

feat: Phoenix LiveDashboard #1874

Merged
merged 3 commits into from
Sep 28, 2023
Merged

feat: Phoenix LiveDashboard #1874

merged 3 commits into from
Sep 28, 2023

Conversation

cmaddox5
Copy link
Contributor

Asana task: ad-hoc

This adds the Phoenix LiveDashboard to the admin endpoint. Will only be accessible by users who are in the screens-admin group. You can access it via /admin/dashboard. On the page, there are various tabs and panels that show different pieces of info about our server. This includes a Processes tab that actually lists out all of the running processes. I also enabled a request logger, but you have to manually enable a cookie to see it work.

  • Tests added?

@github-actions
Copy link

Coverage of commit 662f9ce

Summary coverage rate:
  lines......: 40.4% (2296 of 5684 lines)
  functions..: 41.8% (1051 of 2512 functions)
  branches...: no data found

Files changed coverage rate:
                                                                           |Lines       |Functions  |Branches    
  Filename                                                                 |Rate     Num|Rate    Num|Rate     Num
  ===============================================================================================================
  lib/screens_web/endpoint.ex                                              | 100%      2|36.8%    19|    -      0
  lib/screens_web/router.ex                                                |35.7%     42|30.5%    82|    -      0
  lib/screens_web/telemetry.ex                                             | 0.0%      5| 0.0%     4|    -      0

Download coverage report

Copy link
Member

@jzimbel-mbta jzimbel-mbta left a comment

Choose a reason for hiding this comment

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

Nice, this should definitely come in handy.

@cmaddox5 cmaddox5 merged commit 1dfe895 into master Sep 28, 2023
4 checks passed
@cmaddox5 cmaddox5 deleted the cm/add-live-dashboard branch September 28, 2023 18:06
cmaddox5 added a commit that referenced this pull request Sep 28, 2023
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.

3 participants