Skip to content

Fix 500 in analytics logging from missing constant#6300

Merged
aduth merged 1 commit intomainfrom
aduth-500-submit
May 4, 2022
Merged

Fix 500 in analytics logging from missing constant#6300
aduth merged 1 commit intomainfrom
aduth-500-submit

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented May 4, 2022

Why: So that the build passes, and so that we don't have 500 errors.

Context: https://github.com/18F/identity-idp/pull/6288/files#r864759689

There's probably a more elegant way to implement this than to repeat the event name string, but I'm most concerned with restoring the passing build.

**Why**: So that the build passes, and so that we don't have 500 errors.

Context: https://github.com/18F/identity-idp/pull/6288/files#r864759689

[skip changelog]
@aduth aduth merged commit 0ca3e50 into main May 4, 2022
@aduth aduth deleted the aduth-500-submit branch May 4, 2022 13:06
mitchellhenke pushed a commit that referenced this pull request May 4, 2022
**Why**: So that the build passes, and so that we don't have 500 errors.

Context: https://github.com/18F/identity-idp/pull/6288/files#r864759689

[skip changelog]
peggles2 pushed a commit that referenced this pull request May 5, 2022
**Why**: So that the build passes, and so that we don't have 500 errors.

Context: https://github.com/18F/identity-idp/pull/6288/files#r864759689

[skip changelog]
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