Skip to content

(feat) rewrite#4

Open
ictbeheer wants to merge 2 commits intomainfrom
feat/rewrite
Open

(feat) rewrite#4
ictbeheer wants to merge 2 commits intomainfrom
feat/rewrite

Conversation

@ictbeheer
Copy link
Collaborator

@ictbeheer ictbeheer commented Jul 2, 2025

Having to create a new major version everytime a method is added to will result in a lot of update headaches, forcing us to update all plugins/packages using this package. Switching to classes also takes away the burden to create semi-identical classes in many plugins.

@ictbeheer ictbeheer force-pushed the feat/rewrite branch 2 times, most recently from 7b71952 to 91c33e1 Compare July 3, 2025 07:52
@ictbeheer ictbeheer marked this pull request as ready for review July 4, 2025 10:35
### Example: Providing User Data via Generic Filter (with switch)

You can also use the generic filters and handle multiple IDPs with a switch statement:

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

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.

4 participants