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

Update README.md about community meetings #1282

Merged
merged 4 commits into from
Dec 20, 2023
Merged

Conversation

kfaseela
Copy link
Member

Now that we are restarting our community meetings, clarify a bit about the process to submit the talks. People are directly reaching out to different members currently, asking how to be involved in community meetings.

Now that we are restarting our community meetings, clarify a bit about the process to submit the talks
@kfaseela kfaseela requested a review from a team as a code owner December 19, 2023 09:17
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 19, 2023
README.md Outdated Show resolved Hide resolved
Copy link
Member

@linsun linsun left a comment

Choose a reason for hiding this comment

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

LGTM overall, left small comment

@justinpettit
Copy link
Member

LGTM. I agree with Lin and added another small comment. Thanks!

README.md Outdated
* Istio new feature releases and project updates
* Tutorials and demos
* Use cases
* Content that relates vendor or platform to Istio installation and use
Copy link
Member

Choose a reason for hiding this comment

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

It should be ok as long as it is open sourced, otherwise vendor private should not be encouraged or allowed

Copy link
Member Author

Choose a reason for hiding this comment

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

That has been added as a point in the "Unsuitable Content" section

Copy link
Member

Choose a reason for hiding this comment

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

I suggested a small change to make it clear

@chaltenio
Copy link

I'd suggest creating and publishing a form where folks can submit sessions. This form would help standardize, centralize and organize proposed sessions, in addition to other benefits.

Happy to open a new issue with more details if that makes sense?

@kfaseela
Copy link
Member Author

I'd suggest creating and publishing a form where folks can submit sessions. This form would help standardize, centralize and organize proposed sessions, in addition to other benefits.

Happy to open a new issue with more details if that makes sense?

We can move to forms in future when we have considerably large number of submissions.

Copy link
Member

@linsun linsun left a comment

Choose a reason for hiding this comment

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

LGTM, i think it is ready to be merged after this small change.

README.md Outdated Show resolved Hide resolved
README.md Outdated
* Istio new feature releases and project updates
* Tutorials and demos
* Use cases
* Content that relates vendor or platform to Istio installation and use
Copy link
Member

Choose a reason for hiding this comment

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

I suggested a small change to make it clear

@linsun
Copy link
Member

linsun commented Dec 20, 2023

I'd suggest creating and publishing a form where folks can submit sessions. This form would help standardize, centralize and organize proposed sessions, in addition to other benefits.
Happy to open a new issue with more details if that makes sense?

We can move to forms in future when we have considerably large number of submissions.

Agreed, we can iterate on this.

README.md Outdated Show resolved Hide resolved
Copy link
Member

@linsun linsun left a comment

Choose a reason for hiding this comment

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

all comments have been resolved, LGTM

@istio-testing istio-testing merged commit dcd1174 into master Dec 20, 2023
5 checks passed
@istio-testing istio-testing deleted the kfaseela-patch-1 branch December 20, 2023 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants