Skip to content

Feat: Adding poll support - #18

Merged
KubaZ2 merged 65 commits into
NetCordDev:alphafrom
budgetdevv:polls
Jul 9, 2024
Merged

Feat: Adding poll support#18
KubaZ2 merged 65 commits into
NetCordDev:alphafrom
budgetdevv:polls

Conversation

@budgetdevv

Copy link
Copy Markdown
Contributor

Add poll support to NetCord.

@budgetdevv

Copy link
Copy Markdown
Contributor Author

It is worth noting that this is still WIP. Made a PR just so progress can be tracked

@KubaZ2

KubaZ2 commented Apr 17, 2024

Copy link
Copy Markdown
Member

NetCord doesn't use required keyword anywhere currently. It would be nice not to break the rule. I would make constructors with required/common values instead. Also I don't think making a poll base class is a good idea. Mixing send and receive properties is not what NetCord does currently.

@budgetdevv

Copy link
Copy Markdown
Contributor Author

Okay, will try to refactor with what you said in mind

@budgetdevv

Copy link
Copy Markdown
Contributor Author

I also realized that I have to make IJsonModel wrappers for non-properties ( We discussed this in Discord )

Comment thread NetCord/Rest/MessagePollAnswerCount.cs
Comment thread NetCord/Rest/MessagePoll.cs Outdated
- Removed usages required keyword ( Kuba said that this will be a breaking change )

- Renamed Expiry to ExpireAt for consistency

- Use JsonEmoji instead of Emoji in json data-structures

- Made wrapper entities
Comment thread NetCord/Rest/JsonModels/JsonMessagePoll.cs Outdated
Comment thread NetCord/Rest/JsonModels/JsonMessagePoll.cs Outdated
Comment thread NetCord/Rest/JsonModels/JsonMessagePoll.cs Outdated
@budgetdevv

Copy link
Copy Markdown
Contributor Author

Finally checks are passing

…orks since my understanding of [GenerateAlias] is limited
Comment thread NetCord/Rest/RestClient.Undocumented.cs Outdated
Comment thread NetCord/Rest/RestClient.Undocumented.cs Outdated
Comment thread NetCord/MessagePollResults.cs Outdated
Comment thread NetCord/MessagePollProperties.cs Outdated
@KubaZ2
KubaZ2 merged commit a51199b into NetCordDev:alpha Jul 9, 2024
@KubaZ2

KubaZ2 commented Jul 9, 2024

Copy link
Copy Markdown
Member

Thanks!

@budgetdevv

Copy link
Copy Markdown
Contributor Author

You're welcome for nothing, sorry I haven't had time to work on it ( School sucks )

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.

2 participants