earthchem / EarthChemAdmin2

Admin tools for Earthchem database person, organization, affiliation, citation
Apache License 2.0
0 stars 0 forks source link

Admin New expedition entry not working correctly #10

Closed annikakj closed 5 years ago

annikakj commented 5 years ago

Bai--When I tried to enter two new expeditions into Admin I discovered a problem:

I added the first expedition: AAR2011 and the DB gave it nr 81923 When I tried to add the second expedition: AAR2013, the DB tried to give it the same number and of course the DB will not add another expedition under the same nr. I have included 2 screenshots screen shot 2019-02-01 at 12 22 56 pm screen shot 2019-02-01 at 12 22 16 pm

annikakj commented 5 years ago

screen shot 2019-02-01 at 12 22 56 pm

bhchen8 commented 5 years ago

I addad "AAR2013" without problem. We cannot continue add expedition. After adding first expedition, the popup window have be closed, then click "New" button to add second expedition.

On Fri, Feb 1, 2019 at 1:19 PM annikakj notifications@github.com wrote:

[image: screen shot 2019-02-01 at 12 22 56 pm] https://user-images.githubusercontent.com/43789398/52141508-f2c1aa80-2623-11e9-82a7-96776b9109c6.png

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/iedadata/EarthChemAdmin2/issues/10#issuecomment-459817692, or mute the thread https://github.com/notifications/unsubscribe-auth/AGXmXlXfahfJ0xJFub_nR_T_P0Kcg1vdks5vJIUbgaJpZM4ae1Je .

bhchen8 commented 5 years ago

[image: image.png]

On Fri, Feb 1, 2019 at 2:39 PM Bai-Hao Chen bhchen@ldeo.columbia.edu wrote:

I addad "AAR2013" without problem. We cannot continue add expedition. After adding first expedition, the popup window have be closed, then click "New" button to add second expedition.

On Fri, Feb 1, 2019 at 1:19 PM annikakj notifications@github.com wrote:

[image: screen shot 2019-02-01 at 12 22 56 pm] https://user-images.githubusercontent.com/43789398/52141508-f2c1aa80-2623-11e9-82a7-96776b9109c6.png

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/iedadata/EarthChemAdmin2/issues/10#issuecomment-459817692, or mute the thread https://github.com/notifications/unsubscribe-auth/AGXmXlXfahfJ0xJFub_nR_T_P0Kcg1vdks5vJIUbgaJpZM4ae1Je .

annikakj commented 5 years ago

Close ticket. Will see if same problem happens again next time adding more than one expeditions

klehnert55 commented 5 years ago

We need to change the code. It is non-intuitive that I window needs to be closed before you can add another expedition. When you design an application, it is essential to carefully consider usability. Annika's problem with the entry of expeditions should be considered 'usability test results', and need to lead to decisions and improvements to overcome such usability problems.

bhchen8 commented 5 years ago

I will improve the interface.

On Fri, Feb 1, 2019 at 4:01 PM Kerstin Lehnert notifications@github.com wrote:

We need to change the code. It is non-intuitive that I window needs to be closed before you can add another expedition. When you design an application, it is essential to carefully consider usability. Annika's problem with the entry of expeditions should be considered 'usability test results', and need to lead to decisions and improvements to overcome such usability problems.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/iedadata/EarthChemAdmin2/issues/10#issuecomment-459866105, or mute the thread https://github.com/notifications/unsubscribe-auth/AGXmXiy6QOfXQSvhuAJqyN3IJVy-EcHFks5vJKsLgaJpZM4ae1Je .