-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat: add a few new types #62
Conversation
99158a8
to
7c52f8c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good. I have one concern about the use of plural however, not entirely convinced the alternative (dropping the s) is more correct
Cargo.toml
Outdated
entitlements = [] | ||
moderation = ["serde"] | ||
points = ["serde"] | ||
stream = ["serde"] | ||
timestamp = [] | ||
user = ["serde"] | ||
goal = ["serde"] | ||
extensions = [] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should entitlements
and extensions
be plural or singular?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Granted everything else except points
is singular, I think they should be singular as well (diff).
7c52f8c
to
1034b49
Compare
1034b49
to
4e805c6
Compare
Adds a few types from endpoints/topics we don't support yet. Additionally, it fixes a few rust/clippy warnings.
I didn't add a
GameId
(e.g. inGet Game Analytics
) since I'm not sure if that's the same as a category [?] (but seems to be different...).