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

feat: add logs policy #864

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

rndquu
Copy link
Member

@rndquu rndquu commented Oct 18, 2023

This PR adds a new supabase RLS policy that allows all users to read from the logs table. This is required for the logging app. If policy is not set then the logger app can't read from the logs table using supabase's public anon key.

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for ubiquibot-staging ready!

Name Link
🔨 Latest commit 1a8c12e
🔍 Latest deploy log https://app.netlify.com/sites/ubiquibot-staging/deploys/652faf2a295a290008b5a87f
😎 Deploy Preview https://deploy-preview-864--ubiquibot-staging.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.

@0x4007
Copy link
Member

0x4007 commented Oct 19, 2023

We will need to run this again for the new database from the refactor.

@rndquu
Copy link
Member Author

rndquu commented Oct 19, 2023

We will need to run this again for the new database from the refactor.

No, we need to run this migration only once. So it's enough to merge this PR and then merge the development branch into main to create the policy introduced in the current PR. When #644 is merged there will be no need to run this migration again because RLS policies are kind of decorators over tables and persist across schema updates.

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.

None yet

3 participants