Skip to content

Conversation

daveoconnor
Copy link
Collaborator

  • Update avatars to add use_user_hq_image flag
  • Use high quality Images in fiscal committee list

Copy link
Member

@rbbeeston rbbeeston left a comment

Choose a reason for hiding this comment

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

LGTM!

@karimarie67
Copy link
Collaborator

@daveoconnor can you add some test steps please? Thank you!

@daveoconnor
Copy link
Collaborator Author

daveoconnor commented Aug 26, 2025

@karimarie67

Set up a ReportConfiguration for a version, choosing a few users for the fiscal committee and setting some blurb into the text. (We have 3 members now)

Run a release report (message me on slack if you need info on this) and you'll see that the generated page's shown fiscal committee has either no image or some default images.

Go into the admin edit page of one or more of the users that you set for the fiscal committee and set a jpg for "High Quality Image" for that user.

Regenerate the release report and confirm that the newly set image is used.

Copy link
Collaborator

@GregKaleka GregKaleka left a comment

Choose a reason for hiding this comment

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

Approved with an optional small suggestion.

@daveoconnor daveoconnor force-pushed the doc/1869-fiscal-committee-hq-images branch from d04fd4e to 5964b2e Compare September 2, 2025 17:36
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.

4 participants