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

LSPatch 時の着信音が鳴らない問題を修正 #194

Merged
merged 9 commits into from
Oct 26, 2024

Conversation

areteruhiro
Copy link
Collaborator

@areteruhiro areteruhiro commented Oct 18, 2024

確認項目

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

説明

Fix #67

@areteruhiro
Copy link
Collaborator Author

こちら側でためしてもらってる人の動作確認ができたらレビューリクエスト送信します

@areteruhiro areteruhiro marked this pull request as ready for review October 20, 2024 01:04
@areteruhiro
Copy link
Collaborator Author

コントリビューター記載の件、
image
こんな感じでアイコンだけ乗せてほしいですけど大丈夫ですか、、?

@s1204IT
Copy link
Collaborator

s1204IT commented Oct 20, 2024

コントリビューター記載の件、 image こんな感じでアイコンだけ乗せてほしいですけど大丈夫ですか、、?

これリリースノート内でメンションしないとダメなんですよね。
それでも良ければ

@s1204IT s1204IT removed the request for review from Chipppppppppp October 20, 2024 03:53
@areteruhiro
Copy link
Collaborator Author

areteruhiro commented Oct 20, 2024

Screenshot_2024-10-20-13-18-49-053_com.github.android.jpg

こんな感じで、追加した機能 by
だけで、大丈夫です。
逆に、したのコントリビューター欄は必要無いです。

@s1204IT
Copy link
Collaborator

s1204IT commented Oct 20, 2024

Contributors 項に表示することなくメンションだけってのは出来ないんですよね

@areteruhiro
Copy link
Collaborator Author

え、不便なんですね、、

@areteruhiro
Copy link
Collaborator Author

機能追加は出来たら載せていただきたいです。

バージョンの更新は、直接コードを作ったわけじゃないので必要ないです。

@areteruhiro
Copy link
Collaborator Author

細かい訂正(名前の訂正)などは、報告しなくて修正してもらって大丈夫です。

@s1204IT s1204IT changed the title LsPatch用着信音を鳴らす LSPatch 時の着信音が鳴らない問題を修正 Oct 26, 2024
@s1204IT s1204IT merged commit 789b412 into Chipppppppppp:master Oct 26, 2024
1 check passed
@areteruhiro
Copy link
Collaborator Author

areteruhiro commented Oct 26, 2024

Contributors 項に表示することなくメンションだけってのは出来ないんですよね

一応、機能の追加系は
載せといてもらっていいですか(by〜)
やる気的に

今、既読した人確認出来る機能、あるので早めにPRしときますね

@areteruhiro
Copy link
Collaborator Author

areteruhiro commented Oct 26, 2024

あと、LsPatch下の際、通知が来ないという、報告が、こちらがわで上がってきてたのですが
解決方法は、パッチ済みの14.3.2?で一度ログインしてから、
14.16.0に更新をするという形で解決できました。

あと、LsPatchだと生態情報が連携できていないという、報告も上がっています。

色々、お手間をおかけしてますが、よろしくお願いいたします。

@s1204IT
Copy link
Collaborator

s1204IT commented Oct 26, 2024

Contributors 項に表示することなくメンションだけってのは出来ないんですよね

一応、機能の追加系は 載せといてもらっていいですか(by〜) やる気的に

今、既読した人確認出来る機能、あるので早めにPRしときますね

メインリリースの時は乗せる予定です。

@areteruhiro
Copy link
Collaborator Author

areteruhiro commented Oct 26, 2024

Contributors 項に表示することなくメンションだけってのは出来ないんですよね

一応、機能の追加系は 載せといてもらっていいですか(by〜) やる気的に

今、既読した人確認出来る機能、あるので早めにPRしときますね

メインリリースの時は乗せる予定です。

ありがとうございます!!

いまのissue系はほぼ、解決出来そうですね

いつもありがとうございます。

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.

[Bug]: 音声通話の着信音が鳴らない
2 participants