Skip to content
This repository was archived by the owner on Apr 14, 2026. It is now read-only.

[ENG-7472] fixed recent preprint rejected versions doesnt have option to resubmit#2528

Merged
brianjgeiger merged 3 commits intoCenterForOpenScience:feature/pbs-25-05from
bodintsov:fix/rejected-recent-version-does-not-have-the-option-to-resubmit
Mar 21, 2025
Merged

[ENG-7472] fixed recent preprint rejected versions doesnt have option to resubmit#2528
brianjgeiger merged 3 commits intoCenterForOpenScience:feature/pbs-25-05from
bodintsov:fix/rejected-recent-version-does-not-have-the-option-to-resubmit

Conversation

@bodintsov
Copy link
Copy Markdown
Contributor

Purpose

Make it possible to resubmit rejected preprint versions

Summary of Changes

Make it possible to resubmit rejected preprint versions

Screenshot(s)

Знімок екрана 2025-03-18 о 12 49 51 Знімок екрана 2025-03-18 о 12 49 42 Знімок екрана 2025-03-18 о 12 49 31

Side Effects

TBD

QA Notes

TBD

Copy link
Copy Markdown
Contributor

@brianjgeiger brianjgeiger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a couple of incorrect messages in the test assertions.

});
await settled();
assert.dom('[data-test-edit-preprint-button]')
.exists('Edit button is not displayed for non-initial pre-mod latest rejected');
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this message is correct. Isn't the edit button displayed here?

await settled();
assert.dom('[data-test-edit-preprint-button]')
.doesNotExist('Edit button is not displayed for non-initial pre-mod rejected');
.exists('Edit button is not displayed for non-initial pre-mod not latest rejected');
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this message is correct. Isn't the edit button displayed here?

@brianjgeiger brianjgeiger merged commit 62a0709 into CenterForOpenScience:feature/pbs-25-05 Mar 21, 2025
9 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants