Skip to content

Fix URL parsing in IDE MRs

Paul Slaughter requested to merge 200087-fix-ide-nav-mr into master

What does this MR do?

This fixed a regression introduced by moving the merge_requests route under the /-/ namespace. Currently we used the merge request URL to parse for the project's full path with namespace. Now, we are using the full reference.

This is just a quick fix, and the best fix would be to migrate this bit over to GraphQL so we don't have to worry about the issues that come with parsing strings for specific values. #201762

Screenshots

Before After
ide_bug Screen_Shot_2020-02-03_at_9.00.47_AM

Does this MR meet the acceptance criteria?

Conformity

References

Edited by Paul Slaughter

Merge request reports