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

NEO-954: "no snapshot available" dialog #696

Merged
merged 18 commits into from
Jan 21, 2025

Conversation

mgcm
Copy link
Contributor

@mgcm mgcm commented Dec 27, 2024

A blocking information dialog will now be shown when the net.nordeck.whiteboard.document.create event is not readable due to a user joining rooms where message history is not available.

This will happen in rooms where history is shared only after invite or join. It also applies to encrypted rooms where message keys are not shared for past events.

Screenshot 2025-01-09 at 15 29 27

✔️ Checklist

  • A changeset describing the change and affected packages (more info).
  • Added or updated documentation.
  • Tests for new functionality and regression tests for bug fixes.
  • Screenshots or videos attached (for UI changes).
  • All your commits have a Signed-off-by line in the message (more info).

Copy link

changeset-bot bot commented Dec 27, 2024

🦋 Changeset detected

Latest commit: e320822

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@nordeck/matrix-neoboard-widget Minor
@nordeck/matrix-neoboard-react-sdk Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mgcm mgcm force-pushed the nic/feat/NEO-954-no-snapshot-available-dialog branch 3 times, most recently from c1ba91e to ae27647 Compare January 9, 2025 16:10
@mgcm mgcm marked this pull request as ready for review January 9, 2025 16:14
@mgcm mgcm requested a review from a team January 9, 2025 16:14
Copy link
Contributor

@maheichyk maheichyk left a comment

Choose a reason for hiding this comment

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

Feature looks good to me, I left few questions/comments to the code.

@mgcm mgcm force-pushed the nic/feat/NEO-954-no-snapshot-available-dialog branch 2 times, most recently from e279db3 to 42c696b Compare January 20, 2025 12:58
@mgcm mgcm requested a review from a team as a code owner January 20, 2025 12:58
@mgcm mgcm force-pushed the nic/feat/NEO-954-no-snapshot-available-dialog branch 3 times, most recently from 470adf1 to 62e7d30 Compare January 20, 2025 14:59
@mgcm mgcm force-pushed the nic/feat/NEO-954-no-snapshot-available-dialog branch from c9a34f3 to f3b9a67 Compare January 20, 2025 23:16
Mikhail Aheichyk added 3 commits January 21, 2025 09:39
Signed-off-by: Mikhail Aheichyk <[email protected]>
Signed-off-by: Mikhail Aheichyk <[email protected]>
Mikhail Aheichyk added 3 commits January 21, 2025 10:04
@maheichyk maheichyk merged commit eb347cf into main Jan 21, 2025
5 checks passed
@maheichyk maheichyk deleted the nic/feat/NEO-954-no-snapshot-available-dialog branch January 21, 2025 07:45
HarHarLinks added a commit that referenced this pull request Jan 21, 2025
* Refactored connectionInfoState to snapshotInfoState

Signed-off-by: Milton Moura <[email protected]>

* Draft implementation

Signed-off-by: Milton Moura <[email protected]>

* Show blocking dialog when unable to load a snapshot from room history

Signed-off-by: Milton Moura <[email protected]>

* Remove logging and add support for embedded mode

Signed-off-by: Milton Moura <[email protected]>

* Changes load failure dispatching and associated tests

Signed-off-by: Milton Moura <[email protected]>

* Update dialog message to something more adequate

Signed-off-by: Milton Moura <[email protected]>

* Add changeset

Signed-off-by: Milton Moura <[email protected]>

* Update .changeset/eleven-pans-pull.md

Co-authored-by: Kim Brose <[email protected]>
Signed-off-by: Milton Moura <[email protected]>

* Applying review feedback

Signed-off-by: Milton Moura <[email protected]>

* Applying further review feedback

Signed-off-by: Milton Moura <[email protected]>

* Preparing for merge

Signed-off-by: Milton Moura <[email protected]>

* Rolling back on the connection state store refactor

Signed-off-by: Milton Moura <[email protected]>

* Move snapshotInfoSlice.ts code into connectionInfoSlice.ts.

Signed-off-by: Mikhail Aheichyk <[email protected]>

* Fix typo in name

Signed-off-by: Mikhail Aheichyk <[email protected]>

* Smaller test text changes

Signed-off-by: Mikhail Aheichyk <[email protected]>

* Add variable

Signed-off-by: Mikhail Aheichyk <[email protected]>

* Rename snapshotFailed to snapshotSaveFailed

Signed-off-by: Mikhail Aheichyk <[email protected]>

* Rename setSnapshotSuccessful to setSnapshotSaveSuccessful

Signed-off-by: Mikhail Aheichyk <[email protected]>

---------

Signed-off-by: Milton Moura <[email protected]>
Signed-off-by: Mikhail Aheichyk <[email protected]>
Co-authored-by: Kim Brose <[email protected]>
Co-authored-by: Mikhail Aheichyk <[email protected]>
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