-
Notifications
You must be signed in to change notification settings - Fork 51
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.