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

Put all IOfs function into IOfs{} #71

Open
Maingron opened this issue Sep 21, 2023 · 1 comment
Open

Put all IOfs function into IOfs{} #71

Maingron opened this issue Sep 21, 2023 · 1 comment
Labels

Comments

@Maingron
Copy link
Owner

At this point, we have way too many and badly-named IOfs functions, which just results in absolute chaos.

The upcoming, temporary solution will be to just put all of these into IOfsV1{} (or something alike). (Not IOfs{}!)

Step 2 will be to refactor all IOfs functions (names, parameters, results) so we get an unique system and put them into IOfsV2{} = IOfs. At this point we might think about keeping IOfsV1, maybe in a separate file, for now.

Step 3 will be to refactor all IOfs functions internally, which presumably won't change names, parameters or results, but may add to them. Step 3 may be delayed for good.

Step 4 will be to move from IOfsV2{} to just IOfs{} and remove IOfsV1{} completely. At this point, the existing functions shouldn't break anymore, but Step 4 should probably be reconsidered, also since we may likely eventually upgrade to IOfsV3 (or it may be necessary to go to IOfsV2_1) or something. IOfsV3 would probably change the storage system from localStorage to something bigger and better.

@Maingron Maingron added the IOfs label Sep 21, 2023
@Maingron
Copy link
Owner Author

We now have IOfs{} (I think it's IOfsv2{}). It's better than what we had before, and it's working pretty great. Could probably use some optimisations though.

Please use the new IOfsv2 (?) functions. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant