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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃悰 Bug Report: Double typo in credit card enum #8062

Closed
2 tasks done
iron3oxide opened this issue May 1, 2024 · 1 comment 路 Fixed by #8074
Closed
2 tasks done

馃悰 Bug Report: Double typo in credit card enum #8062

iron3oxide opened this issue May 1, 2024 · 1 comment 路 Fixed by #8074
Assignees
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@iron3oxide
Copy link

馃憻 Reproduction steps

Looking at the OpenAPI / Swagger specs and the code generated with it, I noticed that "cencosud" was misspelled as "censosud", which was then again misspelled as "consosud". I'd put a PR in myself but unfortunately I don't yet understand how the specs are generated.

馃憤 Expected behavior

It should say "cencosud" everywhere

馃憥 Actual Behavior

See above

馃幉 Appwrite version

Version 1.5.x

馃捇 Operating system

Linux

馃П Your Environment

No response

馃憖 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

馃彚 Have you read the Code of Conduct?

@iron3oxide iron3oxide added the bug Something isn't working label May 1, 2024
@stnguyen90
Copy link
Contributor

@iron3oxide, thanks for creating this issue! 馃檹馃徏 There are a couple places where this needs to be updated:

  1. https://github.com/appwrite/appwrite/blob/a30d06bc585390658f2ef6677267756852c9a44e/app/config/avatars/credit-cards/censosud.png
  2. 'censosud' => ['name' => 'Consosud', 'path' => __DIR__ . '/credit-cards/consosud.png'],

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants