Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added NotificationV2 and Group Events to Pipeline Tutorial #134

Merged
merged 2 commits into from
Mar 14, 2023

Conversation

VinyarionHyarmendacil
Copy link
Contributor

The following pipeline events are mostly documented, barring the properties specified:

NotificationV2 Events:

  • notification-v2
    • $.content.type: incomplete enum
    • $.content.category: incomplete enum
    • $.content.relatedNotificationsId: not sure what a gpos is
    • $.content.link: unknown scheme values?
    • $.content.responses[].type: incomplete enum
    • $.content.responses[].data: unknown formats depending on type
    • $.content.responses[].icon: incomplete enum
  • notification-v2-update
  • notification-v2-delete

Group Events:

  • group-joined
  • group-left
  • group-member-updated
    • $.content.member.membershipStatus: incomplete enum
    • $.content.member.visibility: incomplete enum
  • group-role-updated

Verified

This commit was signed with the committer’s verified signature.
elpiel Lachezar Lechev
NotificationV2 Events:
  notification-v2
  notification-v2-update
  notification-v2-delete
Group Events:
  group-joined
  group-left
  group-member-updated
  group-role-updated

Verified

This commit was signed with the committer’s verified signature.
elpiel Lachezar Lechev
@ariesclark ariesclark requested review from Foorack and Rexios80 March 11, 2023 03:10
Copy link
Member

@Foorack Foorack left a comment

Choose a reason for hiding this comment

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

Thank you extremely much for this @VinyarionHyarmendacil

LGTM!

@Rexios80
Copy link

I don't know enough about this to comment. If it looks good to @Foorack it's good for me.

@Foorack Foorack merged commit 6a395a3 into vrchatapi:main Mar 14, 2023
@VinyarionHyarmendacil
Copy link
Contributor Author

#129 addressed, forgot to refer earlier

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants