Skip to content

Conversation

stleerh
Copy link
Contributor

@stleerh stleerh commented Feb 26, 2025

No description provided.

Copy link

github-actions bot commented Feb 26, 2025

🙈 The PR is closed and the preview is expired.

@msherif1234
Copy link
Contributor

I think IMHO it will be good idea to refer to specific feature blog for more details context so we have one place to maintain/update if needed instead of replicating info

@stleerh
Copy link
Contributor Author

stleerh commented Feb 26, 2025

This is more than a summary but not in terms of going as deep as a specific blog. I will add links to the feature blogs.

@jotak
Copy link
Member

jotak commented Feb 27, 2025

Nice blog! 😎

@stleerh
Copy link
Contributor Author

stleerh commented Feb 28, 2025

Nice blog! 😎

Thanks Joel for all the great feedback! See this link for the diffs between my first and second commit.

Copy link
Contributor

@skrthomas skrthomas left a comment

Choose a reason for hiding this comment

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

Really nice blog, Steven. /lgtm

Copy link
Member

@memodi memodi left a comment

Choose a reason for hiding this comment

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

great blog @stleerh , easy to understand. thanks!

@mffiedler
Copy link

/LGTM

Copy link
Member

@jotak jotak left a comment

Choose a reason for hiding this comment

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

Thanks @stleerh , lgtm!

@jotak jotak merged commit 386cc90 into netobserv:main Mar 11, 2025
1 check passed
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.

7 participants