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

Verify and clean REST API Implementation #37

Open
rahulagarwal13 opened this issue Apr 15, 2024 · 2 comments
Open

Verify and clean REST API Implementation #37

rahulagarwal13 opened this issue Apr 15, 2024 · 2 comments
Assignees
Labels

Comments

@rahulagarwal13
Copy link
Collaborator

rahulagarwal13 commented Apr 15, 2024

https://github.com/hasura/ddn_beta_ecommerce/tree/main/Composability/ProductHomePage/REST%20Implementation

Check all of these files. Verify that code looks okay. Does not need to be operational but should look realistic.
Cleaning means - add more comments or remove comments wherever required

@soorajshankar
Copy link
Member

soorajshankar commented Apr 15, 2024

Looks good to me.

  • We have shown some imports here which is not defined/shown anywhere else - but given the scope of the demo I think its okay to not show them
  • I think in part6, after the performance section, we can mention shortly about JS being single threaded while DDN parallelise threads it in real sense with rust using the maximum compute power.
  • I feel like a node js example to nested filtering could have been added with different databases ; that would be more lines of code and hence we could show more value in what they get with Hasura OOTB (by throwing more curiosity, difficulties in maintaining, adding features, etc.)

@rahulagarwal13
Copy link
Collaborator Author

Thanks for the feedback.

  • Let's make change as per second point

  • for third point - let's come up with a query using nested filtering for different databases. See if a query makes sense with all of Postgres, Mongo or Clickhouse. Or we can do two queries with combination of two Dbs. And then provide REST implementation for that one or two queries. IN the future we would need something similar for aggregations and nested aggregations. For Mongo - create a ProductDetailPageQuery and Clickhouse - see if you can augment the ProductHomePage Query with a recentlyviewed section or it could be a UserProfileQuery or maybe part of ProductDetailPageQuery. Make a separate folder for such queries along with the REST implementation
    (third point is not urgent for this week)

@rahulagarwal13 rahulagarwal13 self-assigned this Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants