-
Notifications
You must be signed in to change notification settings - Fork 19
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
Update Code to newer standards, requiring php 8.1 and fix deprecations #12
base: master
Are you sure you want to change the base?
Conversation
@AnnaDamm Thank you for taking the time to make this change! Are you actively using this in any projects? I'd love to see where it's being used. :) Since most of the files showed up as deleted and recreated (ugh, git, right?), it is hard for me to evaluate what's changed here. Nothing looks off at first glance, but I'll need some time to consider this. In the meantime, can you also add a change to
|
Oh, to be clear, the |
@simensen Thanks for looking at it. They show up as changed because I moved the files according to psr-0 to psr-4. As to your question of where we use it: our company uses an hr tool with a REST-API that is using HAWK as their authentication method. We consume this API using your library here. it is also mentioned in their "how to use this rest api" documentation for php: https://documenter.getpostman.com/view/799228/absenceio-api-documentation/2Fwbis#authentication |
any news on this? Used this library for the same api that Anna mentioned but a lot of notices came up on PHP8.3. |
@simensen Could you have a look at it again, please? |
fixes #11
more details, see CHANGELOG.md