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

Disallow pasting into a PK cell #4342

Open
pavish opened this issue Mar 14, 2025 · 0 comments
Open

Disallow pasting into a PK cell #4342

pavish opened this issue Mar 14, 2025 · 0 comments
Labels
ready Ready for implementation type: bug work: frontend Related to frontend code in the mathesar_ui directory
Milestone

Comments

@pavish
Copy link
Member

pavish commented Mar 14, 2025

Description

  • Paste a valid value into a PK cell.
  • The frontend shows a "Saving Changes" loading spinner that never stops.
  • Notice that it succeeds on the backend.
  • Refresh and notice the updated value.
  • We currently do not allow modifying a saved PK cell, nor do we have plans to support it at the moment.
  • We should make this behaviour consistent.
  • I'm proposing showing a toast error when users try to paste into a PK cell.
Screen.Recording.2025-03-14.at.7.38.38.PM.mov
@pavish pavish added ready Ready for implementation type: bug work: frontend Related to frontend code in the mathesar_ui directory labels Mar 14, 2025
@pavish pavish added this to the Backlog milestone Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready Ready for implementation type: bug work: frontend Related to frontend code in the mathesar_ui directory
Projects
None yet
Development

No branches or pull requests

1 participant