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

[18.05] Remove defunct sharing column link #6177

Merged
merged 1 commit into from May 26, 2018

Conversation

guerler
Copy link
Contributor

@guerler guerler commented May 18, 2018

Resolves #5025. This is a minor regression and has to be revisited when we transformed the grid components to vue. This should be handled at the front-end in the future. For now however this PR removes the useless link.

@guerler guerler added this to the 18.09 milestone May 18, 2018
@guerler guerler requested a review from martenson May 18, 2018 17:32
@guerler guerler changed the base branch from dev to release_18.05 May 22, 2018 20:17
@guerler guerler changed the base branch from release_18.05 to dev May 22, 2018 20:17
@guerler guerler changed the base branch from dev to release_18.05 May 22, 2018 20:21
@guerler guerler changed the title Remove defunct sharing column link [18.05] Remove defunct sharing column link May 22, 2018
@dannon
Copy link
Member

dannon commented May 23, 2018

@guerler The milestone set here was 18.09, but it's still targeting release_18.05; which one should it be?

@guerler guerler modified the milestones: 18.09, 18.05 May 23, 2018
@guerler
Copy link
Contributor Author

guerler commented May 23, 2018

Thanks @dannon. Fixed the target. It would be most useful in 18.05, but its not a serious issue.

@martenson martenson merged commit 221897b into galaxyproject:release_18.05 May 26, 2018
@martenson martenson modified the milestones: 18.05, 18.09 May 26, 2018
@martenson
Copy link
Member

@guerler thanks!

sidenote: the milestone should be 18.09 in this case, since otherwise it wouldn't make it to any release notes

@guerler guerler deleted the remove_defunct_link branch February 19, 2020 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants