-
Notifications
You must be signed in to change notification settings - Fork 39
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
feat: add nodeinfo 2.1 document (parallel with 2.0). #181
Conversation
// implementation of http://nodeinfo.diaspora.software/protocol.html | ||
// implementation of http://nodeinfo.diaspora.software/ | ||
// TODO: activeMonth and activeHalfyear should be dynamic, currently static | ||
// TODO: enable override of nodeName and nodeDescription from settings |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this will line up nicely with our eventual switch to storing more of the config in the database and setting everything up on the web on first-time run. I'll try to keep an eye out for that when we get there!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine! If/when we eventually support a third version, we may want to be a little more clever about building the response in a way that duplicates less code, but I think it's fine to do things this way for the time being.
Yeah, agreed on that, it's not the most elegant but does the job for now like, uh, quite a few other areas of the codebase ❤️ |
This PR updates Postmarks support for the nodeinfo specification to include additional fields added in nodeinfo 2.1 (repository and project), as well as adding some optional metadata.
As a side-note, I also sent a PR to the nodeinfo repository to add Postmarks as a Fediverse server that implements the specification, as it has for some time now (albeit 2.0 level)