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

fix(RSS-ECOMM-5_26): lock buttons for response #377

Merged
merged 4 commits into from
Jun 25, 2024

Conversation

stardustmeg
Copy link
Owner

@stardustmeg stardustmeg commented Jun 25, 2024

PR Title ✨

πŸ“ conforms with the following format:

  • prefix (following the convention: feat, fix, hotfix, chore, refactor, revert, docs, style, test)
  • sprint and issue number (e.g. RSS-ECOMM-5_01, where 5 - is the sprint number and 01 - is the issue number)
  • short description

πŸ‘€ Example: feat(RSS-ECOMM-5_01): description

PR Description πŸ§™β€β™‚οΈ

Affected Scope πŸ’‘

  • lock buttons for the time until server response comes

Documentation Updates πŸ“–

  • update project's README with up-to-date information

Provide any other relevant information πŸ¦‰

  • provide .env example file
  • add more detailed info for successful registration user message
  • add more detailed info for failed adding item to cart

Auto-Review πŸ€–

  • The reviewers have been requested πŸ“
    Request Review on PR Open
  • The labels have been added 🎨
    Pull Request Labeler
  • My code doesn't generate any errors or warnings ⛓️
    Continuous Integration
  • My code builds successfully βš™οΈ
    Netlify Status

Self-Check 🌟

  • I wrote the correct title for the PR πŸ—οΈ
  • I wrote a comprehensive description for the PR πŸ“œ
  • I have performed a self-review of my own code βœ…
  • I have commented my code where needed πŸ“
  • I am happy with my PR and ready to merge ❀️‍πŸ”₯

Copy link

netlify bot commented Jun 25, 2024

βœ… Deploy Preview for mad-wizards-greenshop ready!

Name Link
πŸ”¨ Latest commit 88c9b9f
πŸ” Latest deploy log https://app.netlify.com/sites/mad-wizards-greenshop/deploys/667a1beee4c4e800089561ac
😎 Deploy Preview https://deploy-preview-377--mad-wizards-greenshop.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.

@stardustmeg stardustmeg marked this pull request as draft June 25, 2024 01:02
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jun 25, 2024
@stardustmeg stardustmeg marked this pull request as ready for review June 25, 2024 01:25
Copy link
Collaborator

@Kleostro Kleostro left a comment

Choose a reason for hiding this comment

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

Great!❀️‍πŸ”₯

@stardustmeg stardustmeg merged commit 80790a1 into develop Jun 25, 2024
7 checks passed
@stardustmeg stardustmeg deleted the fix(RSS-ECOMM-5_26)/lockButtonsForResponse branch June 25, 2024 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation fix sprint5 style
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants