Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

IBX-7276: Renamed ibexa_get_current_user Twig function to ibexa_current_user #307

Merged
merged 1 commit into from
Dec 13, 2023
Merged

IBX-7276: Renamed ibexa_get_current_user Twig function to ibexa_current_user #307

merged 1 commit into from
Dec 13, 2023

Conversation

adamwojs
Copy link
Member

@adamwojs adamwojs commented Dec 11, 2023 •
edited
Loading

Question Answer
JIRA issue IBX-7276
Type improvement
Target Ibexa version v4.6
BC breaks no

Renamed ibexa_get_current_user Twig function to ibexa_current_user as requested by @lserwatka & @kmadejski

Checklist:

  • Provided PR description.
  • Tested the solution manually.
  • Provided automated test coverage.
  • Checked that target branch is set correctly (main for features, the oldest supported for bugs).
  • Ran PHP CS Fixer for new PHP code (use $ composer fix-cs).
  • Asked for a review (ping @ibexa/engineering).

@adamwojs adamwojs requested review from lserwatka, kmadejski and a team December 11, 2023 21:57
@sonarqubecloud SonarQubeCloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@adamwojs adamwojs merged commit ce348cb into main Dec 13, 2023
23 checks passed
@adamwojs adamwojs deleted the simplified_name branch December 13, 2023 04:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@konradoboza konradoboza konradoboza approved these changes

@mikadamczyk mikadamczyk mikadamczyk approved these changes

@lserwatka lserwatka lserwatka approved these changes

@kmadejski kmadejski Awaiting requested review from kmadejski

Assignees

@kisztof kisztof

@mikadamczyk mikadamczyk

@alongosz alongosz

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@adamwojs @lserwatka @mikadamczyk @konradoboza @kisztof @alongosz