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(Main-modal): 📝 add doc's for main modal #117

Merged
merged 4 commits into from
Mar 10, 2024

Conversation

VipinDevelops
Copy link
Contributor

This pull request adds documentation for the main modal in the application.

The added documentation outlines the usage of the main modal and the corresponding slash command ("/github") to open it.

README.md Outdated
Comment on lines 26 to 27
<li> See Helper Message / Command List -> /github help </li>
<li> The app can be accessed with any of the slash commands /gh or /github </li>
<li> The app can be accessed with any of the slash commands /gh or /github </li>
Copy link
Collaborator

Choose a reason for hiding this comment

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

@VipinDevelops These two should not be in bullet points. Can you please fix these ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tied to impove the overall style of command list here is new one do let me know if any changes required
image

@VipinDevelops
Copy link
Contributor Author

Screencast.from.2024-02-23.19-53-06.webm

@VipinDevelops
Copy link
Contributor Author

Hey @samad-yar-khan I update the Readme of the app according to new main modal also fixed few broken links

Copy link
Collaborator

@samad-yar-khan samad-yar-khan left a comment

Choose a reason for hiding this comment

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

@samad-yar-khan samad-yar-khan merged commit 24bea46 into RocketChat:main Mar 10, 2024
1 check passed
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