search

arXiv / arxiv-submission-ui

User interface of NG submit system.
MIT License
2 stars 6 forks source link

Fix progress bar bugs introduced by workflow refactor #169

Closed bdc34 closed 4 years ago

bdc34 commented 4 years ago

This uses the workflow to check if the stages are before the current stage or after it.

This merges to ARXIVNG-2886 due to branching from there.

SBBCornell commented 4 years ago

Also it looks like the confirmation step is gone from the nav/steps... which I think is great! Reducing steps is fantastic but wanted to double check.

SBBCornell commented 4 years ago

Im hitting a 500 error when I get to the optional metadata page. It's working great up to that point.

I am also hitting this error when running develop, so it doesnt appear to be related to this PR.

bdc34 commented 4 years ago

Also it looks like the confirmation step is gone from the nav/steps... which I think is great! Reducing steps is fantastic but wanted to double check.

That was not intentional...but great!

bdc34 commented 4 years ago

Im hitting a 500 error when I get to the optional metadata page. It's working great up to that point.

I am also hitting this error when running develop, so it doesnt appear to be related to this PR.

I'll take a look at that.

SBBCornell commented 4 years ago

Also it looks like the confirmation step is gone from the nav/steps... which I think is great! Reducing steps is fantastic but wanted to double check.

That was not intentional...but great!

We still need users to hit that final screen because there is important messaging, but I completely agree it doesn't need to be part of the pagination.

bdc34 commented 4 years ago

Im hitting a 500 error when I get to the optional metadata page. It's working great up to that point.

I am also hitting this error when running develop, so it doesnt appear to be related to this PR.

I've fixed this 500 error on optional metadata.

It also looks like I chopped off the confirmation step.

mhl10 commented 4 years ago

@bdc34 did you intend to set the base branch for this PR to ARXIVNG-2886?

bdc34 commented 4 years ago

@bdc34 did you intend to set the base branch for this PR to ARXIVNG-2886?

I had it set to ARXIVNG-2886 but then that got merged to develop so I change the base of this PR.

mhl10 commented 4 years ago

@bdc34 did you intend to set the base branch for this PR to ARXIVNG-2886?

I had it set to ARXIVNG-2886 but then that got merged to develop so I change the base of this PR.

OK. Changeset looked bigger than I expected.

bdc34 commented 4 years ago

@bdc34 did you intend to set the base branch for this PR to ARXIVNG-2886?

I had it set to ARXIVNG-2886 but then that got merged to develop so I change the base of this PR.

OK. Changeset looked bigger than I expected.

I know what you mean. When I did something like this earlier the change set went back to just what were new commits. I'll need to be careful with this merge.

bdc34 commented 4 years ago

Also it looks like the confirmation step is gone from the nav/steps... which I think is great! Reducing steps is fantastic but wanted to double check.

That was not intentional...but great!

We still need users to hit that final screen because there is important messaging, but I completely agree it doesn't need to be part of the pagination.

I added back the confirmation step but it is excluded from the pagination.

SBBCornell commented 4 years ago

I just tested and it looks great!