Skip to content

Comments

fix(guidelines): fix link relations deep links#101

Open
Rawezhham176 wants to merge 3 commits intootto-de:mainfrom
Rawezhham176:fix/link-relations-deep-links
Open

fix(guidelines): fix link relations deep links#101
Rawezhham176 wants to merge 3 commits intootto-de:mainfrom
Rawezhham176:fix/link-relations-deep-links

Conversation

@Rawezhham176
Copy link

Changelog:

Update

  • Update the review changes from last commit.

@BirgitBader BirgitBader changed the title Fix/link relations deep links fix(guidelines): fix link relations deep links Feb 23, 2026
@BirgitBader
Copy link
Contributor

BirgitBader commented Feb 23, 2026

@maxedenharter0507 and @Rawezhham176
Please specify the changelog entry. For users following a public repository it's hard to grasp what's been done. (Even for me ;-))

I did not review the URL changes - I left that for Max ;-) , but left an inline comment.

Comment on lines -17 to -19
The URI should be in the same URL namespace as the API endpoints.
For example, if all API endpoints are located at `https://api.otto.de/payment/`, the custom link relation URIs should als be located at the same context path (e.g., `https://api.otto.de/payment/link-relations/payment-method`).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lines 17 to 19 have been removed in a guideline. This is a guideline update that needs to be reflected in the changelog.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants