Skip to content
This repository was archived by the owner on Sep 19, 2024. It is now read-only.

feat: add logs policy#864

Open
rndquu wants to merge 1 commit intoubiquity:developmentfrom
rndquu:feat/add-logs-policy
Open

feat: add logs policy#864
rndquu wants to merge 1 commit intoubiquity:developmentfrom
rndquu:feat/add-logs-policy

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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants