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 demo link #553

Closed
wants to merge 1 commit into from
Closed

Add demo link #553

wants to merge 1 commit into from

Conversation

MatShir
Copy link

@MatShir MatShir commented Oct 4, 2023

@pawelkierklo is proving us a demo. Big thanks !

@pawelkierklo is proving us a demo. Big thanks !
Copy link
Contributor

@matks matks left a 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

@MatShir
Copy link
Author

MatShir commented Oct 5, 2023

We can share the release version and a comment to report if there is an issue. Let's be simple and not overthink.

@MatShir
Copy link
Author

MatShir commented Oct 5, 2023

If the demo is down, it's not the end of the world. We remove the link when it is reported.

@pawelkierklo
Copy link
Contributor

I will update, I can also prepare a cron.php that will do it for me. I want to develop HB. Let's go!

@MatShir
Copy link
Author

MatShir commented Oct 5, 2023

Hi @pawelkierklo,
As you are a new member of the community, we are not 100% confident to let you handle the demo which we don't have full control of. But we appreciate your involvement in the project, keep going 💪

@MatShir MatShir closed this Oct 5, 2023
@pawelkierklo
Copy link
Contributor

Ready! CRON will update "develop" theme on demo every night

@MatShir
Copy link
Author

MatShir commented Oct 9, 2023

Hi @pawelkierklo,
Sorry for the back and forth, could you share the demo in the discussion on Showcase category. Then, we will link the discussion here, so later on, Users could share report and feedback on the thread directly ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants