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

Chore: update realtime functions to use fully qualified names #862

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mansueli
Copy link
Member

What kind of change does this PR introduce?

Feat: use fully qualified names for most realtime functions. (no update on realtime.apply_rls) as I think that should be done through https://github.com/supabase/walrus. Everything else migrated to use fully qualified names.

Fixes this issue:
https://supabase.com/docs/guides/database/database-linter?queryGroups=lint&lint=0011_function_search_path_mutable

What is the current behavior?

Please link any relevant issues here.

What is the new behavior?

Feel free to include screenshots if it includes visual changes.

Additional context

Add any other context or screenshots.

Copy link

vercel bot commented Apr 23, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
realtime-demo ⬜️ Ignored (Inspect) Visit Preview Jul 2, 2024 8:14am

@filipecabaco filipecabaco force-pushed the feat-use-fully-qualified-names branch from d82cf72 to b34a6b0 Compare July 2, 2024 08:14
@w3b6x9
Copy link
Member

w3b6x9 commented Jul 2, 2024

@filipecabaco can you take a look?

@filipecabaco
Copy link
Contributor

this needs to be redone on a new migration to tackle the changes we did for roles 👀

@mansueli
Copy link
Member Author

mansueli commented Jul 2, 2024

I can start over and make the changes again, in that case it might be worth closing this one and starting fresh :)

@filipecabaco
Copy link
Contributor

sorry for not merging it at the right time 😞 lets do that if you are ok with it

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.

3 participants