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

Fix #505 - a code snippet error in a Japanese document page #507

Merged
merged 1 commit into from
Oct 30, 2021

Conversation

TORIFUKUKaiou
Copy link
Contributor

@TORIFUKUKaiou TORIFUKUKaiou commented Oct 30, 2021

The document of getting_started.md has not already been implemented action_button_click at this time.

Fixes #505

Category (place an x in each of the [ ])

  • slack_bolt.App and/or its core components
  • slack_bolt.async_app.AsyncApp and/or its core components
  • Adapters in slack_bolt.adapter
  • Document pages under /docs
  • Others

Requirements (place an x in each [ ])

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

  • I've read and understood the Contributing Guidelines and have done my best effort to follow them.
  • I've read and agree to the Code of Conduct.
  • I've run ./scripts/install_all_and_run_tests.sh after making the changes.
    Success: no errors found 🎉

The document of getting_started.md has not already been implemented action_button_click at this time.
@CLAassistant
Copy link

CLAassistant commented Oct 30, 2021

CLA assistant check
All committers have signed the CLA.

@seratch seratch changed the title delete action_button_click(body, ack, say) Fix #505 - a code snippet error in a Japanese document page Oct 30, 2021
@seratch seratch added bug Something isn't working docs Improvements or additions to documentation labels Oct 30, 2021
@seratch seratch added this to the 1.9.5 milestone Oct 30, 2021
@seratch seratch self-assigned this Oct 30, 2021
@seratch
Copy link
Member

seratch commented Oct 30, 2021

Hi @TORIFUKUKaiou, thanks for submitting this PR!

As this is your first contribution to @slackapi org's projects, can you check our CLA and sign it if it's fine for you? Without it, we are unable to merge your changes. Please refer to the above comment by the CLA bot.

Also, for future pull requests, can you keep the following points in mind?

  • Have the issue number in the PR title
  • Have "Fixes #505 (the issue number you resolve)" in the description as I edited in this PR - with this, GitHub automatically associates your PR and the issue for us

@codecov
Copy link

codecov bot commented Oct 30, 2021

Codecov Report

Merging #507 (ec15653) into main (e112c7e) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #507   +/-   ##
=======================================
  Coverage   91.43%   91.43%           
=======================================
  Files         167      167           
  Lines        5535     5535           
=======================================
  Hits         5061     5061           
  Misses        474      474           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e112c7e...ec15653. Read the comment docs.

@TORIFUKUKaiou
Copy link
Contributor Author

I sign your CLA. Thanks.

@seratch seratch merged commit 8a0c87e into slackapi:main Oct 30, 2021
@TORIFUKUKaiou TORIFUKUKaiou deleted the fix-issue-505 branch October 30, 2021 01:43
@seratch seratch removed the bug Something isn't working label Nov 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

delete action_button_click(body, ack, say) in Japanese on docs/_tutorials/ja_getting_started.md .
3 participants