diff --git a/wherehows-web/app/components/dataset-compliance.js b/wherehows-web/app/components/dataset-compliance.js index 6a47de37db..2a59c7018e 100644 --- a/wherehows-web/app/components/dataset-compliance.js +++ b/wherehows-web/app/components/dataset-compliance.js @@ -68,7 +68,7 @@ export default Component.extend({ ).mapBy('fieldName'); if (fieldNamesAreUnique(fieldNames.sort())) { - return; + return set(this, '_hasBadData', false); } // Flag this component's data as problematic diff --git a/wherehows-web/app/templates/components/dataset-author.hbs b/wherehows-web/app/templates/components/dataset-author.hbs index f1e42368d1..b9bd823e14 100644 --- a/wherehows-web/app/templates/components/dataset-author.hbs +++ b/wherehows-web/app/templates/components/dataset-author.hbs @@ -33,9 +33,13 @@ of Owner Type Owner +
+

+ Only owners that are not sourced from SCM can be removed + from this list. To remove any such owner(s), please make the change at the + source.

- {{#if errorMessage}} {{/if}} @@ -51,10 +55,70 @@ Full Name ID Type Source - Owner Type + + Owner Type + + + + + + + + + Confirmed Remove - {{/each}} - - \ No newline at end of file + + + {{#each owners as |owner|}} + + + + + + {{owner.name}} + {{owner.idType}} + {{owner.source}} + + {{ember-selector + values=ownerTypes + selected=owner.type + change=(action "updateOwnerType" owner)}} + + + {{input + type="checkbox" + title=(if owner.confirmedBy owner.confirmedBy "Not confirmed") + checked=(readonly owner.confirmedBy) + change=(action "confirmOwner" owner)}} + + + + + + + {{/each}} + + + diff --git a/wherehows-web/app/templates/components/dataset-compliance.hbs b/wherehows-web/app/templates/components/dataset-compliance.hbs index 76145f4df3..5292f0b0cc 100644 --- a/wherehows-web/app/templates/components/dataset-compliance.hbs +++ b/wherehows-web/app/templates/components/dataset-compliance.hbs @@ -29,6 +29,7 @@
+

Can any fields in this dataset's schema be used to identify a member?

@@ -64,6 +65,7 @@ {{#if showOrgPrompt}}
@@ -97,6 +99,7 @@ {{#if showGroupPrompt}}