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

Cannot create Session #92

Closed
crnvl opened this issue Jun 16, 2020 · 7 comments
Closed

Cannot create Session #92

crnvl opened this issue Jun 16, 2020 · 7 comments

Comments

@crnvl
Copy link

crnvl commented Jun 16, 2020

Describe the bug
My Code is

ConfigurationBuilder configurationBuilder = new ConfigurationBuilder();
                    configurationBuilder.setAuthKey(Config.AUTH_KEY);
                    configurationBuilder.setDevId(Config.DEV_ID);
                    configurationBuilder.build();

                    Paladins.initConfig(configurationBuilder);

                    Flowable<MatchDetail> matchDetails = Paladins.getMatchDetails(c.getMatchID());
                    Flowable<PlayerLoadout> playerLoadoutFlowable = Paladins.getPlayerLoadouts(args[0]);

But I always get Errors like
hirez.api.HiRezException: Require generate session for: testsession.

I already tried using

CreateSession session = new CreateSession();

but it doesn't seem to fix anything.
Idk, I heard it's supposed to create them automatically, but that also doesn't work for me somehow.

Desktop (please complete the following information):

  • Win 10
  • JDK 11
@stachu540
Copy link
Owner

ref: #90

@stachu540 stachu540 self-assigned this Jun 16, 2020
@crnvl
Copy link
Author

crnvl commented Jun 16, 2020

Yep, I tried that, but I may not know how to use it correctly, because i can't get it to work properly.

@crnvl
Copy link
Author

crnvl commented Jun 16, 2020

If I use Paladins.createSession().subscribe(); before every other Command related to the API it returns a NullPointerException

@stachu540 stachu540 added this to To do in Hi-Rez API v3.1 via automation Jun 16, 2020
@stachu540 stachu540 added this to the 3.1.0 milestone Jun 16, 2020
@stachu540 stachu540 modified the milestones: 3.1.0, 3.x Jun 16, 2020
@stachu540 stachu540 moved this from To do to In progress in Hi-Rez API v3.1 Jun 16, 2020
@stachu540
Copy link
Owner

Fix probably will be in 3.1.0 with Dependency updates and small API changes

@crnvl
Copy link
Author

crnvl commented Jun 16, 2020

Thanks a lot!

@crnvl crnvl closed this as completed Jun 16, 2020
Hi-Rez API v3.1 automation moved this from In progress to Done Jun 16, 2020
@crnvl crnvl reopened this Jun 16, 2020
Hi-Rez API v3.1 automation moved this from Done to In progress Jun 16, 2020
@stachu540
Copy link
Owner

stachu540 commented Jun 16, 2020

If you try play with jitpack and confirm, the issue has been fixed? I will release new version on Friday.

Also there is changes so.. there is NO static methods, they are in the class after initializing them, using create() static method. This method are consumer so it is simplified to build configuration, inside the Consumer. Also i've add 2 properties to builder which can helps to deliver your own parameters for ObjectMapper and OkHttpClient.Builder.

@stachu540 stachu540 moved this from In progress to Needs review in Hi-Rez API v3.1 Jun 16, 2020
@stachu540 stachu540 added this to In progress in v3 Jun 16, 2020
@stachu540 stachu540 moved this from In progress to Needs review in v3 Jun 16, 2020
@stachu540 stachu540 moved this from Needs review to Reviewer approved in v3 Jun 17, 2020
@stachu540 stachu540 moved this from Needs review to Reviewer approved in Hi-Rez API v3.1 Jun 17, 2020
@stachu540
Copy link
Owner

stachu540 commented Jun 18, 2020

The bug should be fixed, in the next release

v3 automation moved this from Reviewer approved to Done Jun 18, 2020
Hi-Rez API v3.1 automation moved this from Reviewer approved to Done Jun 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
v3
  
Done
Development

No branches or pull requests

2 participants