Skip to content

Conversation

@OlufunkeMoronfolu
Copy link
Collaborator

A core deletion was made on this doc. This decision was made due to the deprecation notice on September 2024 and its subsequent absence in the UI.

Copy link
Collaborator

@MarkvanMents MarkvanMents left a comment

Choose a reason for hiding this comment

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

Hi Olu (@OlufunkeMoronfolu)
This is a really good improvement. The language is better and clearer and you have caught various changes which seem to have slipped through.
I have made a number of comments, but generally this is fine and I am being especially critical.

## Adding New Marketplace Content {#adding}

To get started, click **Add Content** in the top bar of the Marketplace home screen. Follow the steps in the sections below to add and submit the content.
To get started, click **Add Content** in the Marketplace home screen. Follow the steps in the sections below to add and submit the content.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Good change - but I would use "on the … screen"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, "on" is correct. Thanks for catching that!


* **Save Draft** to save the details you have entered so far to the draft (which you can access via the [My Drafts](/appstore/home-page/#my-drafts) link in the top bar)
* **Save & Continue** to go to the next page of the upload flow
* **Save Draft & Exit** to store the details you have entered so far as a draft (accessible via [My Drafts](/appstore/home-page/#my-drafts)) and exit the upload flow
Copy link
Collaborator

Choose a reason for hiding this comment

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

UX update well caught.
Even though it is repetition, I would keep "save the details" to be consistent with what the button says.
Other changes are nice.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Noted. Done.
Thanks.

* To include the repo's *README.md* file on the component's [Documentation](#doc) tab, make sure you check the **Import Documentation** box
* Click **OK** to finish

Optionally select **Compatibility** to test whether your component works with the React Client in Mendix Studio Pro 10.7 or higher. For details, see the [Widgets](/appstore/guidelines-content-creators/#github) section in *Mendix React Client*. Note that this option is only available if you selected **Widget** as the **Content Type** on the **General** page.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Good addition
Perhaps start with "If you selected Widget as the Content Type then you can click Compatibility…" to save me reading it to the end and then discovering it doesn't apply to me 😀

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed. Thanks

* **My Content**
* **Company Content**
* **Content Groups** (note that if an existing Marketplace component is assigned to a [content group](/appstore/home-page/#content-groups) as specific content group [content](/appstore/home-page/#group-content), you can only update the component if you are a member of that group)
* **Content Group** (note that if an existing Marketplace component is assigned to a [content group](/appstore/home-page/#content-groups) as specific content group [content](/appstore/home-page/#group-content), you can only update the component if you are a member of that group)
Copy link
Collaborator

Choose a reason for hiding this comment

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

If there is a note, that is usually an indication to me to put it in a blue {{% alert box. Would that be a good idea here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed. Thank you.

@OlufunkeMoronfolu
Copy link
Collaborator Author

Hello @MarkvanMents , I have incorporated the feedback and it is now ready for review.
Thanks for your detailed review.

Copy link
Collaborator

@MarkvanMents MarkvanMents left a comment

Choose a reason for hiding this comment

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

Thanks for this, Olu.
Approving.
You can merge this to development.

@OlufunkeMoronfolu OlufunkeMoronfolu merged commit c26893d into development May 26, 2025
2 checks passed
@github-actions github-actions bot deleted the olu-proofread-marketplace-upload branch November 7, 2025 09:35
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.

3 participants