Feature request: support `Date` objects as traits
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 38/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- typescript
調査の方向性
まず、types.d.ts で参照されている Flagsmith SDK の trait 型定義と、リクエスト用に traits をシリアライズするコードを見つけます。現在 Date 値がどのように型付けされ、変換されているかを確認し、その後、ネイティブな Date traits とその文字列シリアライズを対象とするテストを追加または更新します。OpenFeature provider が Date オブジェクトをコピーしてマッピングしなくても、Date オブジェクトを traits として受け入れられれば完了です。
索引モデルが issue の本文から書いたものです。
説明
OpenFeature supports Datetime types for context attributes, but Flagsmith traits do not.
Because of this, the OpenFeature provider has to make a copy of all the traits and map any Date objects into strings, which we've added in https://github.com/open-feature/js-sdk-contrib/pull/1351. The OpenFeature provider could potentially have some smarter logic/caching to avoid this copying, but this is overkill for what should be a simple adapter on top of the Flagsmith SDK.
Ideally, the Flagsmith SDK should natively support Date objects, and only serialise them into strings when needed. This avoids having to make a copy of the entire evaluation context whenever it changes, and makes the OpenFeature provider implementation simpler and faster.
- 主要言語
- TypeScript
- スター
- 67
- フォーク
- 47
- 平均マージ
- 2時間 26分
- マージ済み PR（30日）
- 2
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Flagsmith/flagsmith-js-client のほかの issue
-
Dependency Dashboardオープン
難易度 1/5 1時間未満 初心者へのやさしさ 20/100
Flagsmith/flagsmith-js-client#435 ·
メンテナーはふだん 1 日以内に返信
-
Move module-scope Emitter in react.tsx into the Provider対応中かも @EngineerOnTravel が 73 日前に担当しました。 オープンbug good first issue
難易度 3/5 1〜2日 初心者へのやさしさ 58/100
Flagsmith/flagsmith-js-client#391 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
enhancement good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 55/100
Flagsmith/flagsmith-js-client#371 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
Flagsmith/flagsmith-js-client#330 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
Flagsmith/flagsmith-js-client#324 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
Flagsmith/flagsmith-js-client の issue をすべて見る
似ている issue
-
area:desktop area:ui bug platform:macos
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
anthropics/claude-code#106183 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
feature:App-Tour feature:Chat ready Web
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
digitalfabrik/integreat-app#4523 · リアクション 2 件 ·
メンテナーはふだん 3 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
Required Lexical field accepts two empty paragraphs対応中かも @lazerg が今日担当しました。 オープンBug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
payloadcms/payload#19222 ·
メンテナーはふだん 1 日以内に返信