-
Notifications
You must be signed in to change notification settings - Fork 67
adding the crossword_companion sample #61
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
Conversation
…doesn't support out of the box!
…he tool that uses the API.
redbrogdon
left a comment
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.
Just a couple quick nits. Do you mind also adding a license header to each code file (e.g. https://github.com/flutter/demos/blob/main/compass_25/lib/model/activity.dart).
There's a single LICENSE file for the repo, so you don't need to add one here.
crossword_companion/README.md
Outdated
|
|
||
| ### Prerequisites | ||
|
|
||
| - A Firebase project. |
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.
Can this link to Firebase's docs for creating one? Also, add a note that the project needs Generative AI to be turned on (and possibly link to that doc as well).
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.
updated. ptal.
crossword_companion/deploy.sh
Outdated
| @@ -0,0 +1,4 @@ | |||
| flutter clean | |||
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.
I'd leave this file out.
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.
removed
…for the JSON result of the clue solving process.
|
I believe all of the feedback has been addressed, but please double-check me. |
|
Thanks! |
Pre-launch Checklist