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

Update example in README #40

Merged

Conversation

til
Copy link
Contributor

@til til commented Sep 21, 2023

after_magic_link_sent_path_for receives a resource_or_scope, so it should probably do what devise does in stored_location_key_for with it.

`after_magic_link_sent_path_for` receives a `resource_or_scope`, so it
should probably do what devise does in `stored_location_key_for` with
it.
@abevoelker abevoelker merged commit 8e961c3 into abevoelker:master Sep 21, 2023
14 checks passed
@abevoelker
Copy link
Owner

Good catch thank you! 🤘

@til til deleted the minor-readme-update-with-resource-or-scope branch September 21, 2023 15:00
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