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

chore: add Tip Calculator #1277 #1306

Merged
merged 4 commits into from
Oct 19, 2023
Merged

Conversation

ayushsgithub
Copy link
Contributor

@ayushsgithub ayushsgithub commented Oct 6, 2023

First thing, PLEASE READ THIS: ReactPlay Code Review Checklist

Description

Adds a Tip Calculator in the ReactPlay

Fixes #1277

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Tested Manually

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

Screenshots or example output

Screenshot 2023-10-06 175720

@vercel
Copy link

vercel bot commented Oct 6, 2023

@ayushsgithub is attempting to deploy a commit to a Personal Account owned by @reactplay on Vercel.

@reactplay first needs to authorize it.

@netlify
Copy link

netlify bot commented Oct 6, 2023

Deploy Preview for reactplayio ready!

Name Link
🔨 Latest commit e10709c
🔍 Latest deploy log https://app.netlify.com/sites/reactplayio/deploys/6530b454e7571700079ff76c
😎 Deploy Preview https://deploy-preview-1306--reactplayio.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.

@priyankarpal priyankarpal changed the title feat(play/tip-calculator): add Tip Calculator #1277 chore: add Tip Calculator #1277 Oct 7, 2023
Copy link
Member

@priyankarpal priyankarpal left a comment

Choose a reason for hiding this comment

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

please make sure to rename the file extensions to jsx.

@priyankarpal priyankarpal added the changes required need to change some thing label Oct 8, 2023
priyankarpal
priyankarpal previously approved these changes Oct 13, 2023
Copy link
Member

@priyankarpal priyankarpal left a comment

Choose a reason for hiding this comment

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

Looks Good 🎉

@@ -0,0 +1,4 @@
/* enter stlyes here */
:root {
Copy link
Member

Choose a reason for hiding this comment

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

@ayushsgithub Let's not override the :root selector. Could you please add a style in your play level instead? Rest of the changes look good?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey @atapas sir, removed the root selector as per the required changes

Copy link
Member

@atapas atapas left a comment

Choose a reason for hiding this comment

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

@ayushsgithub very good work.

@atapas atapas merged commit 6fef24b into reactplay:main Oct 19, 2023
4 of 5 checks passed
@ayushsgithub
Copy link
Contributor Author

@ayushsgithub very good work.

@atapas Thank You so much sir, I'm fan of your work and content.

@ayushsgithub
Copy link
Contributor Author

@priyankarpal there's a request, can you please add hacktoberfest-accepted label to it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Add a Play]: tip calculator for bills
3 participants