nextcloud/android

status dialog: not able to set only "icon" / only "clear at"

Open

#9,855 opened on 2022年2月17日

GitHub で見る
 (3 comments) (1 reaction) (0 assignees)Java (3,752 stars) (1,719 forks)batch import
approvedbugfeature: statusgood first issue

説明

Steps to reproduce

  1. set only icon or only clear at without to enter status message
  2. click set staus

Expected behaviour

  • icon or clear at are set

Actual behaviour

  • icon or clear at not set

Environment data

Nextcloud app version: 3.19.0

solution

api seems to set single modification for icon or "clear at" only if status message is not empty (must be at least whitespace)

see android talk commit https://github.com/nextcloud/talk-android/pull/1668/commits/a89c5952fa2747c8877c7882a6106311257722bf

コントリビューターガイド

技術スタック
javaandroid
領域
mobile
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Android development basicsJava
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60
調査方針
The issue is about setting only icon or clear at without a status message in the Nextcloud Android app. The expected behavior is that setting just the icon or clear at should work, but the API currently requires a non empty status message (at least whitespace). A similar fix was applied in the Android Talk app: see commit a89c5952fa2747c8877c7882a6106311257722bf. The solution likely involves modifying the API call in the status dialog to handle cases where status message is empty. Look at the status dialog implementation in the Nextcloud Android repository for the relevant network request code.
status dialog: not able to set only "icon" / only "clear at" · nextcloud/android#9855 | Good First Issue