ref(settings): migrate redeemPromoCode page to new form system#113789
ref(settings): migrate redeemPromoCode page to new form system#113789
Conversation
github-actions
Bot
added
the
Scope: Frontend
| // non-field errors can be camelcase or snake case | ||
| const nonFieldErrors = | ||
| error.responseJSON?.non_field_errors || error.responseJSON?.nonFieldErrors; | ||
|
|
||
| if (Array.isArray(nonFieldErrors) && nonFieldErrors.length) { | ||
| addErrorMessage(nonFieldErrors[0], {duration: 10000}); | ||
| } |
There was a problem hiding this comment.
@evanpurkhiser I think this is what you hinted at in the last PR? We should somehow have this be respected everywhere, right ?
There was a problem hiding this comment.
Yes. This idea that they are camel or snake isn’t random though, that’s dependent on if the serualizer is a CamelCaseSerializer or not
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 007f26e. Configure here.
Everything seems to be working well, but are we sure we want the primary button on the same row as the balance info? |
This is how it’s in the current version too 🤷 |
Co-authored-by: Priscila Oliveira <priscila.oliveira@sentry.io>
|
PR reverted: 9fcf3e3 |
4 participants
No description provided.