This PR optimizes the getParams function to prevent the use of needless statements (such as while loops). With less statements, the function will block the event loop less, and run smoother.
Note: The above is the primary reason for this PR. While I was implementing this, I also ran the code through the prettier tool.
Testing recommendations
This function should not require any changes to the original codebase, and will not provide any front-end change. To test these, try running it with varying formats of query parameters.
GitHub issue number
This fixes no issues, but does end up resolving the comment // I think this could be combined into one if.
Related Pull Requests
N/A
Checklist
[x] Code is written and works correctly;
[ ] Changes are covered by tests;
This shouldn't require any changes to existing tests, but I've left this unchecked until you have confirmed
[ ] Documentation reflects the changes;
This shouldn't require any changes to existing documentation, but I've left this unchecked until you have confirmed
[ ] Update rebar.config.script with the correct tag once a new Fauxton release is made
RedYetiDev
commented
3 months ago
Overview
This PR optimizes the
getParamsfunction to prevent the use of needless statements (such as while loops). With less statements, the function will block the event loop less, and run smoother.Testing recommendations
This function should not require any changes to the original codebase, and will not provide any front-end change. To test these, try running it with varying formats of query parameters.
GitHub issue number
This fixes no issues, but does end up resolving the comment
// I think this could be combined into one if.Related Pull Requests
N/A
Checklist