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.

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

Adding eventID header (FCREPO-1507) #73

Closed
wants to merge 2 commits into from
Closed

Conversation

escowles
Copy link
Contributor

@escowles escowles commented Jun 4, 2015

@acoburn
Copy link
Contributor

acoburn commented Jun 4, 2015

There are some existing integration tests that ensure that these constants match the constants defined in the fcrepo4 codebase. Could a similar test be added for this?

For completeness, it would also be good to add this to the unit test here: https://github.com/fcrepo4/fcrepo-camel/blob/master/src/test/java/org/fcrepo/camel/FcrepoConstantsTest.java

@awoods
Copy link

awoods commented Jun 4, 2015

I suspect these Travis failures will go away once the fcrepo PR has been merged (pending response to code review comments).

@awoods
Copy link

awoods commented Jun 5, 2015

Resolved with: c301fff

@awoods awoods closed this Jun 5, 2015
@awoods awoods deleted the audit-identifiers branch June 5, 2015 20:01
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.

None yet

3 participants