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

OC-817: Display red flags on public profile page #751

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

Conversation

finlay-jisc
Copy link
Collaborator

The purpose of this PR was to enable users to see how other users are using the red flagging system by showing a list of any red flags users have raised on their profile.


Acceptance Criteria:

  • On a user’s public profile, below the “Octopus publications” area, a heading is present, “Red flags raised by this user”
    • Below the heading, a checkbox is present to “Include resolved flags”.
      • Resolved red flags are only displayed if this option is checked.
      • This option is unchecked by default
    • Below the checkbox, a list is present listing publication tiles for each publication they have red flagged (1 per red flag, so duplicates can appear if the same user red flags a publication twice). with the following differences:
      • Above the publication type, text is present reading “Raised on for against the following publication:
        • If the red flag is resolved, this text is struck through, and appended with the (not struck through) text “(Resolved)”.
      • This list is paginated
      • Clicking on the tile for a red flag links the user directly to the relevant red flag page

Checklist:

  • Local manual testing conducted
  • Automated tests added
  • Documentation updated

Tests:

API
Screenshot 2025-01-14 151836

UI
Screenshot 2025-01-14 150945

E2E
Screenshot 2025-01-14 150917


Screenshots:

Screenshot 2025-01-14 152354

Screenshot 2025-01-14 152403

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.

1 participant