feature: profile and cover photos

[adrianthedev]

Description

Adds a cover and profile photo to the show view of the record.

Docs: https://docs.avohq.io/3.0/cover-and-profile-photos.html

Todo:

• Add cover images to the index/edit page
• add support for plain paths (interpolated with the record data) (Paul's suggestion)

Checklist:

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works

Screenshots & recording

Manual review steps

1. Step 1
2. Step 2

Manual reviewer: please leave a comment with output from the test if that's the case.

[codeclimate]

Code Climate has analyzed commit 8c32a51 and detected 0 issues on this pull request.

View more on Code Climate.

[Paul-Bob]

Looking great!

I have one concern about the way to declare the cover/photo components. IMO it should not rely on existing fields, and pass the photo URL directly on the API like grid view cover.

Pros:

• Any image can be used even an external URL
• Custom path helpers can be used to get the image URL
• No need of field declaration if user don't want to have that fields.

self.cover_photo = -> do
  {
    # size: :sm,
    source: main_app.url_for(record.cover_photo.url)
  }
end

[adrianthedev]

yup! makes sense!
Thanks for the review!

[github-actions]

This PR has been marked as stale because there was no activity for the past 15 days.

[adrianthedev]

Let's look into memoization of these obejcts (profile_photo & cover_photo) from the concern.

Maybe https://github.com/panorama-ed/memo_wise? Let's evaluate it

[adrianthedev]

Implements RSpec tests for Avo's event page cover and profile photos components, ensuring proper visibility under various conditions.

• Adds component tests for both Avo::CoverPhotoComponent and Avo::ProfilePhotoComponent to verify rendering logic based on the presence and visibility of photos. These tests cover scenarios where photos are present, absent, not visible in the current view, and dynamically sourced.
• Updates feature tests in spec/features/avo/cover_profile_photos_spec.rb to include scenarios for an event page without cover and profile photos, with only a cover photo, with only a profile photo, and with both photos. This ensures the application behaves as expected under each condition.

---

For more details, open the Copilot Workspace session.

[github-actions]

This PR has been merged into main. The functionality will be available in the next release.

Please check the release guide for more information.