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.

Sign up for GitHub

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

Jump to bottom

feat(MessageSend): add flags #1427

Merged
merged 2 commits into from Aug 16, 2023
Merged

feat(MessageSend): add flags #1427

merged 2 commits into from Aug 16, 2023

Conversation

ozraru
Copy link
Contributor

@ozraru ozraru commented Aug 14, 2023

Earlopain
Earlopain reviewed Aug 16, 2023
View reviewed changes
message.go Outdated Show resolved Hide resolved
FedorLap2006
FedorLap2006 requested changes Aug 16, 2023
View reviewed changes
Copy link
Collaborator

@FedorLap2006 FedorLap2006 left a comment •
edited

Choose a reason for hiding this comment

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

As @Earlopain mentioned, Flags should have MessageFlags type instead of int.

@FedorLap2006 FedorLap2006 changed the title Add flags field to MessageSend struct feat(MessageSend): add flags Aug 16, 2023
@FedorLap2006 FedorLap2006 added feature Feature implementation structs API structs and constants related changes labels Aug 16, 2023
@FedorLap2006 FedorLap2006 merged commit ff9176a into bwmarrin:master Aug 16, 2023
8 checks passed
@FedorLap2006
Copy link
Collaborator

Thanks for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Earlopain Earlopain Earlopain left review comments

@FedorLap2006 FedorLap2006 FedorLap2006 approved these changes

Assignees
No one assigned
Labels
feature Feature implementation structs API structs and constants related changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ozraru @FedorLap2006 @Earlopain