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

ArticlefinderRow refactor #5497

Conversation

justiceotuya
Copy link
Contributor

@justiceotuya justiceotuya commented Oct 4, 2023

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

refactors article finder to use functional component from this Issue #5393

Screenshots

Before:
image

After:
image

Open questions and concerns

< anything you learned that you want to share, or questions you're wondering about related to this PR >

@justiceotuya justiceotuya changed the title Article finder row refactor [WIP] ArticlefinderRow refactor Oct 4, 2023
@justiceotuya justiceotuya changed the title [WIP] ArticlefinderRow refactor ArticlefinderRow refactor Oct 4, 2023
@justiceotuya
Copy link
Contributor Author

@ragesoss here. this is part of my contribution as an outreach intern, please add me to the slack group so I can ask questions

@justiceotuya
Copy link
Contributor Author

@Aminehassou

@ragesoss
Copy link
Member

ragesoss commented Oct 4, 2023

@ragesoss here. this is part of my contribution as an outreach intern, please add me to the slack group so I can ask questions

Sure, please send me an email (sage at wikiedu.org) so I know where to send the invite.

@ragesoss
Copy link
Member

ragesoss commented Dec 4, 2023

@justiceotuya are you still working on this? (it looks like there are some article finder specs failing; if you're still interested in getting this merged but you are stuck with something, let us know!)

@ragesoss ragesoss marked this pull request as draft December 4, 2023 20:14
@ragesoss ragesoss closed this Jun 20, 2024
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