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.

Sign up for GitHub

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

Jump to bottom

Add section for acl Link relation type #199

Merged
merged 3 commits into from
Oct 11, 2020
Merged

Add section for acl Link relation type #199

merged 3 commits into from
Oct 11, 2020

Conversation

csarven
Copy link
Member

@csarven csarven commented Sep 16, 2020

With intent to submit to IANA for review, approval, and inclusion in the IANA Link Relations registry.

@csarven csarven added this to the ~First Public Working Draft milestone Sep 16, 2020
@csarven csarven closed this Sep 16, 2020
@csarven csarven deleted the feature/http-definitions branch September 16, 2020 10:34
@csarven csarven restored the feature/http-definitions branch September 16, 2020 10:36
@csarven csarven reopened this Sep 16, 2020
@kidehen
Copy link

kidehen commented Sep 16, 2020

I also recommend approval of these changes.

acoburn
acoburn approved these changes Sep 16, 2020
View reviewed changes
Copy link
Member

@acoburn acoburn left a comment

Choose a reason for hiding this comment

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

LGTM

justinwb
justinwb approved these changes Sep 16, 2020
View reviewed changes
Copy link
Member

@justinwb justinwb left a comment

Choose a reason for hiding this comment

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

✅

@csarven csarven requested a review from a team September 21, 2020 09:20
@justinwb justinwb requested a review from RubenVerborgh October 6, 2020 15:40
@csarven csarven merged commit 01c8cf3 into master Oct 11, 2020
@csarven
Copy link
Member Author

csarven commented Oct 11, 2020

Great, thanks all! All approved and feedback addressed.

@csarven csarven deleted the feature/http-definitions branch December 4, 2020 11:49
@csarven csarven mentioned this pull request Jan 28, 2021
Closed
@matthieubosquet matthieubosquet mentioned this pull request Jun 21, 2021
Open
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bblfish bblfish bblfish approved these changes

@acoburn acoburn acoburn approved these changes

@justinwb justinwb justinwb approved these changes

@melvincarvalho melvincarvalho Awaiting requested review from melvincarvalho

@michielbdejong michielbdejong Awaiting requested review from michielbdejong

@ericprud ericprud Awaiting requested review from ericprud

@RubenVerborgh RubenVerborgh Awaiting requested review from RubenVerborgh

Assignees
No one assigned
Labels
topic: resource access
Projects
Specification
Status: Done
Milestone
~First Public Working Draft
Development

Successfully merging this pull request may close these issues.
6 participants
@csarven @kidehen @bblfish @justinwb @RubenVerborgh @acoburn