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

Update slug #35667

Merged
merged 1 commit into from
Jan 23, 2025
Merged

Update slug #35667

merged 1 commit into from
Jan 23, 2025

Conversation

jingcheng16
Copy link
Contributor

@jingcheng16 jingcheng16 commented Jan 23, 2025

Product Description

I renamed API_USAGE to API_KEYS in #35622 but missed the update in enterprise/view.py. This PR is to update the slug.

Technical Summary

Feature Flag

Safety Assurance

Safety story

Put it on staging and verified the Security Center page no longer 500ed.

Automated test coverage

QA Plan

Rollback instructions

  • This PR can be reverted after deploy with no further considerations

Labels & Review

  • Risk label is set correctly
  • The set of people pinged as reviewers is appropriate for the level of risk of the change

Copy link

sentry-io bot commented Jan 23, 2025

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: corehq/apps/enterprise/views.py

Function Unhandled Issue
security_center AttributeError: type object 'EnterpriseReport' has no attribute 'API_USAGE' /a/{domain}/...
Event Count: 1

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

@jingcheng16 jingcheng16 marked this pull request as ready for review January 23, 2025 02:15
@jingcheng16 jingcheng16 requested review from biyeun and mjriley January 23, 2025 02:22
@AmitPhulera AmitPhulera merged commit f21a0a1 into master Jan 23, 2025
14 checks passed
@AmitPhulera AmitPhulera deleted the jc/fix-slug branch January 23, 2025 02:54
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