WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@FlxRobole
Copy link

Q A
License MIT

What's in this PR?

  • Removes wrong information about optional argument locale in sulu_categories twig extension
  • Adds type information in arguments section

Why?

Not passing the locale argument throws an error as it is mandatory (see code)

vinceAmstoutz and others added 30 commits November 25, 2022 13:33
* add documentation for global-blocks

* fix code-block

* fix code-review

* Update book/templates.rst

Co-authored-by: Prokyonn <[email protected]>

---------

Co-authored-by: Prokyonn <[email protected]>
* Extend Docs for Admin View Debug Command

* Update book/extend-admin.rst

Co-authored-by: Prokyonn <[email protected]>

---------

Co-authored-by: Prokyonn <[email protected]>
* Add reference bundle docs

* Add review suggestion

* Add review suggestion
* add cleanup phpcr database

* Apply suggestions from code review

Co-authored-by: Prokyonn <[email protected]>

---------

Co-authored-by: Prokyonn <[email protected]>
@alexander-schranz
Copy link
Member

@FlxRobole thx, can you target the current 2.5 branch for this fix?

@FlxRobole FlxRobole changed the base branch from 2.6 to 2.5 February 10, 2025 07:53
@FlxRobole
Copy link
Author

@alexander-schranz base branch updated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants