Skip to content
This repository was archived by the owner on Aug 6, 2020. It is now read-only.

Bugfix #7

Merged
merged 1 commit into from
May 22, 2017
Merged

Bugfix #7

merged 1 commit into from
May 22, 2017

Conversation

Awzaw
Copy link

@Awzaw Awzaw commented May 21, 2017

No description provided.

@AvgZing
Copy link

AvgZing commented May 21, 2017

This was meant to be serverauth, as that is what they support.

@dktapps
Copy link

dktapps commented May 21, 2017

since it's SimpleAuth single session integration SimpleAuthSingleSessionIntegration
and it's checking for an instance of SimpleAuth... this is indeed a bug.

@AvgZing
Copy link

AvgZing commented May 21, 2017

Oh. I was thinking that serverauth is made by one of their "allies" (kinda) and they weren't supporting simpleauth. But @dktapps your point is valid.

@SOF3 SOF3 merged commit 235b77c into LegendOfMCPE:master May 22, 2017
@SOF3
Copy link
Member

SOF3 commented May 22, 2017

Oops, right, thanks for pointing that out. I forgot to change that while copying code 😄

I have never tested the singleSession module anyway.

@SOF3 SOF3 added the bug label May 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants