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

Added categories #24

Merged
merged 1 commit into from
Oct 19, 2021
Merged

Conversation

IgorChernyshov
Copy link
Contributor

@IgorChernyshov IgorChernyshov commented Oct 12, 2021

Links were not fetched by specific category when user wanted to browse some specific category. The result was any category displayed all the items.
Fixed this by adding an NSPredicate to the fetch request.

P.S. I'd appreciate if you put the hacktoberfest-accepted label to my PR to count it towards the Hacktoberfest goal.

@acmbot
Copy link

acmbot bot commented Oct 12, 2021

Thanks @IgorChernyshov, for opening the pull request! 🙌
One of our team-mates will review the pull request soon. ✅

@IgorChernyshov
Copy link
Contributor Author

Could you kindly review this PR, please?

@Samridh29 Samridh29 merged commit 5524ca7 into ACM-VIT:main Oct 19, 2021
@acmbot
Copy link

acmbot bot commented Oct 19, 2021

Congratulations @IgorChernyshov, your pull request is merged! 🎉
Thanks for your contributions. 🙌

@Samridh29
Copy link
Collaborator

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants