Skip to content

blog post: converting GenLatte app to fullstack Dart - #13899

Open
sfshaza2 wants to merge 8 commits into
mainfrom
latte
Open

sfshaza2 wants to merge 8 commits into
mainfrom
latte

Conversation

@sfshaza2

Copy link
Copy Markdown
Contributor

Waiting for image(s)

@sfshaza2
sfshaza2 requested a review from a team as a code owner September 17, 2026 16:54

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a new blog post detailing the migration of the GenLatte backend from Node.js to fullstack Dart. The post covers sharing models, consolidating multiple Cloud Run services into a single Firebase function using DTOs, refactoring client-side writes, and adding end-to-end tests. The review feedback highlights several typographical and grammatical errors in the markdown prose, as well as critical syntax and compilation errors in one of the Dart code examples (specifically, parameter shadowing and a missing semicolon).

Comment thread sites/www/content/blog/converted-genlatte-fullstack-dart/index.md Outdated
Comment thread sites/www/content/blog/converted-genlatte-fullstack-dart/index.md Outdated
Comment thread sites/www/content/blog/converted-genlatte-fullstack-dart/index.md Outdated
Comment on lines +195 to +202
(request, response) {
final MessageResponse response = switch (request.data) {
SaveOrderParameters msg => saveOrder(msg),
CompleteOrderParameters msg => completeOrder(msg),
...
}
return response.toJson();
},

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

This code snippet contains two Dart syntax/compilation errors:

  1. The parameter response is shadowed by the local variable final MessageResponse response, which is a compilation error in Dart.
  2. The switch expression assignment is missing a terminating semicolon.
Suggested change
(request, response) {
final MessageResponse response = switch (request.data) {
SaveOrderParameters msg => saveOrder(msg),
CompleteOrderParameters msg => completeOrder(msg),
...
}
return response.toJson();
},
(request) {
final MessageResponse response = switch (request.data) {
SaveOrderParameters msg => saveOrder(msg),
CompleteOrderParameters msg => completeOrder(msg),
...
};
return response.toJson();
},
References
  1. Prioritize reporting broken or misleading examples. (link)

Comment thread sites/www/content/blog/converted-genlatte-fullstack-dart/index.md Outdated
sfshaza2 and others added 4 commits September 17, 2026 09:55
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@flutter-website-bot

flutter-website-bot commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

Staged preview of the updated docs.flutter.dev site (updated for commit dd78c8d):

https://flutter-docs-prod--docs-pr13899-latte-iczre173.web.app


## Fully leaning in to fullstack Dart

Dart and JavaScript are different languages with different strengths.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coward! :)

@flutter-website-bot

Copy link
Copy Markdown
Collaborator

Staged preview of the updated flutter.dev site (updated for commit f84df41):

https://flutter-dev-230821--www-pr13899-latte-il01vvcv.web.app

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.

3 participants