-
-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
✨ Chore: Migrate from Page POST requests to proper API Endpoints #158
Conversation
[skip ci]
🚂 let's gooo |
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.
woohoo! 🎉
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.
Insane refactor @Adammatthiesen, LGTM ✅
|
||
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.
Got a stray space there. Run a linter 💅🏼
packages/studioCMS/src/integrations/studioCMSDashboard/middleware/index.ts
Outdated
Show resolved
Hide resolved
packages/studioCMS/src/integrations/studioCMSDashboard/middleware/index.ts
Outdated
Show resolved
Hide resolved
packages/studioCMS/src/integrations/studioCMSDashboard/routes/api/pages/delete.ts
Show resolved
Hide resolved
...tudioCMS/src/integrations/studioCMSDashboard/routes/dashboard/components/PageListTable.astro
Show resolved
Hide resolved
packages/studioCMS/src/integrations/studioCMSDashboard/routes/api/pages/delete.ts
Show resolved
Hide resolved
!coauthor |
|
Original issue description
Currently we use Page Post requests within the Astro Frontmatter for handling most of the routes on the dashboard ( Example: Lines 45-112 ) This Issue would convert all of these into their own Astro API Endpoint, allowing more flexibility and customization of the StudioCMS Admin dashboard in the future!
closes #157