Skip to content

add * for mandatory field#730

Merged
nicosomb merged 2 commits intoPrestaShop:developfrom
MonsieurRb:feat/required-field
Jul 2, 2025
Merged

add * for mandatory field#730
nicosomb merged 2 commits intoPrestaShop:developfrom
MonsieurRb:feat/required-field

Conversation

@MonsieurRb
Copy link
Contributor

Questions Answers
Description? add * for mandatory field
Type? bug fix
BC breaks? no
Deprecations? no
Fixed ticket? Fixes #728
Sponsor company @PrestaShopCorp
How to test? --

@ps-jarvis
Copy link

Hello @MonsieurRb!

This is your first pull request on hummingbird repository of the PrestaShop project.

Thank you, and welcome to this Open Source community!

@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Jun 26, 2025
@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Jun 26, 2025
Copy link

@SiraDIOP SiraDIOP left a comment

Choose a reason for hiding this comment

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

Hello @MonsieurRb,

Do we need qa or qa by dev tests?

Thank you

@ps-jarvis ps-jarvis moved this from To be tested to Waiting for PM/UX/Dev in PR Dashboard Jun 29, 2025
@paulnoelcholot paulnoelcholot self-assigned this Jul 2, 2025
@paulnoelcholot
Copy link

Hello @MonsieurRb,

I tested your PR and it's good for me! Thanks !

@nicosomb nicosomb added this to the v2.0.0 milestone Jul 2, 2025
@nicosomb nicosomb merged commit 867f0b8 into PrestaShop:develop Jul 2, 2025
6 checks passed
@github-project-automation github-project-automation bot moved this from Waiting for PM/UX/Dev to Merged in PR Dashboard Jul 2, 2025
@ps-jarvis ps-jarvis moved this from Merged to Waiting for PM/UX/Dev in PR Dashboard Jul 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Missing * when required form label

7 participants