Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Mapping Notes #229

Closed
nichtich opened this issue Dec 19, 2018 · 1 comment
Closed

Mapping Notes #229

nichtich opened this issue Dec 19, 2018 · 1 comment
Labels
feature Additional functionality KENOM Required for KENOM instance
Milestone
0.7.0

Comments

@nichtich
Copy link
Member

nichtich commented Dec 19, 2018 •
edited
Loading

Feature request: Add a note to a mapping when mapping is saved.

Notes and comments on individual mappings are more complex (see #190) but limited notes (where author of note equals author of mapping) can be provided as following:

  • Always show JSKOS field note in the mapping editor if a mapping has this field
  • Provide simple text input field to add a note

Open question: how to show mapping notes in the MappingBrowser.
@nichtich nichtich added feature Additional functionality KENOM Required for KENOM instance labels Dec 19, 2018
@stefandesu
Copy link
Member

stefandesu commented Dec 19, 2018 •
edited
Loading

I would either show them using an info icon or by using a separate comment icon that is only shown when a comment exists.

Task List:

  • Support mapping notes

@nichtich nichtich added this to the 0.7.0 milestone Dec 19, 2018
stefandesu added a commit that referenced this issue Dec 27, 2018
@stefandesu
Support mapping notes (#229)
e079855
@stefandesu stefandesu changed the title Support mapping notes Mapping Notes Dec 27, 2018
@stefandesu stefandesu mentioned this issue Dec 27, 2018
Closed
2 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
feature Additional functionality KENOM Required for KENOM instance
Projects
None yet
Milestone
0.7.0
Development

No branches or pull requests
2 participants
@nichtich @stefandesu