From b166dc624d756219980740ae59a368839f370765 Mon Sep 17 00:00:00 2001 From: Uditi Mehta Date: Mon, 21 Oct 2024 10:22:43 -0400 Subject: [PATCH] hide categories for draft registrations --- .../addon/drafts/draft/metadata/controller.ts | 2 ++ .../addon/drafts/draft/metadata/template.hbs | 14 ++++++++------ .../registries/acceptance/branded/new-test.ts | 1 - .../registries/acceptance/draft/draft-test.ts | 16 ---------------- 4 files changed, 10 insertions(+), 23 deletions(-) diff --git a/lib/registries/addon/drafts/draft/metadata/controller.ts b/lib/registries/addon/drafts/draft/metadata/controller.ts index 6d72f08640..77dd69e051 100644 --- a/lib/registries/addon/drafts/draft/metadata/controller.ts +++ b/lib/registries/addon/drafts/draft/metadata/controller.ts @@ -19,6 +19,8 @@ export default class RegistriesDraftMetadata extends Controller { categoryOptions = Object.values(NodeCategory); showAddContributorWidget = false; + hideCategories = true; + @not('media.isDesktop') showMobileView!: boolean; osfUrl = config.OSF.url; diff --git a/lib/registries/addon/drafts/draft/metadata/template.hbs b/lib/registries/addon/drafts/draft/metadata/template.hbs index 42658d9a0a..71ce807511 100644 --- a/lib/registries/addon/drafts/draft/metadata/template.hbs +++ b/lib/registries/addon/drafts/draft/metadata/template.hbs @@ -83,12 +83,14 @@ /> {{/if}} {{/let}} - - - + {{#unless this.hideCategories}} + + + + {{/unless}}
{{t 'registries.registration_metadata.affiliated_institutions'}} diff --git a/tests/engines/registries/acceptance/branded/new-test.ts b/tests/engines/registries/acceptance/branded/new-test.ts index 543c18ad5a..77446c55ae 100644 --- a/tests/engines/registries/acceptance/branded/new-test.ts +++ b/tests/engines/registries/acceptance/branded/new-test.ts @@ -114,6 +114,5 @@ module('Registries | Acceptance | branded.new', hooks => { assert.dom('[data-test-contributor-link]').exists({ count: 1 }, 'Only one contributor'); assert.dom('[data-test-contributor-permission]').containsText('Administrator', 'user is admin'); assert.dom('[data-test-contributor-citation]').containsText('Yes', 'user is bibliographic'); - assert.dom('[data-test-option="uncategorized"]').exists('Category is uncategorized by default'); }); }); diff --git a/tests/engines/registries/acceptance/draft/draft-test.ts b/tests/engines/registries/acceptance/draft/draft-test.ts index ed831697f5..d27c1e590d 100644 --- a/tests/engines/registries/acceptance/draft/draft-test.ts +++ b/tests/engines/registries/acceptance/draft/draft-test.ts @@ -945,16 +945,6 @@ module('Registries | Acceptance | draft form', hooks => { assert.dom('[data-test-validation-errors="description"]') .exists('error in description appears after removing valid string to blank string'); - // Choose category and add a tag - await click('[data-test-metadata-category] > div'); - await percySnapshot('Registries | Acceptance | draft form | metadata editing | metadata: categories opened'); - assert.dom('[data-option-index="1"]').containsText('Other'); - await click('[data-option-index="1"]'); - - await click('[data-test-metadata-tags]'); - await fillIn('[data-test-metadata-tags] input', 'ragtagbag'); - await triggerKeyEvent('[data-test-metadata-tags] input', 'keydown', 'Enter'); - // No errors for nodelicense fields assert.dom('[data-test-validation-errors="subjects"]') .doesNotExist('no error for required fields that user has yet to change: subjects'); @@ -970,12 +960,6 @@ module('Registries | Acceptance | draft form', hooks => { assert.dom('[data-test-validation-errors="subjects"]'); assert.dom('[data-test-validation-errors="license"]'); - // Category and tag added appear on review page - assert.dom('[data-test-review-response="category"]') - .containsText('Other', 'category that was selected in metadata page shows up in review'); - assert.dom('[data-test-tags-widget-tag="ragtagbag"]') - .exists('tag added in metadata shows up in review page'); - // Return to Metadata page and address errors for subjects and license await click('[data-test-link="metadata"]');