Skip to content

Improvements to 'out of sync' notifications#482

Merged
Freymaurer merged 4 commits intonfdi4plants:developerfrom
Thyra:out-of-sync-notification-2
Dec 15, 2025
Merged

Improvements to 'out of sync' notifications#482
Freymaurer merged 4 commits intonfdi4plants:developerfrom
Thyra:out-of-sync-notification-2

Conversation

@Thyra
Copy link
Copy Markdown
Contributor

@Thyra Thyra commented Dec 15, 2025

(re #469 ).

  • I noticed dirty_remote also triggers when there are un-pushed changes so I adapted the wording of the notification
  • the notification now also disappears on its own after syncing
  • when opening an ARC without being logged in, there is a temporary (5sec, info) notification that you need to login in order to sync changes

After we're through with this PR and #481, perhaps we can try a pre-release to get all this on the way to the users? 🙂

@Freymaurer Freymaurer requested a review from Copilot December 15, 2025 09:39
@Freymaurer
Copy link
Copy Markdown
Collaborator

yes i will make a release after this!

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances the user experience around synchronization notifications in ARCitect by improving notification messages and behavior. The changes address issue #469 by making sync status notifications more informative and responsive to user actions.

  • Updated the "dirty remote" notification message to clarify it covers both pull and push scenarios
  • Added automatic dismissal of the out-of-sync notification when the sync status is resolved
  • Introduced a new informational notification when opening an ARC without being logged in

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/renderer/src/App.vue Outdated
@Thyra
Copy link
Copy Markdown
Contributor Author

Thyra commented Dec 15, 2025

doesn't seem to make a difference but I applied it 👍

@Freymaurer
Copy link
Copy Markdown
Collaborator

Update the changelog accordingly for "changed" and i will merge :) By the way, could we maybe have a talk again? I would like to show you something

@Thyra
Copy link
Copy Markdown
Contributor Author

Thyra commented Dec 15, 2025

I don't think changelog makes sense because the whole feature is not released yet at all, so then it would be 'Added: Feature X, Changed: Feature X' 😄
Sure, I'll email you right now ☺️

@Freymaurer Freymaurer merged commit c882a9b into nfdi4plants:developer Dec 15, 2025
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.

3 participants