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: Document Workspace Info reload components #2492

Merged
merged 4 commits into from
Oct 29, 2024

Conversation

leekelleher
Copy link
Member

Description

Fixes umbraco/Umbraco-CMS#17183.

Half feature, half bug fix. When saving/publishing a document, the components on the Info workspace view were not automatically updating. Using the Request Reload Structure event (from the workspace context), the Info components are refreshed on save/publish.

For the Links component, this makes use of a new dedicated Document URL Repository, (as opposed to observing the workspace context's .urls property).

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

How to test?

Go to an existing document, go to the Info workspace view, rename the document (e.g. append a number), press Publish, notice that the URL is updated in the "Links" component and that the "History" component has the latest Save/Publish audit logs.

Refactored to use the new Document URL Repository.

Plus other housekeeping on the main Document Workspace Info component.
to listen for the "Request Reload Structure" event
@leekelleher leekelleher merged commit 0d1b21e into main Oct 29, 2024
9 checks passed
@leekelleher leekelleher deleted the v15/feature/document-workspace-info-reload branch October 29, 2024 08:08
@mjpraxis
Copy link

@leekelleher @iOvergaard

Is the "half bugfix" that it fixes the update of the info-boxes?

And the other half is that the bug regarding update of the property values is not fixed?
If so then this issue should still be "open" umbraco/Umbraco-CMS#17183

image

@leekelleher
Copy link
Member Author

@mjpraxis Thanks for the follow-up. 🙏 My "half bugfix" comment was more that this PR also adds a feature, (the document URL repo code), not exclusively a bug fix, meaning it would be a bit cheeky of me to try to push it into the upcoming v15.0.0 release, so it'll be scheduled for v15.1.0.

I believe the other half of the issue you raised (about the server-side updates), should be resolved by PR #2491 (which will be part of v15.0.0), I just forgot to link to your issue on that PR. 🤦

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.

v14.3 - Components in Content view do not update after save/publish
3 participants