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

Enhanced the UI for NewsApp and add User Authentication #1230

Merged
merged 2 commits into from
Aug 10, 2024

Conversation

Inferno2211
Copy link
Contributor

Hi @apu52

Title : Enhanced the UI for NewsApp and add User Authentication

Closes #1116

Video

demo.mp4

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 gone through the contributing.md file before contributing

Additional context

Are you contributing under any Open-source programme?

  • I'm a GSSOC'24 contributor
  • I'm a SSOC'24 contributor

Copy link

github-actions bot commented Aug 7, 2024

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

Copy link

netlify bot commented Aug 7, 2024

Deploy Preview for cute-quokka-b093d8 ready!

Name Link
🔨 Latest commit 5cb3934
🔍 Latest deploy log https://app.netlify.com/sites/cute-quokka-b093d8/deploys/66b38086d3c7530008597d3c
😎 Deploy Preview https://deploy-preview-1230--cute-quokka-b093d8.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.

Copy link
Owner

@apu52 apu52 left a comment

Choose a reason for hiding this comment

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

@Inferno2211 the deployment u have made in react can't be shown in the website as u have made it by using the react. But I have reviewed your code & its working well ...Just inform me
that if u can see your changes in the project in this deployed link of your PR or not??

https://deploy-preview-1230--cute-quokka-b093d8.netlify.app/

@apu52 apu52 added the question Further information is requested label Aug 9, 2024
@Inferno2211
Copy link
Contributor Author

@apu52 No I can't see my app in the deploy

@apu52
Copy link
Owner

apu52 commented Aug 10, 2024

@apu52 No I can't see my app in the deploy

hmmm..okk but I am merging your PR , don't worry

@apu52 apu52 merged commit b3508c1 into apu52:main Aug 10, 2024
5 checks passed
@apu52 apu52 added enhancement New feature or request gssoc level3 and removed question Further information is requested labels Aug 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request gssoc level3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Enhance the UI for NewsApp and add User Authentication
2 participants