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

fix lumen authentication error #464

Open
wants to merge 6 commits into
base: 3.0
Choose a base branch
from

Conversation

dennisgon
Copy link

this is to solved issues #456

@jgrossi
Copy link
Member

jgrossi commented May 27, 2019

@dennisgon thanks for the PR. can you fix tests please?

@dennisgon dennisgon closed this May 30, 2019
@dennisgon dennisgon deleted the fix-lumen-auth branch May 30, 2019 18:14
@dennisgon dennisgon restored the fix-lumen-auth branch May 30, 2019 18:17
@dennisgon dennisgon reopened this May 30, 2019
@jgrossi
Copy link
Member

jgrossi commented May 30, 2019

@dennisgon do you need some help?

@dennisgon
Copy link
Author

hi @jgrossi i already push my changes but it didn't pass in stylci
can you help me?

@dennisgon
Copy link
Author

nevermind @jgrossi i already solved the problem
thanks for helping me out

@jgrossi
Copy link
Member

jgrossi commented Jun 4, 2019

@dennisgon gonna take a look and get back asap! thanks for sharing!

@jgrossi jgrossi changed the base branch from 2.8 to 3.0 January 27, 2020 21:10
@jgrossi
Copy link
Member

jgrossi commented Jan 27, 2020

@dennisgon can you push a new commit just to enabled CI again? changed the base to 3.0 instead of 2.8 and would like to double check CI before merging. sorry for the delay

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