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

Wait for authentication to complete before you can save a cell #1785

Merged
merged 2 commits into from
Jan 28, 2025

Conversation

pastuxso
Copy link
Contributor

@pastuxso pastuxso commented Nov 7, 2024

This update ensures that users must complete the authentication process before saving a cell. It improves reliability by handling authentication state transitions seamlessly during cell run, enhancing the user experience.

It now makes its best effort to wait for the user to complete the happy path, keeping the save button in a loading state. If authentication is completed within a minute, the cell is saved automatically without requiring the user to press save again. If authentication fails, the user is notified, and the save button returns to its initial state.

  • Event to wait until login finishes.
  • Handle login timeout.
  • Edge case: Running cells again while it is waiting for login.
wait-login.mp4

@pastuxso pastuxso changed the title Approach to save cells on UnAuthed Extension Wait for authentication to complete before you can save a cell Nov 8, 2024
@pastuxso pastuxso force-pushed the cris/saving-unauth-cells branch 3 times, most recently from 5c05ee0 to 1a7d442 Compare November 18, 2024 22:22
@pastuxso pastuxso marked this pull request as ready for review November 18, 2024 22:42
@pastuxso pastuxso marked this pull request as draft January 16, 2025 17:21
@pastuxso pastuxso closed this Jan 23, 2025
@pastuxso pastuxso force-pushed the cris/saving-unauth-cells branch from 1a7d442 to c481e71 Compare January 23, 2025 03:04
@pastuxso pastuxso reopened this Jan 23, 2025
@pastuxso pastuxso force-pushed the cris/saving-unauth-cells branch from 1785fd3 to ddf8e92 Compare January 24, 2025 19:22
@pastuxso pastuxso force-pushed the cris/saving-unauth-cells branch from ddf8e92 to 3e847e6 Compare January 24, 2025 21:47
@pastuxso pastuxso marked this pull request as ready for review January 24, 2025 22:11
Copy link
Member

@sourishkrout sourishkrout left a comment

Choose a reason for hiding this comment

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

✅ LGTM

@sourishkrout sourishkrout merged commit 7109023 into main Jan 28, 2025
1 check failed
@sourishkrout sourishkrout deleted the cris/saving-unauth-cells branch January 28, 2025 20:48
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