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

document: fix cover image in public detailed view #848

Merged
merged 1 commit into from
Mar 17, 2020

Conversation

AoNoOokami
Copy link
Contributor

  • Corrects url type filter.
  • Displays cover as a link for coverImage type.

Co-Authored-by: Peter Weber [email protected]
Co-Authored-by: Alicia Zangger [email protected]

Why are you opening this PR?

To complete US 1262 https://tree.taiga.io/project/rero21-reroils/us/1262?milestone=257446

How to test?

  1. Edit a record to add an electronic location with a type Covert art (suggested url: https://d2v9ipibika81v.cloudfront.net/uploads/sites/21/Africa-990x684.jpg)
  2. Go to the detailed view in public interface
  3. Check that the cover image is a working link

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Extracted translations?

@AoNoOokami AoNoOokami requested review from rerowep, jma and BadrAly March 13, 2020 13:42
@AoNoOokami AoNoOokami force-pushed the zaa-fix-cover-image branch from 8826b15 to 2a05d2f Compare March 13, 2020 13:53
@AoNoOokami AoNoOokami self-assigned this Mar 13, 2020
* Corrects url type filter.
* Displays cover as a link for coverImage type.

Co-Authored-by: Peter Weber <[email protected]>
Co-Authored-by: Alicia Zangger <[email protected]>
@AoNoOokami AoNoOokami force-pushed the zaa-fix-cover-image branch from 2a05d2f to b6a750a Compare March 13, 2020 15:50
@AoNoOokami AoNoOokami removed the request for review from BadrAly March 17, 2020 12:30
@AoNoOokami AoNoOokami merged commit 971a547 into rero:dev Mar 17, 2020
@AoNoOokami AoNoOokami deleted the zaa-fix-cover-image branch April 20, 2020 08:45
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.

3 participants