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

Strip legacy history structure view #6241

Merged
merged 2 commits into from
May 30, 2018

Conversation

dannon
Copy link
Member

@dannon dannon commented May 29, 2018

This removes another (web, not API) controller endpoint and the associated mako. It has been non-navigable from the UI for quite some time now.

Per discussions, we're going to start from scratch here.

@galaxybot galaxybot added this to the 18.09 milestone May 29, 2018
@bgruening
Copy link
Member

Are we going to replace this with something new in this release cycle? If not I would leave it in as long as we have a replacement.

@martenson
Copy link
Member

@bgruening this cannot be reached from the UI and there are no plans to fix it atm

@dannon
Copy link
Member Author

dannon commented May 29, 2018

@bgruening Yep, @martenson Is correct on both points there. We're going to develop something similar in this coming release cycle, but will start from scratch using the current client stack.

@martenson martenson merged commit 8fcc933 into galaxyproject:dev May 30, 2018
@dannon dannon added the kind/refactoring cleanup or refactoring of existing code, no functional changes label May 30, 2018
@galaxyproject galaxyproject deleted a comment from galaxybot May 30, 2018
@mvdbeek
Copy link
Member

mvdbeek commented May 31, 2018

Is it this structure ?

screen shot 2018-05-31 at 17 29 08

That one is quite useful to me and was working in 18.05 ? Do we have an issue to track progress for this ?

@dannon
Copy link
Member Author

dannon commented May 31, 2018

@mvdbeek No, it is not that structure view. That said, I'll have the future work I mentioned written up this week.

@mvdbeek
Copy link
Member

mvdbeek commented May 31, 2018

OK, cool, sorry for the noise

@dannon
Copy link
Member Author

dannon commented May 31, 2018

No worries at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/refactoring cleanup or refactoring of existing code, no functional changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants