Skip to content

Conversation

@vitormattos
Copy link
Contributor

@vitormattos vitormattos commented Mar 1, 2023

🚧 TODO

🏁 Checklist

@vitormattos vitormattos added 2. developing feature: api 🛠️ OCS API for conversations, chats and participants php labels Mar 1, 2023
@vitormattos vitormattos added this to the 🚀 Next Beta (26) milestone Mar 1, 2023
@vitormattos vitormattos self-assigned this Mar 1, 2023
* created developer command to document commands

Signed-off-by: Vitor Mattos <[email protected]>
@vitormattos vitormattos changed the title [WIP] Test to verify if a command is documented Document occ commands Mar 2, 2023
@vitormattos vitormattos marked this pull request as ready for review March 2, 2023 01:00
Signed-off-by: Vitor Mattos <[email protected]>
@vitormattos vitormattos force-pushed the feature/document-commands branch from 9d4a0bd to 32e9e87 Compare March 2, 2023 02:28
@vitormattos vitormattos force-pushed the feature/document-commands branch from 32e9e87 to 2e15b55 Compare March 2, 2023 02:29
Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

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

As mentioned we don't need the unit test, Just add a github action which executes the command and compares the result similar to the .github/workflow/node.yml

@vitormattos vitormattos force-pushed the feature/document-commands branch 2 times, most recently from dfd4208 to 49b493d Compare March 3, 2023 00:57
* Add the markdown occ to mkdocs
* Add GitHub Action to check if the documentation is up to date

Signed-off-by: Vitor Mattos <[email protected]>
@vitormattos vitormattos force-pushed the feature/document-commands branch from 49b493d to 8ca055c Compare March 3, 2023 01:01
@vitormattos vitormattos force-pushed the feature/document-commands branch 3 times, most recently from b96517c to b836838 Compare March 3, 2023 02:16
Signed-off-by: Vitor Mattos <[email protected]>
@vitormattos vitormattos force-pushed the feature/document-commands branch from b836838 to 011c879 Compare March 3, 2023 02:18
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen nickvergessen enabled auto-merge March 3, 2023 06:37
@nickvergessen nickvergessen merged commit c726088 into master Mar 3, 2023
@nickvergessen nickvergessen deleted the feature/document-commands branch March 3, 2023 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review feature: api 🛠️ OCS API for conversations, chats and participants php

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Document occ commands

3 participants