Epics relantioship API

What does this MR do?

Add API endpoints for listing, adding and removing related epics.

What are the relevant issue numbers?

closes #7326 (closed)

Does this MR meet the acceptance criteria?

• Changelog entry added, if necessary
• Documentation created/updated via this MR
• Tests added for this feature/bug
• Tested in all supported browsers
• Conforms to the code review guidelines
• Conforms to the merge request performance guidelines
• Conforms to the style guides
• Conforms to the database guides
• Link to e2e tests MR added if this MR has Requires e2e tests label. See the Test Planning Process.
• EE specific content should be in the top level /ee folder
• For a paid feature, have we considered GitLab.com plans, how it works for groups, and is there a design for promoting it to users who aren't on the correct plan?
• Security reports checked/validated by reviewer

changed milestone to %11.8

added Plan [DEPRECATED] backend direction epics + 1 deleted label

	2 Warnings
	This merge request is quite big (more than 539 lines changed), please consider splitting it into multiple merge requests.
	This merge request includes more than 10 commits. Please rebase these commits into a smaller number of commits.

	1 Message
	This merge request adds or changes files that require a review from the Docs team.

Docs review

The following files require a review from the Documentation team:

• doc/api/README.md
• doc/api/epic_links.md

When your content is ready for review, assign the MR to a technical writer according to the DevOps stages in the table below. If necessary, mention them in a comment explaining what needs to be reviewed.

Tech writer	Stage(s)
@marcia	~Create ~Release + development guidelines
@axil	~Distribution ~Gitaly ~Gitter ~Monitor ~Package ~Secure
@eread	~Manage ~Configure ~Geo ~Verify
@mikelewis	~Plan

You are welcome to mention them sooner if you have questions about writing or updating the documentation. GitLabbers are also welcome to use the #docs channel on Slack.

If you are not sure which category the change falls within, or the change is not part of one of these categories, mention one of the usernames above.

Generated by Danger

added 1 commit

• 6f661827 - Allow to list/add/remove epic relations

Compare with previous version

@jprovaznik can you review this one please?

It is missing changelog and documentation.

assigned to @jprovaznik

added workflowready for review label

Thanks @felipe_artur, nice work. I left some comments inline.

assigned to @felipe_artur

assigned to @jprovaznik

added 1 commit

• ee0db04f - Address review comments

Compare with previous version

thanks for the review @jprovaznik, addressed all comments. Do you mind taking another look?

unmarked as a Work In Progress

added 2 commits

• cc24c741 - Add docs
• 44d59994 - Add changelog and a comment

Compare with previous version

Thanks @felipe_artur, one more comment about deletion, otherwise LGTM.

assigned to @felipe_artur

@jprovaznik

Because we define "parent" of an epic by creating a new association on parent epic (instead of updating child epic with epic's parent id)

I think i did not understand. When we associate a child epic we update its parent_id with the epic id.

added 1 commit

• e9b8f06d - Improve delete endpoint

Compare with previous version

assigned to @smcgivern

@felipe_artur I think what @jprovaznik is saying is that to add a link, we have this endpoint:

Assign a child epic

This operates on the parent, not the child. So it would make sense for the deletion to also act on the parent. (I agree.)

assigned to @felipe_artur

resolved all discussions

added 1 commit

• 998c6a11 - Address review comments

Compare with previous version

added 1 commit

• cea591fb - fix failing spec

Compare with previous version

@mikelewis Can you review docs please?

I added documentation for the new endpoint https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/9188/diffs#0308ba93d3c93bb1ee8936f3146f182cce810f0e_0_1

added documentation label

added 1 commit

• eff9e0db - Re-use epic fixture on specs

Compare with previous version

resolved all discussions

approved this merge request

assigned to @mikelewis

mentioned in issue #7328 (closed)

@felipe_artur I left some comments and suggestions on the docs.

added 1373 commits

• eff9e0db...32d37bc9 - 1372 commits from branch master
• b743115e - Merge master

Compare with previous version

added 1 commit

• 767d350a - Apply suggestion to doc/api/epic_links.md

Compare with previous version

added 1 commit

• 19086f48 - Apply suggestion to doc/api/epic_links.md

Compare with previous version

Thanks @mikelewis. I made the changes you requested.

Can you check again please?

added 1 commit

• 125738bb - Fix docs and rubocop

Compare with previous version

added 344 commits

• 125738bb...71e6a91a - 343 commits from branch master
• 2b39b6b2 - Merge branch 'master' into issue_7326

Compare with previous version

added 1 commit

• 639e7ada - Fix specs

Compare with previous version

resolved all discussions

added 1 commit

• 7c795b80 - Minor edit to intro

Compare with previous version

LGTM, @felipe_artur!

assigned to @felipe_artur

Thank you @mikelewis!

@smcgivern Assigning back to you.

assigned to @smcgivern

merged

mentioned in commit 476af7a9

Thanks @felipe_artur!

added devopsplan label

added Enterprise Edition label

mentioned in issue #26548

added Category:Portfolio Management label

removed Plan [DEPRECATED] label