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

Json API #96

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Json API #96

wants to merge 8 commits into from

Conversation

morganastra
Copy link
Contributor

@morganastra morganastra commented Nov 17, 2018

Building off of @SomeHats's work, this implements a JSON API per #29

What exactly should go in it is still an open question, especially since we've added support for alternate pronoun sets since the issue was opened.

  • PR message includes a link to the relevant issue(s) 💁‍♀️🖇️🧾
  • Commit messages are well-formatted
    (please follow this guide) 📝📐
  • New features have test coverage 📋☑️☑️☑️
  • The app boots and pronoun pages are served correctly
    (try lein ring server) 👩‍💻🚀

@morganastra morganastra self-assigned this Nov 17, 2018
@morganastra
Copy link
Contributor Author

Once we settle on a design we need to add tests for this too

dgw added a commit to sopel-irc/sopel that referenced this pull request Mar 27, 2023
First we waited on an API that might never exist:
witch-house/pronoun.is#96

Then the pronoun.is site went offline completely:
witch-house/pronoun.is#147

Eventually someone came along to make a clone that was easier to host,
written using Svelte instead of Clojure:
https://github.com/lucasew/svelte-pronounisland

We can fork that so we have control over when it's updated:
https://github.com/sopel-irc/pronoun-service

And now we have our own pronoun site to go with this plugin:
https://pronouns.sopel.chat/

(But it still doesn't have an API.)

It's a shame to lose pronoun.is, though. That's the Best™ domain. :(
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