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-6315: Added url param publishedContentId #53

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

ciastektk
Copy link
Contributor

@ciastektk ciastektk commented Aug 30, 2023

https://issues.ibexa.co/browse/IBX-6315

Related to: ibexa/admin-ui#872

This PR adds publishedContentId url param to inform which content have been published. This is needed for communication between open tabs and dynamically updating title of embedded content item.

Example:
First open tab contains form with embedded content item. Second tab contains edit form of embedded content item in first tab. When title for content has been published in second tab, the title of embedded item on first tab should dynamically be updated without refreshing page.

@konradoboza konradoboza requested a review from a team August 30, 2023 07:11
@micszo micszo self-assigned this Oct 10, 2023
@ciastektk ciastektk force-pushed the ibx-6315-added-url-param-published-content-id branch from e095de3 to c8d60e3 Compare October 10, 2023 12:54
@sonarcloud
Copy link

sonarcloud bot commented Oct 10, 2023

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

@micszo micszo removed their assignment Oct 19, 2023
@konradoboza konradoboza merged commit 601b5b4 into main Oct 19, 2023
12 checks passed
@konradoboza konradoboza deleted the ibx-6315-added-url-param-published-content-id branch October 19, 2023 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants