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

Issue assignment clarity #3511

Merged
merged 6 commits into from
May 22, 2023
Merged

Issue assignment clarity #3511

merged 6 commits into from
May 22, 2023

Conversation

joemull
Copy link
Member

@joemull joemull commented May 2, 2023

The issue described in #2750 has likely been fixed, but in investigating it, I found a bug in the View Metadata modal that meant users could not change Projected Issue after it was set, because they were sent to the Issue Manager instead of the Projected Issue assignment form.

I changed the link to the correct form, but I did not want to get rid of the link between View Metadata and the Issue Manager, because even if it was the "wrong" link, some people might be using it to manage primary issue assignment before pre-publication. So I added a table for Primary Issue to View Metadata below Projected Issue, and I clarified the text around these forms so that new editors could more easily get up to speed on the difference between Projected and Primary and manage both flexibly.

image

Projected Issue Assignment

Closes #2750

@joemull joemull requested review from ajrbyers and mauromsl May 2, 2023 12:50
Copy link
Member

@ajrbyers ajrbyers left a comment

Choose a reason for hiding this comment

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

I've added some comments but this looks good!

@joemull joemull requested a review from ajrbyers May 19, 2023 12:59
Copy link
Member

@mauromsl mauromsl left a comment

Choose a reason for hiding this comment

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

One more suggestion to make sure we cover all false positives

@joemull joemull requested a review from mauromsl May 22, 2023 09:36
Copy link
Member

@mauromsl mauromsl left a comment

Choose a reason for hiding this comment

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

Thanks @joemull, it all looks much clearer now

@mauromsl mauromsl merged commit 450cb90 into master May 22, 2023
@mauromsl mauromsl deleted the 2750-issue-assignment-clarity branch May 22, 2023 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Article and Issue can disagree about being friends
3 participants