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.

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

IBX-1689: Change all ez- classes/IDs and eZ variables in frontend part #7

Merged
merged 1 commit into from
Dec 21, 2021

Conversation

GrabowskiM
Copy link
Contributor

@GrabowskiM GrabowskiM commented Dec 13, 2021

Question Answer
JIRA issue IBX-1689
Type improvement
Target Ibexa version v4.0
BC breaks yes

Changed all eZ/ez- frontend classes/variables to ibexa

Related PRs:

Checklist:

  • Provided PR description.
  • Tested the solution manually.
  • Checked that target branch is set correctly (main for features, the oldest supported for bugs).
  • Asked for a review (ping @ibexa/engineering).

<li class="ez-fielddefinition-setting query-type">
<div class="ez-fielddefinition-setting-name">{{ 'ezcontentquery.query_type.label'|trans|desc("Query type:")}}</div>
<div class="ez-fielddefinition-setting-value">
<li class="ibexa-fielddefinition-setting query-type">

Choose a reason for hiding this comment

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

Suggested change
<li class="ibexa-fielddefinition-setting query-type">
<li class="ibexa-field-definition-setting query-type">

?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Outside scope, I don't want to do any more changes than renaming ez/ezplatform to ibexa it his task

@GrabowskiM GrabowskiM requested a review from lucasOsti December 17, 2021 10:55
@dew326 dew326 merged commit a4d30f6 into main Dec 21, 2021
@dew326 dew326 deleted the IBX-1689-rename-ez-to-ibexa-front branch December 21, 2021 14:11
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.

4 participants