-
Notifications
You must be signed in to change notification settings - Fork 81
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 demo link #553
Add demo link #553
Conversation
@pawelkierklo is proving us a demo. Big thanks !
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.
Hi @MatShir I am sorry I am against this change.
I explain: it's really nice from @pawelkierklo but if we put this link inside the README, then we also set very high expectations.
If you are a github user and you go on any GitHub project, you expect that if there is a demo link in the README it is up to date.
Does @pawelkierklo commit to update his demo every time we merge something in the dev
branch? Does he commit to keep it running 24H/24H, 7 days on 7? Does he commit to troubleshoot it when it has bugs?
If he does it's really cool 😄
If he can't (and it's totally acceptable because it's very high expectations) then we should not put this link in the README. It's better to have NO demo instead of an OUTDATED demo.
Yes :/ demos are expensive and time-consuming to maintain
We can share the release version and a comment to report if there is an issue. Let's be simple and not overthink. |
If the demo is down, it's not the end of the world. We remove the link when it is reported. |
I will update, I can also prepare a cron.php that will do it for me. I want to develop HB. Let's go! |
Hi @pawelkierklo, |
Ready! CRON will update "develop" theme on demo every night |
Hi @pawelkierklo, |
@pawelkierklo is proving us a demo. Big thanks !