Closed PDROJACK closed 1 year ago
@PDROJACK I took a quick look at the code. Here are my quick suggestions :-
@PDROJACK I took a quick look at the code. Here are my quick suggestions :-
- Please add a clearer readme listing all the endpoints and the way to call them - todo, todo/{id}
- It looks like the result is shown with binary blob, can you please return the item added? A screenshot in the readme would be good too!
- Finally, some optional comments - would appreciate if you could use SurrealQL anywhere, say using something like record ids.
Hey @aravindputrevu I have made some changes considering your suggestions. I've added open API spec and swagger-UI as well to document the API endpoints. For the last point, I am using SurrealQL while deleting and getting items by id.
@PDROJACK Thanks for the improvements. Kindly add postman
reference to the readme.md
as well as a line about - docker compose is pulling surrealdb
. It'd be a great resource if you add a list of features you are using in SurrealDB.
Again, appreciate and thanks for the contribution.
Well done @PDROJACK! Thank you for contributing to SurrealDB during Hacktoberfest 2023 🎉 Kindly send an email to community@surrealdb.com to coordinate the delivery of your SurrealDB swag! 😁