Skip to content
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

When editing Committee Chair & Member - edited chair/member persists and new is added #1675

Closed
RachelDCE opened this issue Aug 17, 2018 · 2 comments

Comments

@RachelDCE
Copy link

RachelDCE commented Aug 17, 2018

When editing Committee Chair & Member:

I edit the Name field and instead of replacing/changing that name- a new chair/member is added, so that I end up with multiple unintentional chairs/members. New ones added each time I try to edit.

screen shot 2018-08-17 at 3 33 33 pm

Tested in QA | 2017 Emory University. Version master updated 17 August 2018

@bess bess added this to the Release 2.0 milestone Aug 17, 2018
@bess bess added the UI label Aug 17, 2018
little9 added a commit that referenced this issue Aug 18, 2018
The committee chairs/members aren't updating
correctly. When updating an Etd this commit
should remove any existing members/chairs
and replace them with whatever is in your update
submission.

Before members/chairs where never being deleted
but appended to on each update so you would have
many duplicates. Whenever I run this code it seems
like sometimes the update hasn't been indexed
correctly because the members don't show up on
show page, but are present on the `Etd` object.

Connected to #1675
Connected to #1616
little9 added a commit that referenced this issue Aug 18, 2018
The committee chairs/members aren't updating
correctly. When updating an Etd this commit
should remove any existing members/chairs
and replace them with whatever is in your update
submission.

Before members/chairs where never being deleted
but appended to on each update so you would have
many duplicates. Whenever I run this code it seems
like sometimes the update hasn't been indexed
correctly because the members don't show up on
show page, but are present on the `Etd` object.

Connected to #1675
Connected to #1616
little9 added a commit that referenced this issue Aug 18, 2018
The committee chairs/members aren't updating
correctly. When updating an Etd this commit
should remove any existing members/chairs
and replace them with whatever is in your update
submission.

Before members/chairs where never being deleted
but appended to on each update so you would have
many duplicates. Whenever I run this code it seems
like sometimes the update hasn't been indexed
correctly because the members don't show up on
show page, but are present on the `Etd` object.

Connected to #1675
Connected to #1616
@little9 little9 removed their assignment Aug 20, 2018
@little9 little9 added backend and removed UI labels Aug 20, 2018
@val99erie val99erie self-assigned this Aug 21, 2018
val99erie added a commit that referenced this issue Aug 21, 2018
I cleared the committee members and chairs before passing the new params
from the form to the actor.

I fixed the methods that keep the committee_chair_name and
committee_members_names in sync with the committee_chair and
committee_members fields.
@val99erie
Copy link
Contributor

The solution that I implemented works properly from the point-of-view of a student, but it leaves behind orphaned records in fedora. After discussion with the team, since we are so close to the deadline, we decided that this is acceptable for now. I wrote story #1733 to fix this problem in a future release.

val99erie added a commit that referenced this issue Aug 21, 2018
Edit or remove committee members. Connected to stories #1616 and #1675.
@RachelDCE RachelDCE self-assigned this Aug 23, 2018
@RachelDCE
Copy link
Author

Was able to change and add committee chairs and members without unintended multiples showing.

Tested on QA | 2017 Emory University. Version v2.0.0-beta8 updated 22 August 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants