Ensure item
is also embedded in submission form
#3016
Merged
+2
−2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
References
submission.cy.ts
)Description
On
main
anddspace-7_x
after #2988 was merged, the submission e2e tests began to fail (almost randomly).After digging into the issue locally, I found the cause was that the Submission Form would occasionally hang indefinitely at a "Loading ..." page. After testing locally, I found I was able to reproduce that on my local machine via the following steps:
yarn build:prod; yarn serve:ssr
). Might also be reproducible in dev mode.After debugging the issue, I found the line producing this error is here:
https://github.com/DSpace/dspace-angular/blob/main/src/app/submission/objects/submission-objects.effects.ts#L108
The error appears to be that the Submission form will somehow "lose access" to the
item
object (similar to described in #2940 and fixed by #2962)This PR fixes the issue by updating the newly added code in #2988 to also embed the
item
object.Instructions for Reviewers
submission.cy.ts
e2e tests are a side effect of this bug.