Open mwmadsen67 opened 3 years ago
Hi Wenchong, here are some notes on your design docs:
MVP List:
Schema:
State Shape:
Frontend Routes:
docs/:docId/slides/:slideId
or presentations/:presentationId/slides/:slideId
.Backend Routes:
PATCH /api/slides/:id
GET /api/slides
Looking good Wenchong, feel free to comment here on any questions or comments you have for me.
Hi Mike, thank you for all your comments. Your advice is extremely helpful and makes more and more sense as I proceed.
Based upon your comment notes, please let me know if I could change the MVP List to below. Thank you.
Doggie Slides, a clone of Google Slides, is a web-based editing and presentation application that supports the create, read, update, delete, as well as rendering of basic slides, texts, and graphics.
Hi Mike, please see my response to your comments regarding the schema, frontend routes, and backend routes below. Thank you.
why do docs need width and height?
what is a group?
you dont need a url/blob column in images, image urls will be stored using associations from active storage
editing a slide should be PATCH /api/slides/:id
; same for the rest of them
you will likely need a backend route for the slide index - GET /api/slides
I dont think that most of those controllers need their GET routes (GET shapes for example). When you get one slide you will be able to grab all the wrappers and associated info in the jbuilder for that response, and you shouldnt have to make an extra AJAX request.
Thanks for the response Wenchong. Yes I am happy with these updates to your MVP list and the rest of your design docs. Also, I agree that groups could be a bonus feature due to the workload.
Feel free to slack me or comment here if you have any questions.
Will do! Thank you so much, Mike!
Wiki Page Home
MVP List
Database Schema
back_ticked
back_ticked
back_ticked
Sample State
```javascript...```
). This will display the state as a code block instead of a giant line of textentities
session
errors
(here or inui
)ui
(if needed)comments
inside ofposts
Backend Routes
snake_case
GET likes
api endpoint because that info comes through the post showFrontend Routes
camelCase
inline coding text
(backticks)