-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Cluster command detail page #11354
Merged
woody-apple
merged 14 commits into
project-chip:master
from
JasonLiuZhuoCheng:cluster-command-detail-page
Nov 6, 2021
Merged
Cluster command detail page #11354
woody-apple
merged 14 commits into
project-chip:master
from
JasonLiuZhuoCheng:cluster-command-detail-page
Nov 6, 2021
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
PR #11354: Size comparison from d920345 to d1d2091 Decreases (1 build for esp32)
Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
JasonLiuZhuoCheng
force-pushed
the
cluster-command-detail-page
branch
from
November 3, 2021 20:00
c68ff2d
to
f7466c7
Compare
pullapprove
bot
requested review from
andy31415,
anush-apple,
balducci-apple,
bzbarsky-apple,
carol-apple,
cecille,
chrisdecenzo,
chulspro,
Damian-Nordic,
electrocucaracha,
emargolis,
erjiaqing,
franck-apple,
hawk248,
holbrookt,
jelderton,
jepenven-silabs,
jmartinez-silabs,
kghost,
LuDuda,
mlepage-google,
mrjerryjohns,
msandstedt and
pan-apple
November 3, 2021 22:41
PR #11354: Size comparison from 38fe11e to 13471a8 Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
austinh0
suggested changes
Nov 4, 2021
...java/com/google/chip/chiptool/clusterclient/clusterinteraction/ClusterInteractionFragment.kt
Outdated
Show resolved
Hide resolved
src/android/CHIPTool/app/src/main/res/layout/cluster_detail_fragment.xml
Outdated
Show resolved
Hide resolved
src/android/CHIPTool/app/src/main/res/layout/cluster_detail_fragment.xml
Outdated
Show resolved
Hide resolved
...main/java/com/google/chip/chiptool/clusterclient/clusterinteraction/ClusterDetailFragment.kt
Outdated
Show resolved
Hide resolved
...main/java/com/google/chip/chiptool/clusterclient/clusterinteraction/ClusterDetailFragment.kt
Outdated
Show resolved
Hide resolved
...main/java/com/google/chip/chiptool/clusterclient/clusterinteraction/ClusterDetailFragment.kt
Outdated
Show resolved
Hide resolved
...main/java/com/google/chip/chiptool/clusterclient/clusterinteraction/ClusterDetailFragment.kt
Outdated
Show resolved
Hide resolved
...main/java/com/google/chip/chiptool/clusterclient/clusterinteraction/ClusterDetailFragment.kt
Outdated
Show resolved
Hide resolved
PR #11354: Size comparison from 38fe11e to 993110b Increases above 0.2%:
Increases (31 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (1 build for linux)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Fast tracking, given this is a platform change |
JasonLiuZhuoCheng
force-pushed
the
cluster-command-detail-page
branch
from
November 4, 2021 19:16
7320cd0
to
7a0a726
Compare
austinh0
approved these changes
Nov 4, 2021
...main/java/com/google/chip/chiptool/clusterclient/clusterinteraction/ClusterDetailFragment.kt
Outdated
Show resolved
Hide resolved
...main/java/com/google/chip/chiptool/clusterclient/clusterinteraction/ClusterDetailFragment.kt
Show resolved
Hide resolved
...main/java/com/google/chip/chiptool/clusterclient/clusterinteraction/ClusterDetailFragment.kt
Outdated
Show resolved
Hide resolved
PR #11354: Size comparison from 29747a2 to d419b40 Decreases (1 build for esp32)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
JasonLiuZhuoCheng
force-pushed
the
cluster-command-detail-page
branch
from
November 4, 2021 21:32
d419b40
to
651c293
Compare
PR #11354: Size comparison from f874ab3 to 651c293 Full report (28 builds for efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
|
JasonLiuZhuoCheng
force-pushed
the
cluster-command-detail-page
branch
from
November 4, 2021 23:39
651c293
to
6b13e70
Compare
JasonLiuZhuoCheng
force-pushed
the
cluster-command-detail-page
branch
from
November 5, 2021 13:39
6b13e70
to
422df1f
Compare
woody-apple
approved these changes
Nov 5, 2021
PSONALl
pushed a commit
to PSONALl/connectedhomeip
that referenced
this pull request
Dec 3, 2021
* try to find a state where m5stack doesn't bootloop * add back class description * select different cluster, command will remove previous displayed parameter * add responseValueInfo class instead of string split * remove unused variable * Restyled by whitespace * Restyled by google-java-format * Restyled by gn * fix parameter response ui alignment issue * resolve comments * fix format * regenerate clusterInfoMapping.java * resolve comments * merge with master to get pass the check Co-authored-by: Restyled.io <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Change overview
Command detail page includes two autoCompleteTextView: clusterAutocomplete and commandAutoComplete. Based on the user input, these two field will display a list of possible options starting with the user's input. After selecting the commands, depending on the command, it will display a list of parameters for user to fill in. After filling in the parameters and hit the invoke button, it will display a "command success"/"command failed" with the corresponding callback value and type.
Testing
manually tested