Skip to content

Release 2.3.5 #84

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
Merged
nicosomb merged 4 commits into from
Oct 10, 2025
Merged

Release 2.3.5 #84

nicosomb merged 4 commits into from
Oct 10, 2025

Conversation

@nicosomb
Copy link
Contributor

@nicosomb nicosomb commented Oct 10, 2025

bibips and others added 4 commits March 24, 2025 08:45
@bibips
Add hook actionMainMenuModifier
ff78da0 
Thanks this hook it's possible to alter main menu. For instance you can
display menu items per customer or per group of customer
@boherm
Merge pull request #70 from bibips/add-hook-displaymainmenu
173ea0d
@Codencode
Added hook "ActionMetaPageSave" for cache deletion
888269a
@nicosomb
Merge pull request #80 from Codencode/fix-38697-Schema-of-URLs-of-cat…
1d8e881 
…egory-route-does-not-redirect-to-the-right-page

Fix: Schema of URLs of category route does not redirect to the right page
@nicosomb nicosomb added this to the 2.3.5 milestone Oct 10, 2025
@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Oct 10, 2025
@AureRita AureRita self-assigned this Oct 10, 2025
@tleon
Copy link

tleon commented Oct 10, 2025

The CI is broken because of this. The fixes are still in reviews and not merged yet.

I recommend that we merge this regardless of the red phpstan job an then, once merge, we open a new PR like this one
so we can test the phpstan job.

AureRita
AureRita approved these changes Oct 10, 2025
View reviewed changes
Copy link

@AureRita AureRita left a comment

Choose a reason for hiding this comment

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

Hi @nicosomb

Thank you for your PR, I tested it and it seems to works as you can see :

Capture d’écran du 2025-10-10 10-48-10
Capture.video.du.2025-10-10.10-58-41.mp4

Tested on :
8.2.3
9.0.1
9.1.0

Because the PR seems to works as expected, It's QA ✔️

Thank you

@nicosomb nicosomb merged commit c81a1af into master Oct 10, 2025
45 of 48 checks passed
@github-project-automation github-project-automation bot moved this from Ready for review to Merged in PR Dashboard Oct 10, 2025
@ps-jarvis ps-jarvis moved this from Merged to Ready for review in PR Dashboard Oct 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tleon tleon tleon approved these changes

@AureRita AureRita AureRita approved these changes

Assignees

@AureRita AureRita

Labels

QA ✔️

Projects

PR Dashboard
Archived in project

Milestone

2.3.5

Development

Successfully merging this pull request may close these issues.

8 participants

@nicosomb @tleon @AureRita @ps-jarvis @bibips @boherm @Codencode