Skip to content

Conversation

@ryanjcohen
Copy link
Contributor

Purpose of this pull request

This pull request (PR) modifies the app submission guidelines documentation to add guidelines specific to non-downloadable apps.

Affected pages

Links to Magento Open Source code

  • ...

- **Usage instructions**: Add instructions on how to use the app post-installation.
- For non-downloadable apps:
- All installation steps and usage instructions need to be included outside of the app submission's files at the application documentation URL provided in Exchange. Documentation within the app's files will not be accessible to users.
- `README` documentation is a best practice.
Copy link
Contributor

Choose a reason for hiding this comment

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

Delete this line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed


- Configuration files
- **Environment variables**: Provide a clear `.env.dist` file containing all needed keys used by YAML files. Remove any unused keys.
- **Environment variables**:
Copy link
Contributor

Choose a reason for hiding this comment

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

I can't comment near line 97, which says <Webhook />. It looks like a copy/paste error. Delete if it's not needed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ryanjcohen ryanjcohen requested a review from keharper October 29, 2025 18:01
@github-project-automation github-project-automation bot moved this from 📋 Needs Review to 👍 Approved in Commerce - Pull Requests Oct 29, 2025
@keharper keharper merged commit ec757d2 into AdobeDocs:main Oct 29, 2025
4 checks passed
@github-project-automation github-project-automation bot moved this from 👍 Approved to 🏁 Done in Commerce - Pull Requests Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🏁 Done

Development

Successfully merging this pull request may close these issues.

2 participants