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

Adding backward compatibility for 5.38 #922

Merged
merged 3 commits into from Aug 11, 2021

Conversation

jespino
Copy link
Member

@jespino jespino commented Aug 10, 2021

No description provided.

@jespino jespino requested a review from a team as a code owner August 10, 2021 20:15
@jespino jespino requested review from wiggin77 and chenilim and removed request for a team August 10, 2021 20:15
Copy link
Member

@chenilim chenilim left a comment

Choose a reason for hiding this comment

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

Looks like the url needs to be updated. Another issue is the Compass icon is not visible in the channel header on 5.38.

} else {
this.channelHeaderButtonId = registry.registerChannelHeaderButtonAction(<FocalboardIcon/>, () => {
const currentChannel = store.getState().entities.channels.currentChannelId
window.open(`${window.location.origin}/plug/focalboard/workspace/${currentChannel}`)
Copy link
Member

Choose a reason for hiding this comment

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

I believe this should be /plugins/focalboard/...

Copy link
Member Author

Choose a reason for hiding this comment

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

The problem was with other thing, see the commit that I included.

@jespino jespino requested a review from chenilim August 11, 2021 08:35
@jespino jespino dismissed chenilim’s stale review August 11, 2021 09:10

Addressed the changes and reviewed by Miguel

@jespino jespino merged commit ca40dda into mattermost:main Aug 11, 2021
jespino added a commit that referenced this pull request Aug 11, 2021
* Adding backward compatibility for 5.38

* Fixing problem with urls
@mgdelacroix mgdelacroix added the CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone label Aug 11, 2021
@ogi-m ogi-m added the 2: QA Review Requires review by a QA tester label Aug 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2: QA Review Requires review by a QA tester CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants