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

POST playbook api specification includes public boolean definition #1847

Closed
wants to merge 1 commit into from

Conversation

codyhall
Copy link
Contributor

Summary

Identifies a hidden API field required to successfully create a Playbook, using the free tier.

Ticket Link

Not a ticket, but a common error identified and disclosed in the forums.
https://forum.mattermost.com/t/error-unable-to-decode-playboook/15315/5?utm_source=mattermost&utm_medium=in-product&utm_content=post_attachment_opengraph&uid=pitfe5zmwbgixyq13uwdfa1age&sid=93mykbogbjfrbbdqphx3zhze5c

@mattermost-build
Copy link
Contributor

Hello @codyhall,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@lieut-data
Copy link
Member

Thank you, @codyhall! Would you be open to helping move this PR to the mono-repo housing all of server, webapp, playbooks, and boards?

I've updated the scraping script to point there as well: mattermost/mattermost-api-reference#730.

I can also do this on your behalf, but I'd love to preserve your name on the contribution :)

@lieut-data lieut-data self-assigned this Apr 17, 2023
@codyhall
Copy link
Contributor Author

Thank you @lieut-data , I've created a new PR in MM-Server repo. mattermost/mattermost#22986

Thank you for allowing me to contribute! First contrib to a large open source project!

@lieut-data
Copy link
Member

Sweet! I'll close this PR and we can carry on over there 🎉

@lieut-data lieut-data closed this Apr 17, 2023
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.

None yet

3 participants