search

ubiquity / ubiquibot

Putting the 'A' in 'DAO'
https://github.com/marketplace/ubiquibot
MIT License
17 stars 61 forks source link

feat: add logs policy #864

Open rndquu opened 1 year ago

rndquu commented 1 year ago

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[bot] commented 1 year ago

Deploy Preview for ubiquibot-staging ready!

Name Link
Latest commit 1a8c12ef12255ac2d865f38aff9d63da2ec8c36a
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 commented 1 year ago

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

rndquu commented 1 year ago

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 https://github.com/ubiquity/ubiquibot/pull/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.