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.

Sign up for GitHub

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

Jump to bottom

docs: update team config doc #729

Merged
merged 1 commit into from
Mar 4, 2024
Merged

docs: update team config doc #729

merged 1 commit into from
Mar 4, 2024

Conversation

Startrekzky
Copy link
Contributor

@Startrekzky Startrekzky commented Mar 4, 2024 •
edited
Loading

⚠️   Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

  • I have npm run build and npm run serve locally before submitting this PR
  • I have read through the Contributing Documentation

Summary

  1. Add the HTTPs hint
    image

  2. Only show the useful commands in the code block
    image

  3. Add the rest/ path to all CURL commands
    image

  4. Add the auth part to all CURL commands
    image

  5. Unify all CURL commands to the same format -X 'PUT'
    image

  6. Add the 'Grafana way' to connect to the database
    image

  7. Update wordings and grammars.

Does this close any open issues?

N/A

@d4x1 d4x1 merged commit 643f954 into apache:main Mar 4, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@d4x1 d4x1 d4x1 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Startrekzky @d4x1 @merico-devlake