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.

Sign up for GitHub

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

Jump to bottom

Simplify TopicsPage component #10415

Open
MisRob opened this issue Apr 7, 2023 · 3 comments
Open

Simplify TopicsPage component #10415

MisRob opened this issue Apr 7, 2023 · 3 comments
Labels
APP: Learn Re: Learn App (content, quizzes, lessons, etc.) DEV: frontend TAG: tech update / debt Change not visible to user

Comments

@MisRob
Copy link
Member

MisRob commented Apr 7, 2023

Understanding and updating the TopicsPage component is time-consuming and inefficient due to the size of the component, many conditions in the template as well as lots of business JavaScript logic.

It'd be good to simplify it, for example by splitting it into more components to address the template parts and using composables or xState to address JavaScript parts.
@MisRob MisRob added TAG: tech update / debt Change not visible to user APP: Learn Re: Learn App (content, quizzes, lessons, etc.) DEV: frontend labels Apr 7, 2023
@MisRob MisRob mentioned this issue Apr 7, 2023
Merged
9 tasks
@Wck-iipi
Copy link
Contributor

@MisRob Can I work on this issue?

@MisRob
Copy link
Member Author

MisRob commented Apr 21, 2023

Hello, @Wck-iipi, I would rather recommend issues with the "help wanted" tag

@Wck-iipi
Copy link
Contributor

@MisRob Thanks for responding. I will find help wanted issue and try to work on it.

@MisRob MisRob mentioned this issue May 9, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
APP: Learn Re: Learn App (content, quizzes, lessons, etc.) DEV: frontend TAG: tech update / debt Change not visible to user
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MisRob @Wck-iipi