Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
|
I'm not completely reviewing this yet, @matthewp , but first want to clarify: "Renaming Tables" is a "Studio Feature" because if you were just working locally, your DB is entirely built from scratch from your seeded data each time and there's nothing conflicting to "match" your schema to? (i.e. if you were just working locally, you'd rename the table in config and in your |
|
Yes, you would probably never use push for the local database. At least we don't have a reason for it, yet. |
sarah11918
left a comment
There was a problem hiding this comment.
Hi @matthewp ! Just have some suggestions for you to take a look at! When you're happy with this, I'll edit the file directly and make sure all the formatting is good, and add our new ` component for the set of instructions!
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
|
@sarah11918 this looks good to me now: https://docs-2vbv6g27f-astrodotbuild.vercel.app/en/guides/astro-db/#renaming-tables. What do you think? |
|
@matthewp I'm happy with the text if you are! Some of the indenting of the code samples is wonky, and I'll add our new fancy Let me edit this locally with a preview so I can get the syntax all correct, then you can preview the final output! |
|
@matthewp This looked fine locally to me now (though admittedly, the indenting tricks I had to play are a little sus). We'll check they preview here after this builds, and if the code samples look OK, then I think this is good to go! |
|
Looks great! I like the steps. |
|
Great! Looks good to me too, @matthewp ! I see in the linked PR that you want the docs to merge first here. Do you want this merged now? |
|
@sarah11918 Yes please! |
Description (required)
Explains how to rename a table in Astro DB.
Related issues & labels (optional)