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.

Sign up for GitHub

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

Jump to bottom

Movie Details Page and Loading Effect #5

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

Movie Details Page and Loading Effect #5

wants to merge 3 commits into from
+4,998 −42

Conversation

RaghavOG
Copy link

@RaghavOG RaghavOG commented Oct 2, 2024

WhatsApp Image 2024-10-02 at 13 44 53_551a7f45

Movie Details Page

@RaghavOG RaghavOG mentioned this pull request Oct 2, 2024
Open
@kshitizvardhan
Copy link
Member

@RaghavOG
Please make sure to limit your changes to the feature you are working on. I noticed that you made some unnecessary changes to app.css and added an .env file, which were not required for this task. Kindly revert those changes.

Once corrected, please upload a video of the feature as well. Thanks!

@RaghavOG
Copy link
Author

RaghavOG commented Oct 8, 2024

I have done the Changes

searchMovies.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@RaghavOG @kshitizvardhan