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

Phone pe #365

Closed
wants to merge 5 commits into from
Closed

Phone pe #365

wants to merge 5 commits into from

Conversation

Damini2004
Copy link
Contributor

Closes: #314

  • Title : Phonepe Clone
  • Name: : Damini Chachane
  • Idenitfy yourself: : GSSOC

Describe the add-ons or changes you've made πŸ“ƒ

How Has This Been Tested? βš™οΈ

Screenshots πŸ“·

Type of change β˜‘οΈ

What sort of change have you made:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, local variables)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist: β˜‘οΈ

  • My code follows the Contributing Guidelines & Code of Conduct of this project.
  • This PR does not contain plagiarized content.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly wherever it was hard to understand.
  • My changes generate no new warnings.

Note to reviewers πŸ“„

Copy link

vercel bot commented Jun 20, 2024

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
web-master-log βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Jun 22, 2024 4:07pm

Copy link

Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE.

@Avdhesh-Varshney Avdhesh-Varshney added bug πŸ› Something isn't working Requested Changes βš™οΈ Some changes have been requested in this PR labels Jun 21, 2024
@GAVINESHWAR
Copy link
Contributor

GAVINESHWAR commented Jun 21, 2024

@Damini2004 I am getting error while using npm start

PS C:\Users\Avineshwar G\Documents\internship\opensource\WebMasterLog> npm start
npm ERR! Missing script: "start"
npm ERR!
npm ERR! Did you mean one of these?
npm ERR! npm star # Mark your favorite packages
npm ERR! npm stars # View packages marked as favorites
npm ERR!
npm ERR! To see a list of scripts, run:
npm ERR! npm run

npm ERR! A complete log of this run can be found in: C:\Users\Avineshwar G\AppData\Local\npm-cache_logs\2024-06-21T05_25_24_324Z-debug-0.log

Can you say how you open the project

@Damini2004
Copy link
Contributor Author

@GAVINESHWAR The project is designed with the help of HTML, CSS not in react, node js so no need to run command of "npm start" in order to run just run the index.html file

@Damini2004
Copy link
Contributor Author

@Avdhesh-Varshney Kindly review the PR

@Damini2004
Copy link
Contributor Author

@Avdhesh-Varshney Can you increase the level.

Copy link
Owner

Choose a reason for hiding this comment

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

you have updated the README.md but not updating the name of the file?

@Avdhesh-Varshney Avdhesh-Varshney added wontfix ❌ This will not be worked on and removed Requested Changes βš™οΈ Some changes have been requested in this PR labels Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug πŸ› Something isn't working wontfix ❌ This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

πŸ“ƒ: Phonepe landing page
3 participants