Skip to content

Add a more practical example #7

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

Closed
wants to merge 1 commit into from
Closed

Add a more practical example #7

wants to merge 1 commit into from

Conversation

rcemper
Copy link
Contributor

@rcemper rcemper commented Jun 17, 2021

HoleFoods Classes are loaded
Cube is defined but needs to be compiled and built
That's closer to reality

HoleFoods Classes are loaded
Cube is defined but needs to be compiled and built
That's closer to reality
@sduncan01
Copy link
Owner

Hi Robert, we will not include the HoleFoods sample with this application by default. Users need to be able to download and install the Cube Event Monitor in existing production environments without including extraneous data. Users who want this sample should download it from https://github.com/intersystems/Samples-BI/ and populate the source tables. I have opened #8 to consider whether there's a practical way to optionally include sample data as a demo for Cube Event Monitor users installing via ZPM. Thank you.

@sduncan01 sduncan01 closed this Jun 17, 2021
@rcemper
Copy link
Contributor Author

rcemper commented Jun 17, 2021

But in this case, it would be a significant improvement for the example also to
mention such a possibility. Just leaving alone the user with his docker is not at all attractive.
Your reply here will most likely be unseen by 98% of users.
and BTW. my proposal would install it only in a Docker container. If you had checked the changes

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

Successfully merging this pull request may close these issues.

2 participants