-
Notifications
You must be signed in to change notification settings - Fork 3
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
fix: Update demos and documentation #57
Conversation
…d-cycling-data demos
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! FYI, I changed the PR title (it will be the commit message), I hope that's ok
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.
Looks good!
I just put the sentences on individual lines.
docs/modules/demos/pages/jupyterhub-pyspark-hdfs-anomaly-detection-taxi-data.adoc
Outdated
Show resolved
Hide resolved
docs/modules/demos/pages/jupyterhub-pyspark-hdfs-anomaly-detection-taxi-data.adoc
Outdated
Show resolved
Hide resolved
Co-authored-by: Felix Hennig <fhennig@users.noreply.github.com>
…tion-taxi-data.adoc Co-authored-by: Felix Hennig <fhennig@users.noreply.github.com>
…tion-taxi-data.adoc Co-authored-by: Felix Hennig <fhennig@users.noreply.github.com>
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.
thank you!
🚀
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.
Some of the images are smaller than before. Just check what that looks like in the rendered site.
Thanks for the idea! Tried it with local rendering and it looked fine to me. If someone else wants to try it and has some feedback though, I can update the screenshots as well. |
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.
Awesome work thank you :) Just minor things...
I'm ok with the images sizes, though some of the text is on the smaller side. We will add some tips for doing this in the new issue templates for the release process: stackabletech/issues#602 |
Co-authored-by: Malte Sander <malte.sander.it@gmail.com>
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.
LGTM thank you!
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.
LGTM
Some demo configurations (MinIO service name and Druid Middlemanager resources), commands (Kafka probing and SQL queries) and product UIs were outdated / defective. This PR fixes them. Also fixes minor typos and adds consistency between the two nifi-kafka-druid demos.
Extending this PR to other demos and their documentation as well. Similar tasks of updating screenshots, typos etc.