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

feat: Add checkbox to filter new challenges #405

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sadakchap
Copy link
Contributor

@sadakchap sadakchap commented Feb 26, 2024

Title and Issue number

Title : [Challenges] Add checkbox to filter new challenges

Issue No. : #400

Close #<issue_no>

Checklist:

  • I have mentioned the issue number in my Pull Request.
  • I have commented my code, particularly in hard-to-understand areas
  • I have created a helpful and easy to understand README.md
  • I have updated the Index.html file for my contribution

Copy link

netlify bot commented Feb 26, 2024

Deploy Preview for frontend-mini-challenges ready!

Name Link
🔨 Latest commit 04c322f
🔍 Latest deploy log https://app.netlify.com/sites/frontend-mini-challenges/deploys/65dce6d899d2520007624646
😎 Deploy Preview https://deploy-preview-405--frontend-mini-challenges.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@sadakchap
Copy link
Contributor Author

Something like this? @sadanandpai

fast.mov

}, [challenges, searchInput]);

setSortedChallenges(filteredChallenges);
}, [challenges, searchInput, newChallenges]);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doing it in multi-level, will that be efficient & cleaner?

filteredChallenges = filterByTitle(challenges, text)
newChallenges = filterByNew(filteredChallenges, isNew)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @sadanandpai , not sure I follow you. Could you please elaborate the point here again?

@sadanandpai
Copy link
Owner

@sadakchap conflicts

@sadanandpai
Copy link
Owner

@sadakchap let me know if you help here to get unblocked

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.

None yet

2 participants