-
-
Notifications
You must be signed in to change notification settings - Fork 117
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
Show more reviews (and similar "events"), alongside topic posts #58
Comments
Github doesn't treat these events as "posts". But this information is useful and eventually Forge will fetch and display it. |
Design-wise, would you consider them posts, a post subclass, or a different type entirely? |
cool @tarsius changed topic. thanks for info. magit is amazing as im sure you already know. |
I don't know yet. We could also treat posts as a subtype of a new event type. That's what Gitlab does. So with Gitlab we get those non-post events for free (especially after b019dfa), but there are other problems. As mentioned in that commit, events have no "subtype" field (at least I didn't see something like that when I last checked). And Gitlab additionally supports "threads" which are secondary discussions attached to a topic that are separate from the "normal posts" and "non-post events". I haven't looked into this any further, but it looked messy at first sight. |
Github CI integration result is another type of information about a topic that could usefully be displayed in forge. |
We use the github feature for marking a pr reviewed. the posts it generates in the topic don't seem to show up for me. Other posts entered on the pr do seem to show up so not sure where the problem might be.
The text was updated successfully, but these errors were encountered: