-
Notifications
You must be signed in to change notification settings - Fork 52
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 #407 by using authorLabel instead of author #408
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no tests fail, you can merge this @white-gecko
@shinobu did you also check if this looks good in the UI? |
no I didnt, i'm going to check it soon |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
every single author entry disappears (they are all correctly shown prior)
-> this needs to be fixed if the issue actually still exists @k00ni
I wrote the issues with this pull request into the issue #407 . |
I am not sure about this issue anymore. What would be your preferred solution @shinobu? |
I guess the best solution will be to just fix the doap.n3 and turn the 2 authorlabel's entries into one @k00ni . |
Ok, so be it :) |
Background information can be found in issue #407.