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

Consistency in server-side script names #861

Open
adkinsrs opened this issue Aug 9, 2024 · 0 comments
Open

Consistency in server-side script names #861

adkinsrs opened this issue Aug 9, 2024 · 0 comments
Assignees
Labels
code cleanup Fix the ugly, inefficient or just plain bad. Low Priority Not the biggest concern at the moment. Also includes "nice to haves"

Comments

@adkinsrs
Copy link
Member

adkinsrs commented Aug 9, 2024

Was working on another ticket, and I noticed that a lot of the server-side scripts that do the same type of function (i.e. delete), tend to have varying prefixes around that same name (i.e. delete_this, remove_this, etc.). This may lead to a false sense that a CRUD-based script is not present if you are looking for particular verbage (such as "delete" only) and a lot of extra searching or at worst starting a new script when it isn't needed.

For consistency's sake, I think we should try to rename scripts to have the same verbage (create, read, update, delete or some agreed upon derivatives). We could technically do the same with client-side functions as well, but that's probably overkill.

@adkinsrs adkinsrs added code cleanup Fix the ugly, inefficient or just plain bad. Low Priority Not the biggest concern at the moment. Also includes "nice to haves" labels Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code cleanup Fix the ugly, inefficient or just plain bad. Low Priority Not the biggest concern at the moment. Also includes "nice to haves"
Projects
None yet
Development

No branches or pull requests

2 participants