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

feat: make session macroable #84

Closed
wants to merge 2 commits into from
Closed

feat: make session macroable #84

wants to merge 2 commits into from

Conversation

Bricklou
Copy link
Contributor

@Bricklou Bricklou commented Apr 9, 2024

❓ Type of change

  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • πŸ‘Œ Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

πŸ“š Description

I made the Sesssion object macroable, this way, developers are able to extends it with custom behaviours. In example, we can think of a method like flashSuccess which would act the same way as flashErrors.

πŸ“ Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@thetutlage
Copy link
Member

The session class is not exported from the package main entrypoint. Can you please add that too?

@Bricklou
Copy link
Contributor Author

@thetutlage is there any other changes required?

@thetutlage
Copy link
Member

Sorry it took a while to test it locally. Can you please install @poppinss/macroable package, since it is not in the dependencies of the session package.

@thetutlage
Copy link
Member

Hey, as I am about to make a release, I have handled this change myself. Thanks for initiating the PR :)

@Bricklou Bricklou deleted the macroable-session branch April 15, 2024 05:01
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.

2 participants