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.

Sign up for GitHub

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

Jump to bottom

IBX-6136: Changed value validation logic for anchor #236

Merged
merged 1 commit into from
Jul 19, 2023
Merged

IBX-6136: Changed value validation logic for anchor #236

merged 1 commit into from
Jul 19, 2023

Conversation

mateuszdebinski
Copy link
Contributor

Question Answer
JIRA issue IBX-6136
Bug/Improvement yes
New feature no
Target version v3.3
BC breaks no
Tests pass yes
Doc needed no

TODO:

  • Implement feature / fix a bug.
  • Implement tests.
  • Fix new code according to Coding Standards ($ composer fix-cs).
  • Ask for Code Review.

@mateuszdebinski mateuszdebinski added Bug Something isn't working Ready for review labels Jul 7, 2023
@mateuszdebinski mateuszdebinski requested a review from a team July 7, 2023 09:26
@mateuszdebinski mateuszdebinski self-assigned this Jul 7, 2023
@GrabowskiM GrabowskiM requested review from a team, dew326, lucasOsti, Gengar-i and tischsoic and removed request for a team July 7, 2023 12:47
@micszo micszo self-assigned this Jul 10, 2023
micszo
micszo approved these changes Jul 18, 2023
View reviewed changes
Copy link
Member

@micszo micszo left a comment

Choose a reason for hiding this comment

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

QA Approved on Ibexa Commerce 3.3.34-dev.

@micszo micszo removed their assignment Jul 18, 2023
@dew326 dew326 merged commit a2d97eb into 2.3 Jul 19, 2023
@dew326 dew326 deleted the IBX-6136_quotation_marks_error_anchor branch July 19, 2023 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gengar-i Gengar-i Gengar-i approved these changes

@lucasOsti lucasOsti lucasOsti approved these changes

@micszo micszo micszo approved these changes

@dew326 dew326 dew326 approved these changes

@GrabowskiM GrabowskiM GrabowskiM approved these changes

@tischsoic tischsoic Awaiting requested review from tischsoic

Assignees

@alongosz alongosz

@dew326 dew326

@mateuszdebinski mateuszdebinski

Labels
Bug Something isn't working QA approved
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@mateuszdebinski @GrabowskiM @dew326 @micszo @lucasOsti @Gengar-i @alongosz