Skip to content

Conversation

@DJK101
Copy link
Contributor

@DJK101 DJK101 commented Dec 2, 2025

Following from this PR in developer-docs

This change syncs the list of activity_types to be the same

@DJK101 DJK101 requested a review from a team December 2, 2025 17:51
Copy link
Contributor

@fedefrusc fedefrusc left a comment

Choose a reason for hiding this comment

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

LGTM

- admin_impersonation_consent_revoked
- admin_impersonation_end
- admin_impersonation_start
- admin_impersonation_consent_approved
Copy link
Contributor

Choose a reason for hiding this comment

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

@DJK101 Did you change the order on purpose?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This matches the order for developer docs, I can raise a separate PR for both repos to make them alphabetical again.

This was a literal Cmd + C -> Cmd + V job

@DJK101 DJK101 merged commit cf080e4 into main Dec 4, 2025
6 checks passed
@DJK101 DJK101 deleted the dj/tal/match-activity-types branch December 4, 2025 13:49
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.

3 participants