Skip to content

Conversation

@osbre
Copy link
Contributor

@osbre osbre commented Aug 6, 2021

See #22 #43

✨ Progress

  • Resource attribute(s)
  • Integration with different attributes (prefix, middleware, domain etc)
  • Cover with tests

Let me know if I'm moving this PR in the right direction :) Thanks

@osbre osbre changed the title Add "resource" attribute Add "Resource" controller attribute Aug 6, 2021
@osbre osbre marked this pull request as ready for review August 6, 2021 09:28
@osbre
Copy link
Contributor Author

osbre commented Aug 14, 2021

@freekmurze Any chances of getting this reviewed and possibly merged? Thanks a lot

@freekmurze
Copy link
Member

Thanks, could you also update the readme?

@osbre
Copy link
Contributor Author

osbre commented Aug 14, 2021

Updated. Preview.

@freekmurze
Copy link
Member

freekmurze commented Aug 14, 2021 via email

@osbre
Copy link
Contributor Author

osbre commented Aug 14, 2021

Updated. Lmk if that works.

Just in case: would love to maintain a package or participate in any kind of internship / help with something for free

@freekmurze freekmurze merged commit 06201fc into spatie:master Aug 15, 2021
@osbre osbre deleted the controller-resource-attribute branch August 15, 2021 09:55
@freekmurze
Copy link
Member

Thanks for this PR.

Here's a list of open issues that you could take a look at: https://github.com/issues?utf8=✓&q=is%3Aopen+is%3Aissue+user%3Aspatie+is%3Apublic+-label%3Aenhancement+

As for internships: we only offer that for people that have knowledge of Dutch.

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