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

code migration. #5864

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

FreakForever
Copy link

@FreakForever FreakForever commented Jun 28, 2024

refactor : migrated the class component to functional component of file StudentExercise.jsx.

@ragesoss
Copy link
Member

ragesoss commented Jul 1, 2024

I don't think these screenshots show the component you've edited.

@FreakForever
Copy link
Author

FreakForever commented Jul 8, 2024

Hey @ragesoss ,I am not able to fig out the screenshot. Pls have a look at the commit and let me know if there is any changes to be made. Pls give me the UI component location as I am not able to fig it out.

@ragesoss
Copy link
Member

ragesoss commented Jul 8, 2024

This feature is part of wiki_education: 'true' mode. You may need to do some code, UI and doc exploration to figure out where it is used and how to get test it out in your development environment. I'll mark it as a draft in the meantime, and once you've thoroughly tested it you can mark it as ready for review.

@ragesoss ragesoss marked this pull request as draft July 8, 2024 17:23
@FreakForever FreakForever marked this pull request as ready for review July 11, 2024 12:11
@FreakForever FreakForever marked this pull request as draft July 11, 2024 12:12
@FreakForever FreakForever marked this pull request as ready for review July 28, 2024 04:42
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

Successfully merging this pull request may close these issues.

2 participants