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

Migration to TS & overall refactoring #9

Open
aiman-al-masoud opened this issue Mar 17, 2023 · 1 comment
Open

Migration to TS & overall refactoring #9

aiman-al-masoud opened this issue Mar 17, 2023 · 1 comment
Assignees

Comments

@aiman-al-masoud
Copy link
Owner

So I just started working on the add-typescript/ branch. I will simultaneously make some major changes to the overall architecture of the program, to better handle state-management.

The aim is to remove the various singletons & static stuff (Language, Settings, UserProgress, etc...), replacing them with a single Context facade to be passed down to all children from the app's root, as a centralized "source of truth" for the state throughout the app.

It may take me a while to migrate all of the codebase to TS, but I will probably merge to main as soon as I get most of the architecture stuff sorted out, even if there be still some marginal parts in JS.

Another thing I'd like to do is adding some automated tests, eventually, but probably not very soon.

Thanks!

@aiman-al-masoud aiman-al-masoud self-assigned this Mar 17, 2023
@aiman-al-masoud
Copy link
Owner Author

Mostly done! still some issues at hand, detailed in TODOs.

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

No branches or pull requests

1 participant