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 /autolink import/export capability #73

Open
levb opened this issue Aug 21, 2019 · 3 comments
Open

Add /autolink import/export capability #73

levb opened this issue Aug 21, 2019 · 3 comments
Labels
Difficulty/2:Medium Medium ticket Hacktoberfest Help Wanted Community help wanted Tech/Go Tech/ReactJS Type/Enhancement New feature or improvement of existing feature Up For Grabs Ready for help from the community. Removed when someone volunteers

Comments

@levb
Copy link
Contributor

levb commented Aug 21, 2019

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Plugin: Autolink" community channel.

New contributors please see our Developer's Guide and our Plugins Guide.


#75 implemented most of the backend functionality to support this feature. Before it can be merged, we would like to add the capability to upload/save local files. It can be done with client-side commands, see https://github.com/mattermost/mattermost-plugin-jira/blob/6bb4e25bbef08a43b26716626382f518f3e28df1/webapp/src/hooks/hooks.js#L47 for an example.

See also #71 that introduced the config read/write capability.

@levb levb added Enhancement Help Wanted Community help wanted Up For Grabs Ready for help from the community. Removed when someone volunteers labels Aug 21, 2019
@adarj
Copy link

adarj commented Aug 29, 2019

Hello, I'm interested in contributing this feature. Just to confirm, the import and export commands should allow the user to upload/download config files from their local machine, correct?

@levb
Copy link
Contributor Author

levb commented Aug 30, 2019

@adarj Welcome and thank you for taking this on!

RE: "allow the user to upload/download ... from local machine" - correct, but if that proves to be difficult to do directly, 0/5 an acceptable first implementation would be to export/import to mattermost files (uploaded as attachments for import, and for export, posted as a new message, with an attachment)

Also would be cool to have a "merge" capability in the import!

@levb levb added Work In Progress Not yet ready for review and removed Up For Grabs Ready for help from the community. Removed when someone volunteers labels Aug 30, 2019
adarj added a commit to adarj/mattermost-plugin-autolink that referenced this issue Sep 9, 2019
@levb
Copy link
Contributor Author

levb commented Oct 28, 2019

@adarj I am going to re-label this as a "Up For Grabs" to have someone else add the requested "File Import/Export" UI feature. It seems like an essential addition we should have before merging the existing PR. They can be implemented with client-side /commands.

I will explicitly request that it is done as a continuation of your PR, since most of the backend code is already there and you should get the attribution for it.

@levb levb added Up For Grabs Ready for help from the community. Removed when someone volunteers Hacktoberfest Tech/Go Tech/JavaScript Tech/ReactJS and removed Work In Progress Not yet ready for review labels Oct 28, 2019
@levb levb unassigned adarj Oct 28, 2019
@hanzei hanzei added Type/Enhancement New feature or improvement of existing feature and removed Enhancement labels Mar 7, 2020
@levb levb added the Difficulty/2:Medium Medium ticket label Apr 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Difficulty/2:Medium Medium ticket Hacktoberfest Help Wanted Community help wanted Tech/Go Tech/ReactJS Type/Enhancement New feature or improvement of existing feature Up For Grabs Ready for help from the community. Removed when someone volunteers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants