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

Only the first isReplacedBy is shown in the deprecated alert even if there are more values #612

Closed
henriyli opened this issue Jun 2, 2017 · 0 comments
Assignees
Milestone

Comments

@henriyli
Copy link
Collaborator

henriyli commented Jun 2, 2017

At which URL did you encounter the problem?

http://dev.finto.fi/yso/en/page/?uri=p13752

What steps will reproduce the problem?

  1. Open the link above or any other deprecated concept that is replaced by more than one concept.

What is the expected output? What do you see instead?

Expected to see both replacing concepts "health and wellness sector" and "health sector" in the deprecation alert box. However, only one the first one is shown in the alert.

Could be fixed by iterating through all the isReplacedBy property values and adding one row for each. Or separating the concepts with a comma. This would be better, but it would also require pluralizing the translation strings.

What browser did you use? (eg. Firefox, Chrome, Safari, Internet explorer)

Chrome

@henriyli henriyli added this to the 1.10 milestone Jun 2, 2017
@henriyli henriyli self-assigned this Jun 2, 2017
henriyli pushed a commit that referenced this issue Jun 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant