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

Keep unread ボタンをSharedPreferencesではなく、ファイルに設定状態を保存するように #184

Merged
merged 2 commits into from
Oct 8, 2024

Conversation

areteruhiro
Copy link
Collaborator

@areteruhiro areteruhiro commented Oct 8, 2024

LSPatchに対応

確認項目

  • 動作確認済み
  • 誤字脱字無し

Fix #142

@areteruhiro areteruhiro changed the title SharedPreferencesではなくファイルに設定状態を保存するように SharedPreferencesではなくファイルに設定状態を保存するようにすることで未読のままを非rootにも表示させるように変更 Oct 8, 2024
@areteruhiro areteruhiro changed the title SharedPreferencesではなくファイルに設定状態を保存するようにすることで未読のままを非rootにも表示させるように変更 未読のままを非rootにも表示させるように変更 Oct 8, 2024
@s1204IT s1204IT enabled auto-merge (squash) October 8, 2024 19:41
@s1204IT s1204IT merged commit 6771f66 into Chipppppppppp:master Oct 8, 2024
1 check passed
@areteruhiro
Copy link
Collaborator Author

areteruhiro commented Oct 9, 2024

app-release.zip
今のマージされているmasterをダウンロードしてリリースしただけです

自分の環境だと動作できているのですが、

SharedPreferencesが原因なのか推測の域なので

LsPatch環境で動作しているか確認してもらうことはできますか

@areteruhiro areteruhiro changed the title 未読のままを非rootにも表示させるように変更 Keep unread ボタンのSharedPreferencesではなく、ファイルに設定状態を保存するように Oct 9, 2024
@areteruhiro areteruhiro changed the title Keep unread ボタンのSharedPreferencesではなく、ファイルに設定状態を保存するように Keep unread ボタンをSharedPreferencesではなく、ファイルに設定状態を保存するように Oct 9, 2024
@areteruhiro
Copy link
Collaborator Author

なんか、こちらのPRだと、治らなかったっていう人がこっちで、現れて
結局結構前のコードそのままにしたら、表示されましたって人がいたので!
その時のコードに修正してもいいかもしれないです。
(保存機能がない時のコード)

@s1204IT
Copy link
Collaborator

s1204IT commented Oct 26, 2024

Revert すればいいですかね

@areteruhiro
Copy link
Collaborator Author

新しいPRだしますね、このプレリリースは今までのリリースよりかは、ハードル低めでしてくださる感じですか?

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.

「未読のまま閲覧」スイッチがない
2 participants