Skip to content

Updates to .Net docs, generating a self-host token and .complete() note #185

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

dean-journeyapps
Copy link
Contributor

  • .Net page now takes users to all .Net demo app examples instead of to the CLI demo repo.
  • Added a note outlining the importance and potential repercussions of not calling .complete() in the uploadData() function.
  • Added a comment next to the JWKS_URL parameter in the config.yaml file highlighting that it is not applicable if using Supabase
  • Added link on "JOINS" to redirect to parameter query restrictions in the many-to-many sync rule guide.

@@ -41,7 +41,7 @@ Development tokens can be generated via either
# Client (application end user) authentication settings
client_auth:
# JWKS URIs can be specified here
jwks_uri: !env PS_JWKS_URL
jwks_uri: !env PS_JWKS_URL # Not applicable for Supabase
Copy link
Collaborator

@benitav benitav Jun 11, 2025

Choose a reason for hiding this comment

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

Might also be good to add this to the main config skeleton https://docs.powersync.com/self-hosting/installation/powersync-service-setup#powersync-configuration and/or demo config which we also share from that main docs page https://github.com/powersync-ja/self-host-demo/blob/main/config/powersync.yaml, assuming this was added because users get confused by it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, will do. Thank you!

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.

2 participants