Skip to content
This repository has been archived by the owner on Mar 14, 2022. It is now read-only.

Add Location header to created response #120

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add Location header to created response #120

wants to merge 1 commit into from

Conversation

jcoyne
Copy link
Contributor

@jcoyne jcoyne commented Jan 30, 2018

Fixes #116

@jcoyne jcoyne added the review label Jan 30, 2018
@jcoyne jcoyne added backlog and removed duplicate labels Jan 31, 2018
@jcoyne jcoyne force-pushed the location branch 2 times, most recently from ee5c33e to d3e4f3b Compare February 1, 2018 04:03
@jcoyne
Copy link
Contributor Author

jcoyne commented Feb 1, 2018

Having this PR merged would make this test code much simpler:
https://github.com/sul-dlss-labs/taco/blob/master/test/integration_test.go#L78-L82

@jcoyne jcoyne force-pushed the location branch 5 times, most recently from 7b9936b to ba052e1 Compare February 21, 2018 15:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@eefahy eefahy Awaiting requested review from eefahy

@aaron-collier aaron-collier Awaiting requested review from aaron-collier

@cmharlow cmharlow Awaiting requested review from cmharlow

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
backlog review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jcoyne @cmharlow