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

[Feature] Added embedded metadata tab #325

Merged
merged 8 commits into from
Jun 13, 2024
Merged

[Feature] Added embedded metadata tab #325

merged 8 commits into from
Jun 13, 2024

Conversation

Corepex
Copy link
Contributor

@Corepex Corepex commented May 31, 2024

Changes in this pull request

Resolves #151
Blocked by #237

@Corepex Corepex self-assigned this May 31, 2024
@Corepex Corepex requested a review from vin0401 May 31, 2024 09:50
@Corepex
Copy link
Contributor Author

Corepex commented Jun 3, 2024

There is an issue with one of the endpoints. I will double-check that with @lukmzig.

@Corepex
Copy link
Contributor Author

Corepex commented Jun 3, 2024

There is an issue with one of the endpoints. I will double-check that with @lukmzig.

fixed with pimcore/studio-backend-bundle#82

@markus-moser
Copy link
Contributor

markus-moser commented Jun 5, 2024

@Corepex Not sure why but I have a problem with the authentication for the API when I try it with this branch together with the latest 1.x of the backend:

image

Is there something which we can check or should we wait until the session auth is done (should be done relatively soon on backend side)?

@markus-moser
Copy link
Contributor

markus-moser commented Jun 12, 2024

  • Can we please make the left margin of the table more like in the design so that it is in one line with the headline?

image

vs.

image

  • Can we please make the name and especially value column a bit wider so that longer values have enough space?

image

@Corepex From my point of view we can merge this PR as soon as we solved the first two points 👍

Copy link

@Corepex Corepex merged commit b81e5f7 into 1.x Jun 13, 2024
11 checks passed
@Corepex Corepex deleted the 151-embedded-metainfo branch June 13, 2024 07:26
@github-actions github-actions bot locked and limited conversation to collaborators Jun 13, 2024
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.

Embedded Metainfo
3 participants