-
Notifications
You must be signed in to change notification settings - Fork 5
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
Fix construct-matched-person lastName binding #270
Conversation
@blcham |
I worked on the JSON-LD output and identified several issues that I want to explain below:
|
@palagdan I believe we discussed that null values are not shown properly so PR should handle it as well. Thus not merging the PR yet. |
@blcham e.forEach((key, value) -> {
if(value != null){
model.add(
r,
getP(key),
value);
}
});
evidenceRe |
@palagdan yes, that is correct behaviour. Maybe it will be harder to visualize in backend and thus more appropriate solution would be also to include ordered column list as well ... so think about it if it is not needed to simplify frontend visualization of the table, closing this issue |
d3a4d7f
to
733b6ed
Compare
Resolves #269