Skip to content

feat: detect stale sheet/config and prevent concurrent overwrites#877

Merged
auniverseaway merged 4 commits intomainfrom
stale
Apr 28, 2026
Merged

feat: detect stale sheet/config and prevent concurrent overwrites#877
auniverseaway merged 4 commits intomainfrom
stale

Conversation

@auniverseaway
Copy link
Copy Markdown
Member

  • Add StaleCheck class to check for stale configs and sheets
    • Checks every 30 seconds for changes.
    • If changes, but nothing new locally, automatically updates local sheet.
    • If local and remote changes, present a dialog allowing to refresh or keep local edits. Keeping local edits will prevent auto saves and manual blue button saves.
  • da-title updates to support the following:
    • disabledText property which doubles for disabling actions
    • Remove paper plane if only one action available
    • Any sheet inside /.da/ can no longer be previewed or published.
    • Lazily check for disablePublish to ensure actions show ASAP.
    • Move isSending to class property to allow showing send event for save only actions.
    • Created a class-level event for save action

Resolves: #704

How Has This Been Tested?

Sheet

https://stale--da-live--adobe.aem.live/sheet#/aem-sandbox/block-collection/.da/temp-translate

Config

https://stale--da-live--adobe.aem.live/config#/aem-sandbox/block-collection/

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

* Add StaleCheck class to check for stale configs and sheets
  * Checks every 30 seconds for changes.
  * If changes, but nothing new locally, automatically updates local
    sheet.
  * If local and remote changes, present a dialog allowing to refresh or
    keep local edits. Keeping local edits will prevent auto saves
    and manual blue button saves.
* da-title updates to support the following:
  * disabledText property which doubles for disabling actions
  * Remove paper plane if only one action available
  * Any sheet inside `/.da/` can no longer be previewed or published.
  * Lazily check for disablePublish to ensure actions show ASAP.
  * Move `isSending` to class property to allow showing send event for
    save only actions.
  * Created a class-level event for save action

Resolves: #704

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@aem-code-sync
Copy link
Copy Markdown

aem-code-sync Bot commented Apr 17, 2026

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-sync branch
Commits

Comment thread blocks/edit/da-title/da-title.js
Comment thread blocks/edit/da-title/da-title.js Outdated
Co-authored-by: Usman Khalid <usman.khalid@live.ca>
@auniverseaway auniverseaway merged commit f01e285 into main Apr 28, 2026
3 of 4 checks passed
@auniverseaway auniverseaway deleted the stale branch April 28, 2026 00:46
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.

[config] da-title web component should show save

2 participants