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

Anonymous Users #187

Merged
merged 1 commit into from
Sep 13, 2023
Merged

Anonymous Users #187

merged 1 commit into from
Sep 13, 2023

Conversation

lishaduck
Copy link
Member

@lishaduck lishaduck commented Sep 10, 2023

Description

This PR adds support for anonymous users.
And, in doing so, makes quite a few other changes.

Closes: #189

Type of Change

  • ✨ New feature
  • πŸ› Bug fix
  • πŸ’₯ Breaking change
  • ♻️ Code refactor
  • πŸ’š CI
  • πŸ› οΈ Build configuration change
  • πŸ—‘οΈ Chore

@lishaduck lishaduck added enhancement New feature or request dart Regards Dart code labels Sep 10, 2023
@lishaduck lishaduck added this to the Future milestone Sep 10, 2023
@lishaduck lishaduck self-assigned this Sep 10, 2023
@netlify
Copy link

netlify bot commented Sep 10, 2023

βœ… Deploy Preview for pattonville-wallet ready!

Name Link
πŸ”¨ Latest commit 6dd36cb
πŸ” Latest deploy log https://app.netlify.com/sites/pattonville-wallet/deploys/6501f219bc2dd100081221c7
😎 Deploy Preview https://deploy-preview-187--pattonville-wallet.netlify.app/
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@lishaduck lishaduck marked this pull request as ready for review September 12, 2023 02:19
@lishaduck lishaduck requested a review from a team as a code owner September 12, 2023 02:19
PetalCat
PetalCat previously approved these changes Sep 12, 2023
* This change officially makes development require using Docker,
  though, if you don't log in as anonymous, it _should_ still work fine.
* This change mostly just resolves an SDK version conflict.
  The Appwrite SDK v11 requires Appwrite Server v1.4.1.
  Appwrite Cloud is still only at v.1.1.2.
  Thus, some bugs were popping up.
* This change also fixes logging, mostly.
  We now use proper OS detection to determine if ASCI output is enabled.
  Before, it was always on, causing a crash.
  However, fixing it lead to the discovery of a new bug-
  Errors and stacktraces are not logged to the console,
  which is quite annoying when debugging.
  However, after a week or two spent debugging it,
  I think fixing it will need to be deferred.
* This commit also adds a bit more customization for Docker support.
  Self-signed certificates for Appwrite Server now work by default.
  Also, `dart_define` invocations grew shorter.
@lishaduck lishaduck requested review from PetalCat and a team and removed request for PetalCat September 13, 2023 17:32
@lishaduck lishaduck modified the milestones: Future, ELO Work Sep 13, 2023
@lishaduck lishaduck added this pull request to the merge queue Sep 13, 2023
Merged via the queue into main with commit 07551d8 Sep 13, 2023
13 of 14 checks passed
@lishaduck lishaduck mentioned this pull request Sep 14, 2023
@lishaduck lishaduck deleted the anonymous branch September 14, 2023 02:52
This was referenced Sep 30, 2023
This was linked to issues Oct 2, 2023
@lishaduck lishaduck changed the title Anonymous Anonymous Users Oct 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dart Regards Dart code enhancement New feature or request
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Support Anonymous Users Minor, Misc App routing issues
2 participants