Skip to content

Display all contributions in 'My contributions' #5575

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

Merged
merged 2 commits into from
Nov 24, 2022

Conversation

tomasr8
Copy link
Member

@tomasr8 tomasr8 commented Nov 21, 2022

Instead of showing only contributions for which the user is a submitter,
we show all contributions categorized by primary/secondary author, speaker or submitter.

Contributions in the Submitter section are only shown if the user neither an author or a speaker.

Not-so-obvious edge case: a user can have submission rights on a contribution w/o having a person link (= not added in the People field in contribution settings) - these contributions are included.

image

@tomasr8 tomasr8 force-pushed the my-contributions branch 3 times, most recently from 6ce0b39 to c783a94 Compare November 21, 2022 15:06
@ThiefMaster
Copy link
Member

What do you think about hiding contributions in the "submitter" list if it's already listed anywhere else?

@tomasr8 tomasr8 force-pushed the my-contributions branch 4 times, most recently from 46669fc to 1c913bd Compare November 23, 2022 15:48
Instead of showing only contributions for which the user is
a submitter, we show all contributions categorized by
primary/secondary author & speaker.
@ThiefMaster ThiefMaster added this to the v3.2 milestone Nov 24, 2022
@ThiefMaster ThiefMaster enabled auto-merge (squash) November 24, 2022 16:40
@ThiefMaster ThiefMaster merged commit 27a1a9f into indico:master Nov 24, 2022
@ThiefMaster ThiefMaster deleted the my-contributions branch November 24, 2022 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done 🚀
Development

Successfully merging this pull request may close these issues.

2 participants