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

Support CC licenses #12

Merged
merged 14 commits into from
Jun 9, 2022
Merged

Support CC licenses #12

merged 14 commits into from
Jun 9, 2022

Conversation

rlskoeser
Copy link
Contributor

No description provided.

- similar to handling for rightsstatement.org

Also cache property methods on the model
djiffy/models.py Fixed Show fixed Hide fixed
djiffy/models.py Fixed Show fixed Hide fixed
djiffy/models.py Fixed Show fixed Hide fixed
djiffy/models.py Fixed Show fixed Hide fixed
djiffy/models.py Fixed Show fixed Hide fixed
djiffy/models.py Fixed Show fixed Hide fixed
@codecov-commenter
Copy link

codecov-commenter commented Jun 9, 2022

Codecov Report

Merging #12 (fabfc8b) into develop (5514192) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop      #12      +/-   ##
===========================================
+ Coverage    99.15%   99.20%   +0.04%     
===========================================
  Files           13       13              
  Lines          948     1007      +59     
===========================================
+ Hits           940      999      +59     
  Misses           8        8              

@rlskoeser rlskoeser requested a review from blms June 9, 2022 18:27
Copy link
Contributor

@blms blms left a comment

Choose a reason for hiding this comment

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

The code all looks good to me. (I forgot to mention in the previous Geniza PR, but cached properties are new to me!)

I made the necessary changes to the SVGs and added notes in the readme. Not sure that's the best place, so feel free to move them elsewhere. It might also be worthwhile to include links to the following articles on CSS-Tricks:

https://css-tricks.com/svg-use-with-external-reference-take-2/
https://css-tricks.com/lodge/svg/21-get-two-colors-use/

@rlskoeser
Copy link
Contributor Author

@blms thanks — your updates look good to me. Readme for the SVGs seems good enough for now, can always move if we think of a better place. I'm ok without adding the links to SVG styling, that seems findable enough for anyone who wants to use it.

@rlskoeser rlskoeser merged commit 73f7953 into develop Jun 9, 2022
@rlskoeser rlskoeser deleted the feature/support-cc-licenses branch June 9, 2022 18:34
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