-
Notifications
You must be signed in to change notification settings - Fork 189
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
Update telescope_profiles and feeds tables #2977
Update telescope_profiles and feeds tables #2977
Conversation
6c09e63
to
44934a8
Compare
@tcvan0707 This made a slight change to how a feed is considered to be invalid. A user can add an existing feed if it is not already linked with a Telescope user, or the user_id is null. |
@humphd and @JerryHue, I might miss some properties as suggested here #2860 (comment), let me know. |
@JerryHue said to change |
|
44934a8
to
ac5d3ef
Compare
ac5d3ef
to
511cdc4
Compare
I changed link -> html_url |
I noticed you initially created the From Supabase Tables docs
|
@cindyledev, we want to put them public schema so they are exposed to the client. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, a few small things.
@DukeManh followed the steps. I'm getting this 🔽 |
You'll need to run the frontend server with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, I'm wrong about html url from wiki. Looks good
c2a28a6
to
4bf8268
Compare
Thank you, everyone. |
Issue This PR Addresses
Part of #2860
This updates
feeds
table schema in preparation for migrating Planet feed list to Supabase.Schema
Steps to test the PR