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

fix(image): properly lock loremflickr.com images #1396

Merged

Conversation

wesen
Copy link
Contributor

@wesen wesen commented Sep 26, 2022

https://loremflickr.com requires the lock= parameter to be given
to serve an identical image (at least as long as its cache is not
updated).

@wesen wesen requested a review from a team as a code owner September 26, 2022 18:35
@ST-DDT ST-DDT added c: bug Something isn't working p: 1-normal Nothing urgent s: accepted Accepted feature / Confirmed bug m: image Something is referring to the image module labels Sep 26, 2022
@ST-DDT ST-DDT added this to the v7 - Current Major milestone Sep 26, 2022
@ST-DDT
Copy link
Member

ST-DDT commented Sep 26, 2022

Your commit is marked as "not verified". Did you forget to sign them?

@ST-DDT ST-DDT requested review from a team September 26, 2022 18:51
src/modules/image/index.ts Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Sep 26, 2022

Codecov Report

Merging #1396 (d4f1f9d) into main (be87581) will not change coverage.
The diff coverage is 100.00%.

❗ Current head d4f1f9d differs from pull request most recent head 4a6a407. Consider uploading reports for the commit 4a6a407 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1396   +/-   ##
=======================================
  Coverage   99.62%   99.62%           
=======================================
  Files        2163     2163           
  Lines      241306   241306           
  Branches     1017     1017           
=======================================
  Hits       240399   240399           
  Misses        886      886           
  Partials       21       21           
Impacted Files Coverage Δ
src/modules/image/index.ts 100.00% <100.00%> (ø)

@wesen wesen force-pushed the fix/image/properly-lock-loremflickr-images branch from c49ee33 to 9b9f839 Compare September 26, 2022 19:21
src/modules/image/index.ts Outdated Show resolved Hide resolved
https://loremflickr.com requires the lock= parameter to be given
to serve an identical image (at least as long as its cache is not
updated).
@wesen wesen force-pushed the fix/image/properly-lock-loremflickr-images branch from 9b9f839 to b94d9ce Compare September 26, 2022 19:22
@Shinigami92 Shinigami92 enabled auto-merge (squash) September 29, 2022 14:25
@Shinigami92 Shinigami92 merged commit 2539e6a into faker-js:main Sep 29, 2022
wael-fadlallah pushed a commit to wael-fadlallah/faker that referenced this pull request Oct 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: bug Something isn't working m: image Something is referring to the image module p: 1-normal Nothing urgent s: accepted Accepted feature / Confirmed bug
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants