Skip to content
This repository was archived by the owner on Nov 28, 2022. It is now read-only.

🐛 Fix missing Unsplash icons in uploaders when using config override#829

Merged
aileen merged 1 commit into
TryGhost:masterfrom
kevinansfield:fix-image-uploader-unsplash-logic
Aug 17, 2017
Merged

🐛 Fix missing Unsplash icons in uploaders when using config override#829
aileen merged 1 commit into
TryGhost:masterfrom
kevinansfield:fix-image-uploader-unsplash-logic

Conversation

@kevinansfield
Copy link
Copy Markdown
Member

no issue

  • the config injection in gh-image-uploader was missing so the Unsplash button was only being displayed for the length of the session after visiting the Unsplash app screen (creates in-memory settings record based on config) or more permanently by saving the Unsplash settings (in which case the image uploader component reads the settings from the DB rather than config)

no issue
- the `config` injection in `gh-image-uploader` was missing so the Unsplash button was only being displayed for the length of the session after visiting the Unsplash app screen (creates in-memory settings record based on config) or more permanently by saving the Unsplash settings (in which case the image uploader component reads the settings from the DB rather than config)
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 72.995% when pulling 89e22fd on kevinansfield:fix-image-uploader-unsplash-logic into 4d215e2 on TryGhost:master.

@aileen aileen merged commit 67ee5c1 into TryGhost:master Aug 17, 2017
@kevinansfield kevinansfield deleted the fix-image-uploader-unsplash-logic branch June 4, 2018 10:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants