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

Threadcap: Add support for the "summary" property #6

Closed
dellagustin opened this issue Jan 30, 2023 · 3 comments
Closed

Threadcap: Add support for the "summary" property #6

dellagustin opened this issue Jan 30, 2023 · 3 comments

Comments

@dellagustin
Copy link

dellagustin commented Jan 30, 2023

Hello @johnspurlock-skymethod ,

While testing I noticed that threadcap is not supporting the summary property.
According to my tests, this property is used to store Content Warnings (e.g. spoiler alert, sensitive content). See references below.

It would be nice to have this property supported in threadcap.

image

image

https://fosstodon.org/@dellagustin/109780550965092145

{
    "@context": [
        "https://www.w3.org/ns/activitystreams",
        {
            "ostatus": "http://ostatus.org#",
            "atomUri": "ostatus:atomUri",
            "inReplyToAtomUri": "ostatus:inReplyToAtomUri",
            "conversation": "ostatus:conversation",
            "sensitive": "as:sensitive",
            "toot": "http://joinmastodon.org/ns#",
            "votersCount": "toot:votersCount"
        }
    ],
    "id": "https://fosstodon.org/users/dellagustin/statuses/109780550965092145",
    "type": "Note",
    "summary": "Testing content warning",
    "inReplyTo": null,
    "published": "2023-01-30T22:37:50Z",
    "url": "https://fosstodon.org/@dellagustin/109780550965092145",
    "attributedTo": "https://fosstodon.org/users/dellagustin",
    "to": [
        "https://www.w3.org/ns/activitystreams#Public"
    ],
    "cc": [
        "https://fosstodon.org/users/dellagustin/followers"
    ],
    "sensitive": true,
    "atomUri": "https://fosstodon.org/users/dellagustin/statuses/109780550965092145",
    "inReplyToAtomUri": null,
    "conversation": "tag:fosstodon.org,2023-01-30:objectId=84208846:objectType=Conversation",
    "content": "<p>A message with content warning.</p>",
    "contentMap": {
        "en": "<p>A message with content warning.</p>"
    },
    "attachment": [],
    "tag": [],
    "replies": {
        "id": "https://fosstodon.org/users/dellagustin/statuses/109780550965092145/replies",
        "type": "Collection",
        "first": {
            "type": "CollectionPage",
            "next": "https://fosstodon.org/users/dellagustin/statuses/109780550965092145/replies?only_other_accounts=true&page=true",
            "partOf": "https://fosstodon.org/users/dellagustin/statuses/109780550965092145/replies",
            "items": []
        }
    }
}
@johnspurlock-skymethod
Copy link
Contributor

Remember threadcap started as a way to normalize comments coming from AP or twitter (and an old thing based on fountain comments). But now that AP is a bit more in focus, I agree that summary, at least, would be a nice addition to the payload.

@johnspurlock-skymethod
Copy link
Contributor

Added in 0.1.10

https://www.npmjs.com/package/threadcap/v/0.1.10

@dellagustin
Copy link
Author

That is great! Thanks a lot.

Remember threadcap started as a way to normalize comments coming from AP or twitter (and an old thing based on fountain comments). But now that AP is a bit more in focus, I agree that summary, at least, would be a nice addition to the payload.

Even if twitter was still in the game, I think it would still make sense to add summary. It is a backwards compatible addition for a feature that would simply not be present for twitter (shame on them 😃).

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

No branches or pull requests

2 participants