Bumped to v1.2.0

[levb]

Bumped release number.

First part of #51

[hanzei]

Did you run make apply?

Edit: On my machine make apply is broken: #54

Maybe there is no need to run it.

[aaronrothschild]

LGTM

[jfrerich]

Perhaps they did not you the manifest files and that is something we should incorporate into the plugin to match it up with the other plugins.

[levb]

@hanzei I ran make all - it usually updates everything. 0/5 that should be the standard operating procedure, not a specific make apply. Not sure if it needs the extra files, @mickmister @jfrerich can you comment?

[DHaussermann]

Not sure if this is helpful to the discussion above but - when I run make it seems to remove and re-add all NPM dependencies as if it does a make clean every time. I thought there may be some reason this was being done intentionally.

[DHaussermann]

Tested and passed

• Ran through end to End testing
• Tested on Confluence Cloud and Server
• Tested upgrade and confined version bump successful
  LGTM! Will re-test if there any changes based on Dev review.

[hanzei]

Blocking because #53 hasn't been merged

[DHaussermann]

End to end testing was already done on this. #52 (review) I was unaware that we were holding off for this PR #53

Once it's merged I can quickly double check the icon.

[hanzei]

@levb I'm unsure if the CI correctly build and deploy github releases with the current setup in https://github.com/mattermost/mattermost-plugin-confluence/blob/master/.circleci/config.yml. Maybe it's more safe to sync with https://github.com/mattermost/mattermost-plugin-demo/blob/master/.circleci/config.yml first. Let me know if I can help.

[jfrerich]

@hanzei, we discussed the issue of the circleCI and makefile differences. For the release and adding to marketplace, we don't have to have the circleCi fixes. I'll address the circleCi fixes in #54

[hanzei]

@jfrerich I've noticed that v1.2.0 is missing from https://github.com/mattermost/mattermost-plugin-confluence/releases. Do you need help with cutting it?