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
feat(ontology): update cardinality in resource class (DSP-1266) #377
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.
some tiny things
@@ -1,6 +1,7 @@ | |||
<div *ngIf="projectAdmin" class="desktop-only"> | |||
|
|||
<p class="note warning center">This is a first version of the ontology editor. Some features may not work as intended.</p> | |||
<p class="note warning center">This is a first version of the ontology editor. Some features may not work as |
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.
should we stick to "data model" instead of "ontology editor"?
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.
done in 4df6c48
@@ -164,8 +178,9 @@ export class PropertyFormComponent implements OnInit { | |||
* @returns Filtered list of options | |||
*/ | |||
filter(list: AutocompleteItem[], label: string) { | |||
console.log(list) |
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.
this console.log must be removed
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.
done in d74a77d
this.properties.setControl | ||
resourceClass.properties.forEach(prop => { | ||
this.properties[i] = new FormControl(prop); | ||
// this.properties[i].setValue(prop); |
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.
should we keep these commented lines?
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.
done in de4ac1f
resolves DSP-1266
It works with new created ontologies. Most of the test ontologies doesn't have a
lastModificationDate
and they can't be edited.Here are two screen records about "How to update cardinality":
update-res-class-card-interview.mp4
create-and-update-res-class-photo.mp4
The wording will be done in a separate task.