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

feat(studio): remove legacy blogPost schema #142

Merged
merged 1 commit into from
Jan 15, 2025
Merged

Conversation

TheKnarf
Copy link
Collaborator

Relates to #138

@TheKnarf TheKnarf requested review from gramlar and an2n as code owners January 14, 2025 10:09
Copy link

vercel bot commented Jan 14, 2025

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

Name Status Preview Comments Updated (UTC)
skjer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 14, 2025 10:34am

@an2n
Copy link
Collaborator

an2n commented Jan 14, 2025

@TheKnarf Have you removed it from the dataset as well?

@TheKnarf
Copy link
Collaborator Author

@TheKnarf Have you removed it from the dataset as well?

No

@an2n
Copy link
Collaborator

an2n commented Jan 15, 2025

@TheKnarf Have you removed it from the dataset as well?

No

Could this cause a side effect where data cannot be deleted if the content type doesn’t exist?

@TheKnarf
Copy link
Collaborator Author

@TheKnarf Have you removed it from the dataset as well?

No

Could this cause a side effect where data cannot be deleted if the content type doesn’t exist?

No clue. Feel free to remove the data before approving the PR?

@an2n
Copy link
Collaborator

an2n commented Jan 15, 2025

@TheKnarf Have you removed it from the dataset as well?

No

Could this cause a side effect where data cannot be deleted if the content type doesn’t exist?

No clue. Feel free to remove the data before approving the PR?

Done.

@TheKnarf TheKnarf merged commit 6f5ede0 into main Jan 15, 2025
3 checks passed
@TheKnarf TheKnarf deleted the remove-blogposts branch January 15, 2025 12:25
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.

2 participants