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

README: remove defunct/abandoned links #17

Merged
merged 1 commit into from
Dec 23, 2024
Merged

Conversation

acusti
Copy link
Contributor

@acusti acusti commented Dec 18, 2024

i discovered this project thanks to @getify’s interview on the local-first podcast. it’s great! i love the concept, and proper local-first client-only auth is a true breakthrough. finally, something truly serverless (too bad that word means “servers” 🙄). i noticed two things while going through the readme:

  1. the main demo link is 404ing, but i was able to use the library tests to demo the functionality very effectively, so i reworked that part of the readme to just point to the library tests.
  2. the second link to a library with server integration with WebAuthn points to the raohwork/webauthn-client repo, which says that it’s a “WIP Project” and that “It is not tested in any means yet”, but the last commit was more than 5 ½ years ago. that makes me think that it isn’t so WIP any more and has actually been abandoned, so i removed that link.
  3. fixed duplicate "completes completes" typo

README.md Outdated Show resolved Hide resolved
@acusti acusti force-pushed the patch-1 branch 2 times, most recently from 154e62f to 4616f0f Compare December 22, 2024 19:19
README.md Show resolved Hide resolved
• the vella.ai/auth link is 404ing
• the raohwork/webauthn-client repo says WIP Project but hasn’t been updated in 5 ½ years
Copy link
Member

@getify getify left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@getify getify merged commit 792377a into mylofi:main Dec 23, 2024
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.

None yet

3 participants