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

fix: Update demos and documentation #57

Merged
merged 21 commits into from
Jul 16, 2024

Conversation

xeniape
Copy link
Member

@xeniape xeniape commented Jun 25, 2024

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.

@xeniape xeniape changed the title Fix/update nifi-kafka-druid demos and documentation fix: update nifi-kafka-druid demos and documentation Jun 25, 2024
@xeniape xeniape requested a review from a team June 25, 2024 14:52
@xeniape xeniape self-assigned this Jun 25, 2024
@sbernauer sbernauer self-requested a review June 26, 2024 14:08
@xeniape xeniape changed the title fix: update nifi-kafka-druid demos and documentation fix: update demos and documentation Jun 27, 2024
@sbernauer sbernauer changed the title fix: update demos and documentation fix: Update HBase demo and documentation Jun 27, 2024
sbernauer
sbernauer previously approved these changes Jun 27, 2024
Copy link
Member

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

@sbernauer sbernauer changed the title fix: Update HBase demo and documentation fix: Update demos and documentation Jun 27, 2024
Copy link
Contributor

@fhennig fhennig left a 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.

fhennig
fhennig previously approved these changes Jul 4, 2024
Copy link
Contributor

@fhennig fhennig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

🚀

@xeniape xeniape requested a review from fhennig July 9, 2024 07:59
Copy link
Member

@NickLarsenNZ NickLarsenNZ left a 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.

@xeniape
Copy link
Member Author

xeniape commented Jul 12, 2024

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.

Copy link
Member

@maltesander maltesander left a 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...

docs/modules/demos/pages/trino-iceberg.adoc Outdated Show resolved Hide resolved
docs/modules/demos/pages/trino-taxi-data.adoc Outdated Show resolved Hide resolved
@NickLarsenNZ NickLarsenNZ self-requested a review July 15, 2024 08:56
@NickLarsenNZ
Copy link
Member

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

maltesander
maltesander previously approved these changes Jul 16, 2024
Copy link
Member

@maltesander maltesander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thank you!

Copy link
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@xeniape xeniape added this pull request to the merge queue Jul 16, 2024
Merged via the queue into main with commit e513db2 Jul 16, 2024
1 check passed
@xeniape xeniape deleted the fix/update-nifi-kafka-druid-documentation branch July 16, 2024 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

6 participants