8000 :sparkles: Ensure read-only mode on non-workspace access to file by niwinz · Pull Request #6447 · penpot/penpot · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

✨ Ensure read-only mode on non-workspace access to file #6447

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

Open
wants to merge 2 commits into
base: staging
Choose a base branch
from

Conversation

niwinz
Copy link
Contributor
@niwinz niwinz commented May 9, 2025

Summary

This ensures that all non-workspace access to file where migrations need to be applied are executed in a read-only mode. Ensuring that only opening a file on workspace apply migrations.

It also fix old FIXME that makes imposible use penpot on read-only mode at 100%

How to test

It is a bit difficult to test but the use case is:

  • Create a file
  • Remove the last migration from file_migration table for that file
  • Mark as removed all thumbnails from file_thumbmnail table.
  • Go to dashboard and see file thumbnail is generated
  • Look at DB and the migration is still not applied
  • Open workspace of that file
  • The migration is now applied

@niwinz niwinz changed the title Niwinz staging bugfixes ✨ Ensure read-only mode on non-workspace access to file May 9, 2025
@niwinz niwinz added the hotfix label May 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0