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

feat: add pizza-menu play #1571

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aaqib605
Copy link
Contributor

@aaqib605 aaqib605 commented Nov 5, 2024

Description

A pizza menu that showcases a variety of pizzas, including detailed information such as ingredients, prices, and availability. Each pizza listing clearly indicates if it’s currently sold out, helping customers make quick and informed choices.

Type of change

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

How Has This Been Tested?

Manually

Checklist:

  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes

Screenshots or example output

cover

Copy link

netlify bot commented Nov 5, 2024

Deploy Preview for reactplayio ready!

Name Link
🔨 Latest commit 028cf65
🔍 Latest deploy log https://app.netlify.com/sites/reactplayio/deploys/672a22ea2f65910008d0eec3
😎 Deploy Preview https://deploy-preview-1571--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.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hey! contributor, thank you for opening a Pull Request 🎉.

@reactplay/maintainers will review your submission soon and give you helpful feedback. If you're interested in continuing your contributions to open source and want to be a part of a welcoming and fantastic community, we invite you to join our ReactPlay Discord Community.
Show your support by starring ⭐ this repository. Thank you and we appreciate your contribution to open source!
Stale Marking : After 30 days of inactivity this issue/PR will be marked as stale issue/PR and it will be closed and locked in 7 days if no further activity occurs.

Comment on lines +3 to +7
* {
margin: 0;
padding: 0;
box-sizing: border-box;
}
Copy link
Member

Choose a reason for hiding this comment

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

please avoid targeting directly

Comment on lines +9 to +11
html {
font-size: 62.5%;
}
Copy link
Member

Choose a reason for hiding this comment

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

same as the above ^^

@priyankarpal priyankarpal added the changes required need to change some thing label Nov 11, 2024
@aaqib605
Copy link
Contributor Author

aaqib605 commented Nov 11, 2024

@priyankarpal I need help. Once i checkout to play/pizza-menu it complains about steps play not found. Why is that? Steps play is in a separate branch. This is a blocker. I can't see the results of the changes. Please can you look into this?
Same happens in the play/bio-card branch.

Screenshot 2024-11-11 at 6 39 28 PM Screenshot 2024-11-11 at 6 40 24 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes required need to change some thing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants