-
Notifications
You must be signed in to change notification settings - Fork 3
ISS-120468: Update documentation for the marketplace related commands #95
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
Conversation
Added documentation regarding marketplace listings. https://app.devrev.ai/devrev/works/ISS-120468
Published docs preview URL: https://devrev-preview-e09bfb15-d3eb-45e0-8ebd-7a920ca281d9.docs.buildwithfern.com |
🌟 EkLine ReviewerHello! I’m here to help improve your docs. I’ve reviewed your pull request, and left in-line suggestions for quick fixes. For details, visit the Analytics Page. For questions or feedback, please email [email protected]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
EkLine
🚫 [EkLine] reported by reviewdog 🐶
Prefer English terms over Latin abbreviations. Use 'for example' instead of 'e.g.'. (EK00026)
properties (e.g. name, description, icon, etc.). |
📝 [EkLine] reported by reviewdog 🐶
Where possible, do not structure sentences in future tense. Use present tense instead. (EK00005)
To create a new marketplace listing, run the following command, which will start a __wizard__ to guide you through the |
📝 [EkLine] reported by reviewdog 🐶
Suggestions:
- Avoid using 'we' as it can come off as patronizing. For example, write 'The configuration is complete' rather than 'We are now done with the configuration.' (EK81039)
state), we can modify the listing by creating a new submission. First we have to create a submission which we can later |
📝 [EkLine] reported by reviewdog 🐶
Where possible, do not structure sentences in future tense. Use present tense instead. (EK00005)
To create a new submission for an existing marketplace listing, run the following command, which will start a __wizard__ |
📝 [EkLine] reported by reviewdog 🐶
Where possible, do not structure sentences in future tense. Use present tense instead. (EK00005)
Once a submission has been created, it can be updated by running the following command, which will start a __wizard__ to |
📝 [EkLine] reported by reviewdog 🐶
Avoid using 'we' as it can come off as patronizing. For example, write 'The configuration is complete' rather than 'We are now done with the configuration.' (EK81039)
The newly created marketplace submission is in the `draft` state. In order to publish it, we first have to transition it |
📝 [EkLine] reported by reviewdog 🐶
Where possible, do not structure sentences in future tense. Use present tense instead. (EK00005)
to`waiting_for_review` state. This can be done by running the following command, which will start a __wizard__ to guide |
📝 [EkLine] reported by reviewdog 🐶
Where possible, do not structure sentences in future tense. Use present tense instead. (EK00005)
While in review, the state of the submission is `in_review`. Once it is reviewed, the state will be changed to either |
📝 [EkLine] reported by reviewdog 🐶
Avoid using 'we' as it can come off as patronizing. For example, write 'The configuration is complete' rather than 'We are now done with the configuration.' (EK81039)
If the submission was rejected, we can transition it back to `draft` state and modify it to satisfy the requirements. |
📝 [EkLine] reported by reviewdog 🐶
Avoid using 'we' as it can come off as patronizing. For example, write 'The configuration is complete' rather than 'We are now done with the configuration.' (EK81039)
Once the submission is updated, we can transition it back to `waiting_for_review` state to be reviewed again. |
📝 [EkLine] reported by reviewdog 🐶
Suggestions:
- Where possible, do not structure sentences in future tense. Use present tense instead. (EK00005)
- Avoid using 'we' as it can come off as patronizing. For example, write 'The configuration is complete' rather than 'We are now done with the configuration.' (EK81039)
If the submission was approved, we can publish it using the following command, which will start a __wizard__ to guide |
📝 [EkLine] reported by reviewdog 🐶
Avoid using 'we' as it can come off as patronizing. For example, write 'The configuration is complete' rather than 'We are now done with the configuration.' (EK81039)
To make sure that marketplace item was published we can retrieve it using its ID: |
📝 [EkLine] reported by reviewdog 🐶
Where possible, do not structure sentences in future tense. Use present tense instead. (EK00005)
the `draft` or `rejected` state. You can delete a submission using the following command, which will start a __wizard__ |
📝 [EkLine] reported by reviewdog 🐶
Where possible, do not structure sentences in future tense. Use present tense instead. (EK00005)
Once a new version for a marketplace-item is published, the respective Snap-Ins will be auto-upgraded to the latest |
🚫 [EkLine] reported by reviewdog 🐶
Confirm the link. The URL 'https://github.com/devrev/snap-in-docs/blob/main/references/cli.md' seems inaccessible with error code 404. It might be incorrect or improperly formatted. (EK20001)
guide [visit here](https://github.com/devrev/snap-in-docs/blob/main/references/cli.md). |
🚫 [EkLine] reported by reviewdog 🐶
Confirm the link. The URL 'https://github.com/devrev/snap-in-docs/blob/main/start.md' seems inaccessible with error code 404. It might be incorrect or improperly formatted. (EK20001)
Snap-Ins, [follow this guide here](https://github.com/devrev/snap-in-docs/blob/main/start.md), to create your first |
Also address comments from EkLine. |
Published docs preview URL: https://devrev-preview-d1cd921f-94a4-4384-9856-d55ee8128409.docs.buildwithfern.com |
Published docs preview URL: https://devrev-preview-3e7a02cb-aa77-48a0-b235-6aed00f60caa.docs.buildwithfern.com |
Published docs preview URL: https://devrev-preview-65ef9da5-d0d7-4cc1-94ba-d3ddde4e2e55.docs.buildwithfern.com |
… into ISS-120468-marketplace-listings
Published docs preview URL: https://devrev-preview-9e039dc0-c3b2-4d6a-b5f9-4566c4884da5.docs.buildwithfern.com |
Published docs preview URL: https://devrev-preview-fe09c0d2-9f1c-4f4d-9dde-ac66beec64e5.docs.buildwithfern.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two main things to fix before merging:
- Put states in italics, not code phrase.
- Don't use "we"; use "you" instead.
Published docs preview URL: https://devrev-preview-cbc5314a-f64d-4b9c-a251-139686bc6f7a.docs.buildwithfern.com |
Added documentation regarding marketplace listings.
https://app.devrev.ai/devrev/works/ISS-120468