Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Adding Profiler to Debug Performance #2760

Merged
merged 1 commit into from
Sep 22, 2023
Merged

Adding Profiler to Debug Performance #2760

merged 1 commit into from
Sep 22, 2023

Conversation

jasperfurniss
Copy link
Contributor

What does this PR do? A clear and concise description with your runway ticket url.
This a next step because we are still experiencing 500 errors, and we need a basic profiler added in order to help us look under the hood and see what the bottlenecks are.

Screenshots: Screenshots to visualize your addition/change

How to test? Steps to confirm the desired behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See addition/change

Checklist:

  • LABELS Add a label: enhancement, bug, improvement, new kit, deprecated, or breaking. See Changelog & Labels for details.
  • DEPLOY I have added the milano label to show I'm ready for a review.
  • TESTS I have added test coverage to my code.

@jasperfurniss jasperfurniss requested a review from a team as a code owner September 22, 2023 22:14
@jasperfurniss jasperfurniss self-assigned this Sep 22, 2023
@jasperfurniss jasperfurniss added the milano 20 MAX - Deploy this PR to a review environment via Milano label Sep 22, 2023
@jasperfurniss jasperfurniss added this pull request to the merge queue Sep 22, 2023
Merged via the queue into master with commit 9215228 Sep 22, 2023
@jasperfurniss jasperfurniss deleted the profiler branch September 22, 2023 23:43
@sentry-io
Copy link

sentry-io bot commented Sep 23, 2023

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ **NameError: undefined local variable or method current_user' for #<PagesController:0x0000000000a370>** PagesController#kit_show_rails` View Issue

Did you find this useful? React with a 👍 or 👎

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
milano 20 MAX - Deploy this PR to a review environment via Milano
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants