-
Notifications
You must be signed in to change notification settings - Fork 51
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 index.md #1053
base: main
Are you sure you want to change the base?
Update index.md #1053
Conversation
Hi team - I would like to change the "Caution" section from "Caution" to "Info" or "Note"
✅ Deploy Preview for snowplow-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I agree with removing @johnmicahreid |
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 - @danielyyy4 note that the convention is the reviewer approves the change, and the contributor is responsible for actually merging the change / closing the PR.
I understand, @johnmicahreid. I currently do not have permissions to merge, but when Nick comes back I will ask him to add me to the Snowplow group so that I can |
Hi team - I would like to change the "Caution" section from "Caution" to "Info" or "Note" OR ca we remove the note that it is in public preview or is this necessary? @johnmicahreid