Project

General

Profile

Actions

Bug #19732

closed

Error message incorrectly displayed when adding in key-value metadata in WB2

Added by Sarah Zaranek over 1 year ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench2
Target version:
Story points:
0.5
Release relationship:
Auto

Description

So, if I go to Edit Collection on a Collection Page and Put in a Pair and hit Add - the pair adds but I can given this error message "the field is required" in red under the field and it keeps the key. I am not sure why it adds the key- but since there isn't a value (cause WB2 removed it), it thinks there is a problem. This is super confusing because it looks like it had a problem adding your key-value pair

I took a snap shot and will add in comments below.


Files


Subtasks 1 (0 open1 closed)

Task #19801: Review 19732-properties-error-fixResolvedPeter Amstutz12/07/2022Actions
Actions #1

Updated by Sarah Zaranek over 1 year ago

Here is a snapshot of me doing this on a collection in WB2.
https://workbench2.pirca.arvadosapi.com/collections/pirca-4zz18-6uvkbqb6rv2t6na --> this is the collection for completeness but I don't think this is a collection specific bug.

Actions #2

Updated by Sarah Zaranek over 1 year ago

It does the same thing with the new collection page via the New Button - I am assuming they use the same code, but just in case they don't.

Actions #3

Updated by Peter Amstutz over 1 year ago

  • Target version set to 2022-12-07 Sprint
Actions #4

Updated by Lucas Di Pentima over 1 year ago

  • Assigned To set to Lucas Di Pentima
Actions #5

Updated by Lucas Di Pentima over 1 year ago

  • Status changed from New to In Progress
Actions #6

Updated by Lucas Di Pentima over 1 year ago

  • Target version changed from 2022-12-07 Sprint to 2022-12-21 Sprint
Actions #7

Updated by Lucas Di Pentima over 1 year ago

  • Story points set to 0.5
Actions #8

Updated by Lucas Di Pentima over 1 year ago

Updates at arvados-workbench2|c064afd - branch 19732-properties-error-fix
Test run: developer-tests-workbench2: #1016

  • Removes the require validation on property key & value fields.
  • Makes the Add button to enable itself when both fields have some value, in addition to the form's validations.
  • Updates test case.
Actions #9

Updated by Lucas Di Pentima over 1 year ago

Rebased to the newest main at arvados-workbench2|74e6ba2

Test run: developer-tests-workbench2: #1017

Actions #10

Updated by Peter Amstutz over 1 year ago

Lucas Di Pentima wrote in #note-9:

Rebased to the newest main at arvados-workbench2|74e6ba2

Test run: developer-tests-workbench2: #1017

Tested with & without vocabulary, LGTM!

Actions #11

Updated by Lucas Di Pentima over 1 year ago

  • Status changed from In Progress to Resolved
Actions #12

Updated by Peter Amstutz over 1 year ago

  • Release set to 47
Actions

Also available in: Atom PDF