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

Add 'Limit' header to LDP GET #958

Closed
wants to merge 2 commits into from
Closed

Conversation

awoods
Copy link

@awoods awoods commented Nov 28, 2015

Limit HTML UI to 99 children
Add 'numberOfChildren' triple to containers

Resolves: https://jira.duraspace.org/browse/FCREPO-1808

Limit HTML UI to 99 children
Add 'numberOfChildren' triple to containers

Resolves: https://jira.duraspace.org/browse/FCREPO-1808
}

protected Response getContent(final String rangeValue,
final int limit,
Copy link
Contributor

Choose a reason for hiding this comment

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

Javadocs. What is the unit for limit?

@ajs6f, please do not merge this commit before squashing it.
I will do it if you like.
@awoods
Copy link
Author

awoods commented Dec 10, 2015

@acoburn, I believe this ticket is ready to go. @ajs6f, you are not suggesting holding this PR on the non-functional, syntactic polish of Java8/Jena3, are you?
p.s. These two commits should be squashed.

@acoburn
Copy link
Contributor

acoburn commented Dec 11, 2015

merged via 03adc0b

@acoburn acoburn closed this Dec 11, 2015
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