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

Removed sample retains values from cloned proposal #763

Closed
simonfernandes opened this issue Dec 5, 2022 · 3 comments · Fixed by UserOfficeProject/user-office-core#157
Closed
Assignees
Labels
area: uop/stfc An issue relating to the STFC user office project. type: bug The issue describes a bug report.

Comments

@simonfernandes
Copy link
Contributor

When a user cloned a proposal, but then removed their sample, the old data was still present in the cloned proposal.

Probably applies to all generic template type questions (samples, grants, publications).

We have tried to use the cloned proposal as a start for the new proposal.

The original proposal required a deuteration facility, and we had to specify the deuterated samples, including the jpeg figures of structural formulas of the deuterated samples.

Upon cloning the proposal and changing the samples (removing the old and adding new) in the cloned proposal, and submitting the new/cloned proposal, we have found that the now spurious information about the deuterated samples (including the jpeg figures) was still present in the new proposal that has been modified from the cloned one, despite the complete removal of the deuterated samples and answering No for the need to use the deuteration facility.

Originates from ISIS Direct feedback document.

@simonfernandes simonfernandes added type: bug The issue describes a bug report. area: uop/stfc An issue relating to the STFC user office project. labels Dec 5, 2022
@simonfernandes simonfernandes added this to the Feedback: ISIS Direct milestone Dec 5, 2022
@simonfernandes simonfernandes changed the title Removing sample retains values from cloned proposal Removed sample retains values from cloned proposal Dec 5, 2022
@simonfernandes
Copy link
Contributor Author

Notes from meeting with Sukh, Simon, Philip and Emma 12th Dec 2022:

Although this only seemed to affect one user this time, many more users will be cloning proposals in the next round and so this could be an important issue to fix.

@TickleThePanda TickleThePanda modified the milestones: Feedback: ISIS Direct , 23/03/01 - ISIS-Direct Dec 14, 2022
@mutambaraf mutambaraf self-assigned this Dec 23, 2022
@mutambaraf
Copy link
Contributor

Managed to recreate this bug but it is affecting editing of proposals when you add new samples information in publication answers are being cleared such that when you get to publication it will show empty table but data will still be in the proposal. Will address this bug and see if this will happen again.

@simonfernandes
Copy link
Contributor Author

We will need to check whether this affects all generic template questions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: uop/stfc An issue relating to the STFC user office project. type: bug The issue describes a bug report.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants