-
-
Notifications
You must be signed in to change notification settings - Fork 321
v5 Entity overhaul #1653
Copy link
Copy link
Open
Labels
big-changecoreepic-nullabilityfor pull requests and issues relating to enabling nullability in v5for pull requests and issues relating to enabling nullability in v5up for grabs
Milestone
Description
Activity
Metadata
Metadata
Assignees
Labels
big-changecoreepic-nullabilityfor pull requests and issues relating to enabling nullability in v5for pull requests and issues relating to enabling nullability in v5up for grabs
Type
Fields
No fields configured for Epic.
Details
All entities need to:
Outbound REST or Gateway payloads also need to represent optionals and nullability corretly to discord. The public-facing entities will continue to combine optionality and nullability, and thus both states should be represented in C# as nullable.
Contribution
This is a bigger issue, but its complexity isn't very high. Please don't hesitate to create pull requests for a specific portion and reference this issue. Once the changes get integrated, I'll make sure to revise the list.
Entities
Notes
Yes i did take 20 minutes to c&p those names