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

Fix broken deauthorization links. Use href instead of navigate. #251

Closed
wants to merge 2 commits into from

Conversation

jbosse
Copy link

@jbosse jbosse commented Jul 14, 2024

From the docs:

method (:string) - The HTTP method to use with the link. This is intended for usage outside of LiveView and therefore only works with the href={...} attribute. It has no effect on patch and navigate instructions.

From the docs:
> method (:string) - The HTTP method to use with the link. This is intended for usage outside of LiveView and therefore only works with the href={...} attribute. It has no effect on patch and navigate instructions.
@jbosse
Copy link
Author

jbosse commented Jul 14, 2024

I was working through the Read Me to try and get assent up and running and noticed my "Remove Github authentication" link was throwing an error.

@jbosse
Copy link
Author

jbosse commented Jul 14, 2024

Oops. I thought I had searched the issues first but I guess not. #250

@jbosse jbosse closed this Jul 14, 2024
@jbosse jbosse deleted the fix-deauthorization-link branch July 14, 2024 14:53
@jbosse jbosse restored the fix-deauthorization-link branch July 14, 2024 14:55
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.

1 participant