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 color as hex works #561

Merged
merged 1 commit into from
Aug 7, 2023
Merged

Add color as hex works #561

merged 1 commit into from
Aug 7, 2023

Conversation

jcastroa87
Copy link
Member

This PR fix the issue about Yellow color in our demo Chart

image

Issue: https://github.com/Laravel-Backpack/PRO/pull/195

@pxpm
Copy link
Contributor

pxpm commented Aug 7, 2023

Thanks @jorgetwgroup can confirm it works with hex color.

They are the same color, but hey 🤷 nice catch!
image

@pxpm pxpm merged commit 4597bd2 into main Aug 7, 2023
1 of 2 checks passed
@welcome
Copy link

welcome bot commented Aug 7, 2023

WHOOP-WHOOP! Congrats, your first PR on this repo has officialy been merged.

party

If you want to help out the community in other ways, you can:

  • give your opinion on other Github Issues & PRs;
  • chat with others in the Gitter Chatroom (usually for quick help: How do I do X);
  • answer Backpack questions on Stackoverflow; you get points, people get help; you can subscribe to the backpack-for-laravel tag by adding a new filter; that will send you emails when new questions come up with our tag;

Again. Thank you for the PR. You are a wonderful person. Keep 'em coming :-)
Cheers!

--
Justin Case
The Backpack Robot

P.S. Help in the Backpack community is rewarded with free Backpack commercial licenses. It's the least we can do. If you feel you've helped the community with PRs, help & other stuff, please shoot Tabacitu an email and ask him if you qualify for free licenses. You scratch my back, I scratch your back. Thank you!

@pxpm pxpm deleted the fix-yellow-chart branch August 7, 2023 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants