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

Instant opening of a stream #2729

Merged
merged 11 commits into from
Oct 1, 2024
Merged

Instant opening of a stream #2729

merged 11 commits into from
Oct 1, 2024

Conversation

MeyerPV
Copy link
Collaborator

@MeyerPV MeyerPV commented Aug 30, 2024

https://daostack.notion.site/Instant-opening-of-a-stream-d588a3bf0ed84cee8d2fa50ff8b58b01?pvs=74

What was changed?

  • Added OptimisticDiscussion type.
  • Added optimistic store values.
  • Added support for optimistic messages for Discussions

Added OptimisticDiscussion type.
Added optimistic store values.
Added support for optimistic messages for Discussions
@MeyerPV MeyerPV requested a review from elatif2020 August 30, 2024 09:53
Copy link

netlify bot commented Aug 30, 2024

Deploy Preview for preview-common ready!

Name Link
🔨 Latest commit 6c332c7
🔍 Latest deploy log https://app.netlify.com/sites/preview-common/deploys/66fb0366dc69d9000852f6db
😎 Deploy Preview https://deploy-preview-2729--preview-common.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@elatif2020
Copy link
Collaborator

@MeyerPV somehow we still have 2+ sec loader appearing after clicking Publish. Can we shorten this? Actually I think the new stream appears already before, but the discussion creation form hangs longer

Added instant close on creation modal
@elatif2020
Copy link
Collaborator

elatif2020 commented Sep 8, 2024

@MeyerPV see the recording here

  1. The new item appears immediately 🥇 , but after a few seconds there is a jump in which it's minimized and re-expands. Can we avoid this?
  2. Specifically there is an issue when I try to create another discussion before the jump: the new discussion form disappears with the jump...

Improvements to optimistic:
1. Fixed glitched in card while transitioning from Optimistic to Real
2. Added focuses for ChatComponent and DiscussionCreation
3. Remove unnecessary  proposalCreation modal
@MeyerPV MeyerPV merged commit d0483cf into dev Oct 1, 2024
5 checks passed
@MeyerPV MeyerPV deleted the CW-Optimistic-streams branch October 1, 2024 07:19
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.

2 participants