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

Add troubleshooting for each rpc #2163

Merged
merged 5 commits into from
Sep 13, 2023

Conversation

vankichi
Copy link
Contributor

@vankichi vankichi commented Aug 31, 2023

Description:

I have added the troubleshooting section at each API docs.

Related Issue:

Versions:

  • Go Version: 1.21.0
  • Docker Version: 20.10.8
  • Kubernetes Version: v1.28.1
  • NGT Version: 2.1.2

Checklist:

Special notes for your reviewer:

@vdaas-ci
Copy link
Collaborator

[CHATOPS:HELP] ChatOps commands.

  • 🙆‍♀️ /approve - approve
  • 🍱 /format - format codes and add licenses
  • /gen-test - generate test codes
  • 🏷️ /label - add labels
  • 🔚 2️⃣ 🔚 /label actions/e2e-deploy - run E2E deploy & integration test

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Aug 31, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 39b874a
Status: ✅  Deploy successful!
Preview URL: https://46018e02.vald.pages.dev
Branch Preview URL: https://documentation-docs-add-error.vald.pages.dev

View logs

@vankichi vankichi force-pushed the documentation/docs/add-error-handling-for-each-API branch from 5ae804d to af4ff32 Compare August 31, 2023 09:24
@github-actions github-actions bot added size/L and removed size/M labels Aug 31, 2023
@vankichi vankichi force-pushed the documentation/docs/add-error-handling-for-each-API branch from af4ff32 to ed96036 Compare August 31, 2023 12:40
@vankichi vankichi requested a review from kpango September 4, 2023 01:38
docs/api/insert.md Outdated Show resolved Hide resolved
docs/api/insert.md Outdated Show resolved Hide resolved
docs/api/insert.md Outdated Show resolved Hide resolved
docs/api/insert.md Outdated Show resolved Hide resolved
docs/api/insert.md Outdated Show resolved Hide resolved
docs/api/insert.md Outdated Show resolved Hide resolved
docs/api/insert.md Outdated Show resolved Hide resolved
docs/api/insert.md Outdated Show resolved Hide resolved
docs/api/object.md Outdated Show resolved Hide resolved
docs/api/object.md Outdated Show resolved Hide resolved
docs/api/update.md Outdated Show resolved Hide resolved
docs/api/update.md Outdated Show resolved Hide resolved
docs/api/upsert.md Outdated Show resolved Hide resolved
docs/api/upsert.md Outdated Show resolved Hide resolved
docs/api/upsert.md Outdated Show resolved Hide resolved
@vankichi vankichi force-pushed the documentation/docs/add-error-handling-for-each-API branch from 7e19052 to a6227e0 Compare September 8, 2023 05:30
@github-actions github-actions bot removed the size/L label Sep 8, 2023
@vankichi vankichi force-pushed the documentation/docs/add-error-handling-for-each-API branch from a6227e0 to 937e9c2 Compare September 8, 2023 06:32
docs/api/insert.md Show resolved Hide resolved
docs/api/insert.md Show resolved Hide resolved
docs/api/insert.md Outdated Show resolved Hide resolved
docs/api/insert.md Show resolved Hide resolved
@vankichi vankichi force-pushed the documentation/docs/add-error-handling-for-each-API branch 3 times, most recently from f4dc8ee to caa0a7a Compare September 8, 2023 06:52
docs/api/object.md Show resolved Hide resolved
docs/api/object.md Show resolved Hide resolved
docs/api/object.md Show resolved Hide resolved
docs/api/object.md Show resolved Hide resolved
docs/api/object.md Show resolved Hide resolved
docs/api/object.md Outdated Show resolved Hide resolved
docs/api/object.md Show resolved Hide resolved
docs/api/object.md Show resolved Hide resolved
docs/api/object.md Show resolved Hide resolved
docs/api/object.md Show resolved Hide resolved
@vankichi vankichi force-pushed the documentation/docs/add-error-handling-for-each-API branch 3 times, most recently from 2d11bdd to 3e9c738 Compare September 8, 2023 08:11
docs/api/remove.md Show resolved Hide resolved
docs/api/remove.md Show resolved Hide resolved
docs/api/remove.md Show resolved Hide resolved
docs/api/remove.md Show resolved Hide resolved
docs/api/remove.md Show resolved Hide resolved
docs/api/remove.md Show resolved Hide resolved
docs/api/remove.md Show resolved Hide resolved
docs/api/remove.md Show resolved Hide resolved
docs/api/remove.md Show resolved Hide resolved
@vankichi vankichi force-pushed the documentation/docs/add-error-handling-for-each-API branch from 3e9c738 to 3f5dc55 Compare September 8, 2023 10:33
@vankichi vankichi force-pushed the documentation/docs/add-error-handling-for-each-API branch from 162bbf3 to 51eb5c4 Compare September 8, 2023 11:35
docs/api/upsert.md Show resolved Hide resolved
docs/api/upsert.md Show resolved Hide resolved
docs/api/upsert.md Show resolved Hide resolved
@vankichi vankichi force-pushed the documentation/docs/add-error-handling-for-each-API branch from 51eb5c4 to 9c429f3 Compare September 8, 2023 11:37
@vankichi vankichi changed the title [WIP] Add troubleshooting for each rpc Add troubleshooting for each rpc Sep 8, 2023
@vankichi vankichi requested review from kpango, a team and hlts2 and removed request for a team September 8, 2023 11:45
vankichi and others added 4 commits September 8, 2023 20:49
Signed-off-by: vankichi <kyukawa315@gmail.com>
Co-authored-by: Yusuke Kato <kpango@vdaas.org>
Signed-off-by: vankichi <kyukawa315@gmail.com>
@vankichi vankichi force-pushed the documentation/docs/add-error-handling-for-each-API branch from 9c429f3 to 2996b4f Compare September 8, 2023 11:49
@vankichi vankichi marked this pull request as ready for review September 8, 2023 11:49
@hlts2 hlts2 merged commit b017c4b into main Sep 13, 2023
10 checks passed
@hlts2 hlts2 deleted the documentation/docs/add-error-handling-for-each-API branch September 13, 2023 09:11
@hlts2 hlts2 mentioned this pull request Sep 13, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
kmrmt pushed a commit that referenced this pull request Dec 12, 2023
* 📝 Add troubleshooting for each rpc

Signed-off-by: vankichi <kyukawa315@gmail.com>

* Apply suggestions from code review

Co-authored-by: Yusuke Kato <kpango@vdaas.org>

* style: Format code with prettier and gofumpt

* 📝 Add troubleshooting for each rpc v2

Signed-off-by: vankichi <kyukawa315@gmail.com>

---------

Signed-off-by: vankichi <kyukawa315@gmail.com>
Co-authored-by: Yusuke Kato <kpango@vdaas.org>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: Hiroto Funakoshi <hiroto.funakoshi.hiroto@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants