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

Added Satori client support #74

Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

Added Satori client support #74

wants to merge 27 commits into from

Conversation

britzl
Copy link
Collaborator

@britzl britzl commented Mar 5, 2024

No description provided.

@britzl britzl marked this pull request as draft March 5, 2024 18:42
@britzl britzl marked this pull request as ready for review June 1, 2024 06:06
@britzl britzl requested a review from lugehorsam June 1, 2024 06:07
@lugehorsam
Copy link

@britzl Should the satori folder be added to include_dirs so that it can be fetched from Defold via git using the dependencies section?

@lugehorsam
Copy link

Can you add and use a session object just like in Nakama?

@britzl
Copy link
Collaborator Author

britzl commented Jun 5, 2024

@britzl Should the satori folder be added to include_dirs so that it can be fetched from Defold via git using the dependencies section?

Ah, yes, it should. I've added the folder.

@britzl
Copy link
Collaborator Author

britzl commented Jun 5, 2024

Can you add and use a session object just like in Nakama?

Yes, satori.authenticate() returns an api session. You can use satori.set_bearer_token() to set the session token for future requests. What that what you meant?

@lugehorsam
Copy link

@britzl could you add an example to the README?

@lugehorsam
Copy link

Could you also leave instructions for running the Satori SDK tests in the README?

@britzl
Copy link
Collaborator Author

britzl commented Jun 7, 2024

@britzl could you add an example to the README?

I've updated the readme with Satori info.

Could you also leave instructions for running the Satori SDK tests in the README?

Added!

@britzl
Copy link
Collaborator Author

britzl commented Jun 9, 2024

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.

None yet

2 participants