arXiv / arxiv-submission-ui

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

ARXIVNG-1451 ARXIVNG-1423 submitter endorsement display on verify user step #56

Closed erickpeirson closed 5 years ago

erickpeirson commented 5 years ago

This leverages https://github.com/arXiv/arxiv-auth/pull/30 (ARXIVNG-1468) to display the submitter's endorsement status more coherently on the verify user page.

Note that https://github.com/arXiv/arxiv-auth/blob/develop/generate_token.py can be used to generate auth tokens with different endorsement authorizations.

No endorsements: image

Some endorsed categories: astro-ph.CO,astro-ph.GA image

Endorsed for entire archives (all categories in an archive): hep-ph.*,nucl-th.*,math-ph.*,physics.*,q-bio.*,nlin.*,hep-th.* image note that the ``.`` after the archive keys is removed in https://github.com/arXiv/arxiv-base/pull/92*

Endorsed for everything (*.*): image

erickpeirson commented 5 years ago

A request: could the Verify User step be dependent on having at least one category of endorsement?

We could do this. But I'd want to get buy-in from Jim and Steinn. Can we raise this as an 0.2 possibility?

A question: is there still a separate issue to resolve what shows up on the classification page?

Nope; will look at that before merging.

eawoods commented 5 years ago

Validation on submission without endorsement: agreed to pass as possibility for 0.2. Interesting to note point in milestone S11, endorsement checks happen at the end of submission workflow (why?).

erickpeirson commented 5 years ago

I don't know. I wonder if that is a typo. More investigation needed.

erickpeirson commented 5 years ago

Here's what it looks like on the select primary page for a user endorsed for astro-ph.CO,astro-ph.GA

image

Endorsed for hep-ph.*,nucl-th.*,math-ph.*,physics.*,q-bio.*,nlin.*,hep-th.*

image

Endorsed for cs.*

image

No endorsements:

image

eawoods commented 5 years ago

Awesome! Have you tried it with favorites/default category set? (If not I can try it, if so and it doesn't change the list we can shove that into 0.2 but might want to suppress the Favorites info box.)

erickpeirson commented 5 years ago

Endorsed for cs.*, default is cs.AI:

image

eawoods commented 5 years ago

Yeah I checked it out with cs.DL, that flows nicely.