Commit e3769e1f authored by Etienne Baqué's avatar Etienne Baqué

Added feature flag around release link urls

Added feature flags in Release API entity class.
Removed mentions of mr and issues urls in API doc.
parent 1b354ed3
...@@ -122,10 +122,6 @@ Example response: ...@@ -122,10 +122,6 @@ Example response:
} }
] ]
}, },
"_links":{
"merge_requests_url": "https://gitlab.example.com/root/awesome_app/merge_requests?release_tag=v0.2&scope=all&state=opened",
"issues_url": "https://gitlab.example.com/root/awesome_app/issues?release_tag=v0.2&scope=all&state=opened"
}
}, },
{ {
"tag_name":"v0.1", "tag_name":"v0.1",
...@@ -182,10 +178,6 @@ Example response: ...@@ -182,10 +178,6 @@ Example response:
] ]
}, },
"_links":{
"merge_requests_url": "https://gitlab.example.com/root/awesome_app/merge_requests?release_tag=v0.1&scope=all&state=opened",
"issues_url": "https://gitlab.example.com/root/awesome_app/issues?release_tag=v0.1&scope=all&state=opened"
}
} }
] ]
``` ```
...@@ -297,10 +289,6 @@ Example response: ...@@ -297,10 +289,6 @@ Example response:
] ]
}, },
"_links":{
"merge_requests_url": "https://gitlab.example.com/root/awesome_app/merge_requests?release_tag=v0.1&scope=all&state=opened",
"issues_url": "https://gitlab.example.com/root/awesome_app/issues?release_tag=v0.1&scope=all&state=opened"
}
} }
``` ```
...@@ -426,10 +414,6 @@ Example response: ...@@ -426,10 +414,6 @@ Example response:
} }
] ]
}, },
"_links":{
"merge_requests_url": "https://gitlab.example.com/root/awesome_app/merge_requests?release_tag=v0.3&scope=all&state=opened",
"issues_url": "https://gitlab.example.com/root/awesome_app/issues?release_tag=v0.3&scope=all&state=opened"
}
} }
``` ```
...@@ -531,10 +515,6 @@ Example response: ...@@ -531,10 +515,6 @@ Example response:
] ]
}, },
"_links":{
"merge_requests_url": "https://gitlab.example.com/root/awesome_app/merge_requests?release_tag=v0.1&scope=all&state=opened",
"issues_url": "https://gitlab.example.com/root/awesome_app/issues?release_tag=v0.1&scope=all&state=opened"
}
} }
``` ```
...@@ -617,10 +597,6 @@ Example response: ...@@ -617,10 +597,6 @@ Example response:
] ]
}, },
"_links":{
"merge_requests_url": "https://gitlab.example.com/root/awesome_app/merge_requests?release_tag=v0.1&scope=all&state=opened",
"issues_url": "https://gitlab.example.com/root/awesome_app/issues?release_tag=v0.1&scope=all&state=opened"
}
} }
``` ```
......
...@@ -1315,8 +1315,8 @@ module API ...@@ -1315,8 +1315,8 @@ module API
end end
end end
expose :_links do expose :_links do
expose :merge_requests_url expose :merge_requests_url, if: -> (_) { release_mr_issue_urls_available? }
expose :issues_url expose :issues_url, if: -> (_) { release_mr_issue_urls_available? }
end end
private private
...@@ -1347,6 +1347,10 @@ module API ...@@ -1347,6 +1347,10 @@ module API
{ scope: 'all', state: 'opened', release_tag: object.tag } { scope: 'all', state: 'opened', release_tag: object.tag }
end end
def release_mr_issue_urls_available?
::Feature.enabled?(:release_mr_issue_urls, project)
end
def project def project
@project ||= object.project @project ||= object.project
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment