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

docs: modify update-message-flag to contain information about flags. #12480

Closed
wants to merge 1 commit into from
Closed

docs: modify update-message-flag to contain information about flags. #12480

wants to merge 1 commit into from

Conversation

Hypro999
Copy link
Member

@Hypro999 Hypro999 commented Jun 4, 2019

@Hypro999
Copy link
Member Author

Hypro999 commented Jun 4, 2019

@timabbott, all that's left is just filling in the blanks 🙂

@timabbott
Copy link
Sponsor Member

Nice, merged as c6b0343, after adding fa77467 and clarifying what belongs here.

Thinking about how this could be better, I think we should probably add a link to that section from other articles that interact with message flags (primarily the GET /messages endpoint, which returns that day).

@Hypro999 can you take care of that in a follow-up commit?

@Hypro999
Copy link
Member Author

Hypro999 commented Jun 4, 2019

@timabbott: Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants