Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

regard non_id_actions in do_load_resource like in do_authorize_resour… #73

Closed
wants to merge 1 commit into from

Conversation

philippneugebauer
Copy link

…ce to prevent crashes

fixes #71

@philippneugebauer
Copy link
Author

@cpjk What do you think about this?

@cpjk
Copy link
Owner

cpjk commented Dec 13, 2019

👋 Hey! I'm sorry for the lack of movement on this. I shouldn't have left this for so long without a reply.

I am not currently writing Elixir at all, and therefore, I think it would be irresponsible of me to review PRs on this project. I just kept putting off reviewing the issues/PRs on this repo until I started writing Elixir again, but never got around to it.

That is my mistake, and I'm really sorry about that 🙏. You put in the time to make this PR, and you deserve a response at the very least.

I've marked the project as "not being actively maintained" in the readme. Please feel free to fork it with a different name and put it up on hex with current fixes.

@cpjk cpjk closed this Dec 13, 2019
@philippneugebauer
Copy link
Author

No worries, I switched to bodyguard anyway

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nil comparison forbidden
3 participants