This repository was archived by the owner on Nov 30, 2024. It is now read-only.
append message to enable ondemand endpoint#32
Merged
Conversation
Contributor
Author
|
@akiomik file の差分はどれも visual studio が生成するやつです |
shoe116
force-pushed
the
mod_ondemand_ribbon
branch
from
ca98ade to
cb9abab
Compare
shoe116
commented
Jul 26, 2021
|
|
||
| } | ||
|
|
||
| private void textAPIKey_TextChanged(object sender, EventArgs e) |
Contributor
Author
There was a problem hiding this comment.
消してもGUIからいじると復活してしまうっぽいので今回はこのままいきます
akiomik
reviewed
Jul 28, 2021
| <value>バフェット・コードの従量課金エンドポイントを利用できるようにします。 | ||
|
|
||
|
|
||
| 従量課金APIを使うことで、現在5年以内の財務数値しか取得できなかったところ、5年より前の財務数値もBCODE関数で取得することができるようになります。 |
Contributor
There was a problem hiding this comment.
現在5年以内の財務数値しか取得できなかったところ
ちょっとだけ日本語が気になりました。「現在〜できなかった」という表現は今までできなかったことが急にできるようになったようなニュアンスが強いけど、利用者からみて時間軸はいつであろうといいので、機能比較を説明した方が伝わりやすいと思います。
例えば:
「月額料金の範囲では5年以内の財務数値しか取得できないところを」
みたいなイメージです。
Contributor
Author
There was a problem hiding this comment.
スプシのやつコピペなんだけど、これアレだな リリースノートっぽい文言なんだな
直しますわ
akiomik
approved these changes
Jul 28, 2021
Contributor
akiomik
left a comment
akiomik
left a comment
There was a problem hiding this comment.
コードの差分はOKです。文言はちょっとだけ気になったので検討してもらえると！
Contributor
Author
|
@akiomik 文言修正と、注記へのlinkを追加しました |
akiomik
reviewed
Jul 28, 2021
shoe116
force-pushed
the
mod_ondemand_ribbon
branch
from
8c0d61e to
a1afa4e
Compare
akiomik
reviewed
Jul 28, 2021
|
|
||
|
|
||
| 従量課金エンドポイントを利用すると、定額料金の範囲（直近5年）より過去の財務数値を取得することができるようになります。 | ||
| 財務数値の取得に従量課金のエンドポイントを利用する必要の有無は、エクセルアドイン内部で都度判定します。 |
Contributor
There was a problem hiding this comment.
細かいけど、「〜の有無は」となる場合は「必要性の有無は」としないとちょっと日本語としては不自然かなと思う。
一応別の言い換え案も載せておきます。
- 「従量課金エンドポイントの利用の要否は」
- 「従量課金エンドポイントを利用する必要があるかないかは」
- 「従量課金エンドポイントを利用する必要があるかどうかの判定は、エクセルアドイン内部で都度行います」
(文字の接続が多いので、従量課金のエンドポイントの「の」はない方が読みやすい気がする)
akiomik
approved these changes
Jul 28, 2021
shoe116
force-pushed
the
mod_ondemand_ribbon
branch
from
a1afa4e to
af87577
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
従量課金エンドポイントのボタンのしたに注記とlinkを追加
to solve: #24