tanwarAalok commented 1 year ago

Description

Added maxLength={10} to limit the number of digits that can be entered in Phone number and added blog link to Blogs button present in Home page.

Pull Request Checklist

Please make sure that your PR meets the following requirements:

[x] The app is working correctly after applying the changes.
[x] The app is responsive and displays properly on different screen sizes and devices.
[x] The changes are thoroughly documented in the appropriate files or documentation sections.
[x] The coding conventions and style guidelines are followed consistently.

Please check all applicable boxes and provide any additional information if needed.

[x] I have tested the changes locally and ensured that the app is functioning as expected.
[x] I have verified that the app is responsive and looks good on various screen sizes and devices.
[x] I have reviewed the documentation and ensured that it accurately reflects the changes made.
[x] I have followed the coding conventions and style guidelines defined in the project.
[x] I have run the automated tests and all of them pass.
[x] I have considered the performance implications of the changes made.
[x] I have considered the security implications of the changes made.
[x] I have checked for any potential regressions or side effects caused by the changes.
[x] I have performed a self-review of my code.
[x] Starred the repository.
[x] Added My Name to the Contributor's List.
[x] Followed the repository's Contributing Guidelines.
[x] I ran the app and tested it locally to verify that it works as expected.

Please provide any additional information or context that may be helpful for reviewing the pull request.

Please list any related issues or pull requests that are associated with this pull request.

Please tag the relevant team members or reviewers who should review this pull request.

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
project-amupedia	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 30, 2023 8:34am

AmanJain18 commented 1 year ago

@tanwarAalok Which issue u are referring ?

tanwarAalok commented 1 year ago

@tanwarAalok Which issue u are referring ?

[BUG] - Want to limit the digits that can be added to phone number in contact us form and provide link of Blogs to Blogs button in Home page #315

Sar-thak-3 commented 1 year ago

Hey @tanwarAalok I think this isn't the way to work under GSSoC or any Open Source event. You first need to be assigned for the issue.

tanwarAalok commented 1 year ago

Hey @tanwarAalok I think this isn't the way to work under GSSoC or any Open Source event. You first need to be assigned for the issue.

So what do I do now close this pull request, and get the issue assigned and then again create PR ?

AmanJain18 commented 1 year ago

Actually got these issue solved in the latest PR

AmanJain18 commented 1 year ago

Issue already got fixed... check pr #335