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.

Sign up for GitHub

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

Jump to bottom

[9.x] Use Macroable in Session facade #45310

Merged
merged 2 commits into from Dec 14, 2022
Merged

[9.x] Use Macroable in Session facade #45310

merged 2 commits into from Dec 14, 2022

Commits on Dec 14, 2022

  1. Use Macroable in Session Store 

    I found myself wanting to add a macro to encapsulate a session getter I was using in a few different places but noticed the Session facade is not using the Macroable trait like other facades do.

Do you see any issues with just importing the Macroable trait directly in the Session class?
    @karkowg
    karkowg committed Dec 14, 2022
    Configuration menu
    Copy the full SHA
    de611ae View commit details
    Browse the repository at this point in the history

  2. update docblock for Session facade

    @karkowg
    karkowg committed Dec 14, 2022
    Configuration menu
    Copy the full SHA
    7aae1d1 View commit details
    Browse the repository at this point in the history