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

Resolved save Post Bug #301

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

Conversation

Deepanshu0703
Copy link

Related Issue

Issue is that the saved movies deleted itself while going to different routes, also they are very slow while fetching the saved movies.

Closes: #294

Description of Changes

I have firstly fixed the itself deleting bug on moving to different routes and made it fast to fetch save posts by storing them in local cache.

For example:

  • Added fast fetching of saved movies
  • Updated SavedMovies and favourite to fix the bug

Checklist:

  • My code adheres to the established style guidelines of this project.
  • I have conducted a self-review of my code.
  • I have included comments in areas that may be difficult to understand.
  • I have made corresponding updates to the project documentation.
  • My changes have not introduced any new warnings.
Video.mp4

@vercel
Copy link

vercel bot commented Jun 23, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @sourabhsikarwar on Vercel.

@sourabhsikarwar first needs to authorize it.

@sourabhsikarwar
Copy link
Owner

Recently, the page has been updated with some features and UI changes. Kindly go through that and also resolve the merge conflicts @Deepanshu0703

@Deepanshu0703
Copy link
Author

Recently, the page has been updated with some features and UI changes. Kindly go through that and also resolve the merge conflicts @Deepanshu0703

Resolved

@Deepanshu0703
Copy link
Author

please review the resolved code @sourabhsikarwar

@Deepanshu0703
Copy link
Author

Please check the updated code @sourabhsikarwar

@Deepanshu0703
Copy link
Author

Please check the updated code @sourabhsikarwar

@sourabhsikarwar

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.

[BUG]: Error in saved Posts
2 participants