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

Adds the ability to define a getMixPanelKey method on the user class … #81

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Arkitecht
Copy link

Adds the ability to define a getMixPanelKey method on the user class to use for the identifier

  • Addresses Issue 54
  • Adds tests for default identifier and alternative identifier
  • Updates composer to add laravel/ui which is now needed for Auth routes
  • Adds config in Fixtures to load the Laravel\UI\UiServiceProvider for Auth routes

@Arkitecht
Copy link
Author

Unfortunately Travis is failing due to composer memory issues

PHP Fatal error: Allowed memory size of 1610612736 bytes exhausted (tried to allocate 4096 bytes) in phar:///home/travis/.phpenv/versions/7.3.22/bin/composer/src/Composer/DependencyResolver/Solver.php on line 223

@mikebronner
Copy link
Owner

@Arkitecht thanks for this PR. I will review it this weekend. :)

@juancho48
Copy link

COMPOSER_MEMORY_LIMIT=-1 might help to run Travis

@Arkitecht
Copy link
Author

Hey @mikebronner - wondering if you had a chance to review this PR. Please let me know if there is anything I can do to help get it in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants