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

Add dynamic logos based on color scheme in README (close #38) #39

Merged
merged 1 commit into from Oct 19, 2022

Conversation

3t8
Copy link
Contributor

@3t8 3t8 commented Oct 19, 2022

Description

Logo images in README change dynamically, adapting to the user's theme

Issue Ticket Number

Fixes #38

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist

  • I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
  • I have created an issue ticket for this PR
  • I have checked to ensure there aren't other open Pull Requests for the same update/change?
  • I have performed a self-review of my own code
  • I have run tests locally to ensure they all pass
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes needed to the documentation

@netlify
Copy link

netlify bot commented Oct 19, 2022

Deploy Preview for netlify-plugin-cloudinary ready!

Name Link
🔨 Latest commit ada2456
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-cloudinary/deploys/634f5cd07f2c03000832fbb5
😎 Deploy Preview https://deploy-preview-39--netlify-plugin-cloudinary.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@colbyfayock
Copy link
Collaborator

I'm not sure why these are triggering a failure here specifically it may be an env variable issue. i tested locally to make sure. merging in

thanks for the contribution here as well. the hacktoberfest swag is 1 per person, so see my message on the other repository for how to claim

@colbyfayock
Copy link
Collaborator

@all-contributors please add @3t8 for doc

@allcontributors
Copy link
Contributor

@colbyfayock

I've put up a pull request to add @3t8! 🎉

@colbyfayock colbyfayock merged commit 313a359 into cloudinary-community:main Oct 19, 2022
colbyfayock pushed a commit that referenced this pull request Oct 19, 2022
Adds @3t8 as a contributor for doc.

This was requested by colbyfayock [in this
comment](#39 (comment))

[skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
@github-actions
Copy link

🎉 This PR is included in version 1.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

matiasfha pushed a commit to matiasfha/netlify-plugin-cloudinary that referenced this pull request Sep 23, 2023
…ary-community#38) (cloudinary-community#39)

# Description

Logo images in README change dynamically, adapting to the user's theme

## Issue Ticket Number

<!-- Specifiy which issue this fixes by referencing the issue number
(`cloudinary-community#11`) or issue URL. -->
<!-- Example: Fixes
cloudinary-community#1 -->

Fixes cloudinary-community#38 

## Type of change

<!-- Please select all options that are applicable. -->

- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] This change requires a documentation update


# Checklist

<!-- These must all be followed and checked. -->

- [x] I have followed the contributing guidelines of this project as
mentioned in [CONTRIBUTING.md](/CONTRIBUTING.md)
- [x] I have created an
[issue](https://github.com/colbyfayock/netlify-plugin-cloudinary/issues)
ticket for this PR
- [x] I have checked to ensure there aren't other open [Pull
Requests](https://github.com/colbyfayock/netlify-plugin-cloudinary/pulls)
for the same update/change?
- [x] I have performed a self-review of my own code
- [x] I have run tests locally to ensure they all pass
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes needed to the documentation
matiasfha pushed a commit to matiasfha/netlify-plugin-cloudinary that referenced this pull request Sep 23, 2023
Adds @3t8 as a contributor for doc.

This was requested by colbyfayock [in this
comment](cloudinary-community#39 (comment))

[skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Improve Readme images
2 participants