-
Notifications
You must be signed in to change notification settings - Fork 573
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
Conversation
Now that we are restarting our community meetings, clarify a bit about the process to submit the talks
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 overall, left small comment
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 |
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.
It should be ok as long as it is open sourced, otherwise vendor private should not be encouraged or allowed
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.
That has been added as a point in the "Unsuitable Content" section
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.
I suggested a small change to make it clear
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. |
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, i think it is ready to be merged after this small change.
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 |
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.
I suggested a small change to make it clear
Agreed, we can iterate on this. |
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.
all comments have been resolved, LGTM
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.