appwrite / awesome-appwrite

Carefully curated list of awesome Appwrite resources 💪
https://appwrite.io
966 stars 281 forks source link

Add Flutter Real-time Voting App #483

Closed 0niel closed 1 year ago

0niel commented 1 year ago

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

Yes

adityaoberai commented 1 year ago

Hi @0niel Were you able to check the review I sent earlier and make the recommended changes?

0niel commented 1 year ago

Hi @0niel Were you able to check the review I sent earlier and make the recommended changes?

Hi! I deleted the secrets, thank you!

I don't see much point in deleting the collection id in `constants.dart'. The application implies manually changing them in the source when required

adityaoberai commented 1 year ago

Hi @0niel Were you able to check the review I sent earlier and make the recommended changes?

Hi! I deleted the secrets, thank you!

I don't see much point in deleting the collection id in `constants.dart'. The application implies manually changing them in the source when required

Fair enough 👍 This should be good to merge then!