-
Notifications
You must be signed in to change notification settings - Fork 440
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
Make the example for Cron Scaler match the screenshot #1330
Conversation
Thank you for your contribution! 🙏 We will review your PR as soon as possible.
Learn more about:
|
✅ Deploy Preview for keda ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like it, great, could you please also fix other versions? Thanks
I'm not sure I understand what you meant, I update the other example the one about having same |
You fixed this in version 2.13, would be great if we can fix this also in other version (2.14, 2.12,...) |
Signed-off-by: Ruben Laguna <ruben.laguna@gmail.com>
Got you now, I added the changes in all other versions now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
The example now match the screenshot
Checklist
Fixes #