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

Migrated code from Class based component to Function based component of Meetings component #5832

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

Conversation

shiven16
Copy link
Contributor

@shiven16 shiven16 commented Jun 16, 2024

NOTE: Please review the pull request process before opening your first PR: https://github.com/WikiEducationFoundation/WikiEduDashboard/blob/master/CONTRIBUTING.md#pull-request-process

What this PR does

This pull request refactors the Meetings component from a class-based component to a functional component.

##Screenshots

Before:

1stpr.before.mov

After:

1st.pr.after.mov

Open questions and concerns

This is my first time contributing to the Wiki Education Foundation. Refactoring has improved my understanding of code organization. If you find any errors or poor coding practices, please let me know and provide feedback.

@ragesoss
Copy link
Member

Please include before/after screenshots.

@shiven16 shiven16 changed the title MIgrated code from Class based component to Function based component of Meetings component Migrated code from Class based component to Function based component of Meetings component Jul 7, 2024
@shiven16 shiven16 reopened this Jul 7, 2024
@shiven16
Copy link
Contributor Author

shiven16 commented Jul 7, 2024

Hey @ragesoss I have updated my code and uploaded the screen recordings for the same.Can you please review it again.

@ragesoss
Copy link
Member

Is this spec failing for you locally as well? It may be related to your change.

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