feature: profile and cover photos

[adrianthedev]

Description

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

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 c161264 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.