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

Adds JP translation for processBeforeResponse & AWS guide #1524

Merged
merged 5 commits into from
Jul 20, 2022

Conversation

wongjas
Copy link
Member

@wongjas wongjas commented Jul 20, 2022

Summary

Fixes #1230 and fixes #1246, the actual files changed can be found in:

#1229: https://github.com/slackapi/bolt-js/pull/1229/files
#1245: https://github.com/slackapi/bolt-js/pull/1245/files

Requirements (place an x in each [ ])

@wongjas wongjas self-assigned this Jul 20, 2022
@wongjas wongjas requested a review from seratch July 20, 2022 04:21
@wongjas wongjas force-pushed the jwong-fix-1230-1246 branch from 326a8e9 to ee63b59 Compare July 20, 2022 04:22
@wongjas wongjas marked this pull request as ready for review July 20, 2022 04:22
@codecov
Copy link

codecov bot commented Jul 20, 2022

Codecov Report

Merging #1524 (c48f550) into main (86486e0) will not change coverage.
The diff coverage is n/a.

❗ Current head c48f550 differs from pull request most recent head 74ed79e. Consider uploading reports for the commit 74ed79e to get more accurate results

@@           Coverage Diff           @@
##             main    #1524   +/-   ##
=======================================
  Coverage   82.07%   82.07%           
=======================================
  Files          18       18           
  Lines        1495     1495           
  Branches      435      435           
=======================================
  Hits         1227     1227           
  Misses        172      172           
  Partials       96       96           

```

> 💡 [Serverless CLI をグローバルにインストールする](https://www.serverless.com/framework/docs/getting-started/)こともできます。その場合は`npm install -g serverless` というコマンドを実行します。
Serverless でも [macOS、Windows、Linux](https://www.serverless.com/framework/docs/getting-started/) にインストールして利用できるコマンドラインインターフェイス(CLI)のツールが用意されています。インストールするには Serverless の[入門ガイド(英語)](https://www.serverless.com/framework/docs/getting-started/) をお読みください。
Copy link
Member

Choose a reason for hiding this comment

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

You removed the first link to the getting started page. Can you make this part consistent?

docs/_deployments/ja_aws-lambda.md Outdated Show resolved Hide resolved
docs/_deployments/ja_aws-lambda.md Outdated Show resolved Hide resolved
docs/_deployments/ja_aws-lambda.md Outdated Show resolved Hide resolved
docs/_deployments/ja_aws-lambda.md Outdated Show resolved Hide resolved
docs/_deployments/ja_aws-lambda.md Outdated Show resolved Hide resolved
docs/_tutorials/ja_reference.md Outdated Show resolved Hide resolved
docs/_tutorials/ja_reference.md Outdated Show resolved Hide resolved
@seratch seratch added the docs M-T: Documentation work only label Jul 20, 2022
@seratch seratch added this to the 3.12.1 milestone Jul 20, 2022
@wongjas wongjas requested a review from seratch July 20, 2022 05:39
docs/_tutorials/ja_reference.md Outdated Show resolved Hide resolved
@seratch seratch merged commit f203826 into slackapi:main Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs M-T: Documentation work only
Projects
None yet
2 participants