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

move event attributes to the registry #543

Closed
wants to merge 4 commits into from

Conversation

mlunadia
Copy link
Contributor

This PR moves event attributes to the registry. It is part of #407

Signed-off-by: Miguel Luna <39376769+mlunadia@users.noreply.github.com>

**[1]:** Events across different domains may have same `event.name`, yet be unrelated events.

Copy link
Member

Choose a reason for hiding this comment

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

Please render the table with full (check other examples) so we don't loose the enum table values

Copy link
Contributor Author

Choose a reason for hiding this comment

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

as event.domain has been removed it looks like we don't need the full render anymore

@@ -0,0 +1,25 @@
<!--- Hugo front matter used to generate the website version of this page:
linkTitle: Events
Copy link
Contributor

Choose a reason for hiding this comment

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

Should it be events or event?

Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Jan 25, 2024
Copy link
Member

@ChrsMark ChrsMark left a comment

Choose a reason for hiding this comment

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

lgtm!

@joaopgrassi could you have another look?

Copy link

github-actions bot commented Feb 3, 2024

Closed as inactive. Feel free to reopen if this PR is still being worked on.

@github-actions github-actions bot closed this Feb 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants