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

fix: tooltip error for not editable many to many relation #704

Conversation

kjkooistra-youwe
Copy link
Contributor

Error: application style and tooltip aborted, nor matching element found.
Caused by missing object_field class, which is used by edit.js to find the relevant element to apply the tooltip to.
Fixed by adding class(es) to component in getLayoutShow.

Error: application style and tooltip aborted, nor matching element found.
Caused by missing object_field class, which is used by edit.js to find the relevant element to apply the tooltip to.
Fixed by adding class(es) to component in getLayoutShow.
Copy link

sonarcloud bot commented Sep 27, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
E Maintainability Rating on New Code (required ≥ B)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@kingjia90 kingjia90 changed the base branch from 1.5 to 1.6 October 1, 2024 08:59
@mattamon mattamon added this to the 1.6.1 milestone Oct 4, 2024
@mattamon mattamon self-assigned this Oct 4, 2024
Copy link
Contributor

@mattamon mattamon left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution, works like a charm!

@mattamon mattamon added the Bug label Oct 4, 2024
@mattamon mattamon merged commit 29551b0 into pimcore:1.6 Oct 4, 2024
10 of 11 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Oct 4, 2024
@kjkooistra-youwe kjkooistra-youwe deleted the fix/tooltip-error-for-not-editable-many-to-many-relation branch October 4, 2024 06:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants