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

Add v1.1.0 of mattermost-plugin-apps to the Marketplace #281

Closed
wants to merge 3 commits into from

Conversation

levb
Copy link

@levb levb commented May 24, 2022

@levb levb added 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester labels May 24, 2022
"homepage_url": "https://mattermost.com/marketplace/app-framework",
"icon_data": "",
"download_url": "https://plugins-store.test.mattermost.com/release/mattermost-plugin-apps-v1.1.0.tar.gz",
"release_notes_url": "https://github.com/mattermost/mattermost-plugin-apps/releases/tag/v1.0.1",
Copy link
Member

Choose a reason for hiding this comment

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

Looks like this was missed in mattermost/mattermost-plugin-apps#345. Maybe something we can automate?

Suggested change
"release_notes_url": "https://github.com/mattermost/mattermost-plugin-apps/releases/tag/v1.0.1",
"release_notes_url": "https://github.com/mattermost/mattermost-plugin-apps/releases/tag/v1.1.0",

Copy link
Author

Choose a reason for hiding this comment

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

Right, was overlooked in mattermost/mattermost-plugin-apps#345.

I wonder if we can just autogenerate release_notes_url and leave it empty in plugin.json

Can we leave with the incorrect URL in the plugin itself and not re-cut the release? I can fix this PR so that we have the right URL in the marketplace.

Copy link
Member

Choose a reason for hiding this comment

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

Can we leave with the incorrect URL in the plugin itself and not re-cut the release? I can fix this PR so that we have the right URL in the marketplace.

@levb Yep works for me

Copy link
Member

@mickmister mickmister May 24, 2022

Choose a reason for hiding this comment

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

I wonder if we can just autogenerate release_notes_url and leave it empty in plugin.json

This would be a "update all plugin repos" change, which has proven to not scale well. Having a common tooling source may be a better way to go about managing the plugin repository tooling. @hanzei What do you think about this? Something like a shared CLI defined in a different repository. Custom make commands for each project would still need to be supported.

@levb levb requested a review from mickmister May 24, 2022 16:58
@mickmister
Copy link
Member

Should this be merged/closed?

@hanzei hanzei removed 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester labels Sep 18, 2022
@hanzei hanzei closed this Sep 18, 2022
@hanzei
Copy link
Collaborator

hanzei commented Sep 18, 2022

We don't need this as the plugin is pre packaged

@hanzei hanzei deleted the add_mattermost-plugin-apps_v1.1.0 branch September 18, 2022 03:23
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.

4 participants