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

background listenerをMaybe形式からEither形式にする #10

Open
ncaq opened this issue May 6, 2021 · 1 comment
Open

background listenerをMaybe形式からEither形式にする #10

ncaq opened this issue May 6, 2021 · 1 comment
Labels
Priority: Medium Type: Maintenance Repository Maintenance Type: Refactoring A code change that neither fixes a bug nor adds a feature

Comments

@ncaq
Copy link
Owner

ncaq commented May 6, 2021

何故タイトルが取得できないのか報告してふとした検索をやりやすくする。

Promise<string | undefined>からfp-tsを使用したEither形式に変更します。
また、oldTitleもバックグラウンドに渡して比較メソッドなどもバックグラウンドに渡した方が良いかもしれません。

@ncaq ncaq added Type: Refactoring A code change that neither fixes a bug nor adds a feature Priority: Medium Type: Maintenance Repository Maintenance labels May 6, 2021
@ncaq
Copy link
Owner Author

ncaq commented May 13, 2021

これIndexedDBに保存するデータもEitherにするべきだと思ってましたが、
保存するのは T | undefined のままでも良いかもしれません。
別にエラーの理由は保存されてもそんなに嬉しくないので。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Medium Type: Maintenance Repository Maintenance Type: Refactoring A code change that neither fixes a bug nor adds a feature
Projects
None yet
Development

No branches or pull requests

1 participant