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

Test Allow headers for GET and HEAD #63

Merged
merged 8 commits into from
Jan 7, 2022
Merged

Test Allow headers for GET and HEAD #63

merged 8 commits into from
Jan 7, 2022

Conversation

kjetilk
Copy link
Collaborator

@kjetilk kjetilk commented Dec 24, 2021

No description provided.

@kjetilk kjetilk requested a review from edwardsph December 24, 2021 23:37
edwardsph
edwardsph approved these changes Jan 7, 2022
View reviewed changes
Copy link
Collaborator

@edwardsph edwardsph left a comment

Choose a reason for hiding this comment

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

LGTM with suggested changes

protocol/solid-protocol-test-manifest.ttl Outdated Show resolved Hide resolved
protocol/read-write-resource/read-method-allow.feature Outdated Show resolved Hide resolved
protocol/read-write-resource/read-method-allow.feature Outdated Show resolved Hide resolved
protocol/read-write-resource/read-method-allow.feature Show resolved Hide resolved
@kjetilk kjetilk merged commit c4f35ee into main Jan 7, 2022
@kjetilk kjetilk deleted the feature/read-method-allow branch January 7, 2022 21:58
@kjetilk
Copy link
Collaborator Author

kjetilk commented Jan 7, 2022

argh, forgot to push before I merged.

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

@edwardsph edwardsph edwardsph approved these changes

Assignees
No one assigned
Labels
Proposed Test Case
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kjetilk @edwardsph