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

Improved error message for duplicate code application #880

Conversation

mayank-sahai
Copy link
Contributor

Issue link / number:

#816

What changes did you make?

Handled the new ALREADY_APPLIED status error to show a relevant error message in all supported languages.

Why did you make the changes?

To improve the error message displayed to users when they attempt to apply a code that has already been used with their account.

This commit improves the error message displayed to users when they attempt to apply a code that has already been used with their account. Different language support is also added for the same
Copy link

vercel bot commented Apr 25, 2024

@mayank-sahai is attempting to deploy a commit to the Chayn Team on Vercel.

A member of the Team first needs to authorize it.

@kyleecodes kyleecodes self-assigned this Apr 26, 2024
@kyleecodes
Copy link
Member

Thanks @mayank-sahai this looks great! 👏
The dev team is off until Monday and will review then.

@eleanorreem
Copy link
Contributor

eleanorreem commented Apr 29, 2024

Hey @mayank-sahai this looks great thank you. Don't worry about the failing tests - it is failing because of credential issues on our side. We will sort out asap and get this merged. Thank you again ⭐

@mayank-sahai
Copy link
Contributor Author

cool! thank you 😊

@eleanorreem eleanorreem merged commit fa9ec6f into chaynHQ:develop Apr 30, 2024
2 of 4 checks passed
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.

None yet

3 participants