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

Remove erroneous naming for things like TransactionEvent #126

Closed
ejmg opened this issue Jul 3, 2024 · 0 comments
Closed

Remove erroneous naming for things like TransactionEvent #126

ejmg opened this issue Jul 3, 2024 · 0 comments
Labels
documentation Improvements or additions to documentation P-Low Low priority relative to all other tasks.

Comments

@ejmg
Copy link
Collaborator

ejmg commented Jul 3, 2024

Description

Title mostly says it. From the earliest versions of the app, there was naming choices that were made largely off of misunderstanding the domain at the time. Transaction Events is a big example but there may be others. Removing these from the actual UI are a high priority but it would be good to remove any and all such names from the code base (validators, queried key names returned by the API, etc).

@ejmg ejmg added documentation Improvements or additions to documentation P-Low Low priority relative to all other tasks. labels Jul 3, 2024
@ejmg ejmg closed this as completed Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation P-Low Low priority relative to all other tasks.
Projects
No open projects
Status: Done
Development

No branches or pull requests

1 participant