Skip to content

入力値チェックプラグインのjQuery削除 - #231

Merged
ashinohito merged 4 commits into
kintone-samples:masterfrom
juridon:checkvalue-remove-jquery
Jun 5, 2026
Merged

入力値チェックプラグインのjQuery削除#231
ashinohito merged 4 commits into
kintone-samples:masterfrom
juridon:checkvalue-remove-jquery

Conversation

@juridon

@juridon juridon commented May 28, 2026

Copy link
Copy Markdown
Contributor

jQuery依存を削除しました。

また、document.addEventListener('DOMContentLoaded', ...) が発火しなかっため、
以下のようにreadyStateを判定して初期化処理を実行するよう修正しました。

if (document.readyState === 'loading') {
  document.addEventListener('DOMContentLoaded', initCheckValue);
} else {
  initCheckValue();
}

@SilverRusk

Copy link
Copy Markdown
Contributor

README.md の OSS ライセンス一覧について

jQuery 剥がしに伴い、examples/checkvalue/README.md の OSS ライセンス一覧の見直しをお願いします。

1. jQuery の記載を削除してください(今回の対応漏れ)

jQuery を剥がしたため不要になっていますが、記載が残っています。

* jQuery - https://jquery.com/
  * Author:  jQuery Foundation, Inc.
  * License: [MIT License](https://github.com/jquery/jquery/blob/1.11.1/MIT-LICENSE.txt)

2. KintoneRestAPIClient の記載を追加してください

これは今回の変更で入ったものではなく元からの記載漏れですが、config.jsnew KintoneRestAPIClient() として使用しており、manifest でも読み込んでいます。せっかく README を触るタイミングなので、一緒に対応していただけると助かります。

* @kintone/rest-api-client - https://github.com/kintone/js-sdk/tree/main/packages/rest-api-client
  * Author: Cybozu, Inc.
  * License: [MIT License](https://github.com/kintone/js-sdk/blob/main/packages/rest-api-client/LICENSE)

(なお kintone-config-helper の記載がありますが、コード上で使用箇所が見当たりません。こちらも元からのものですが、あわせてご確認いただけると幸いです。)

@ashinohito

Copy link
Copy Markdown
Contributor

(横から追記失礼します🙇‍♀️)

@SilverRusk

Copy link
Copy Markdown
Contributor

README の対応ありがとうございます。1点だけ追加でお願いします。

JsRender のバージョン表記の不整合

manifest.json で JsRender を 1.0.61.0.16 に更新いただきましたが、README.md のライセンスリンクが v1.0.6 のままになっています。v1.0.16 に合わせてください。

* JsRender - https://www.jsviews.com/
  * Author: Boris Moore
  * License: [MIT License](https://github.com/BorisMoore/jsrender/blob/v1.0.16/MIT-LICENSE.txt)

@SilverRusk

Copy link
Copy Markdown
Contributor

@ashinohito さんのご指摘分もあわせて対応確認しました。

  • kintone-rest-api-client v5.7.1 → 6.2.0 への更新:対応済み(manifest・README ともに反映、ライセンス表記も指定形式に対応済み)
  • jsrender 1.0.16 への更新:対応済み(manifest・README ともに反映)

私からの README の指摘分もすべて対応済みです。マージOKです。

@ashinohito
ashinohito merged commit a545e13 into kintone-samples:master Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants