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.

Sign up for GitHub

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

Jump to bottom

Need to display the QR code using the Github pages of CHIP repository #3266

Closed
dhrishi opened this issue Oct 15, 2020 · 1 comment · Fixed by #19873
Closed

Need to display the QR code using the Github pages of CHIP repository #3266

dhrishi opened this issue Oct 15, 2020 · 1 comment · Fixed by #19873
Assignees
@dhrishi
Labels
documentation Improvements or additions to documentation enhancement New feature or request future p3 priority 3 work V1.X
Milestone
V1.0

Comments

@dhrishi
Copy link
Contributor

dhrishi commented Oct 15, 2020

Problem

With PR #3242 we have added the support of displaying the QR code on a webpage. This will be helpful for devices that do not have a physical display (to scan/print the QR code). Currently, the page is under the Github pages of my personal Github fork. We need to move this to CHIP repository.

Proposed Solution

I think, the CHIP maintainers should enable Github pages (gh-pages) for the CHIP repository. Once that is done, we can merge the changes through a PR.
@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the label feature_request to this issue, with a confidence of 0.91. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@dhrishi dhrishi mentioned this issue Oct 15, 2020
Merged
@woody-apple woody-apple added enhancement New feature or request p3 priority 3 work V1.0 and removed feature request labels Oct 22, 2020
@woody-apple woody-apple added this to the V1.0 milestone Oct 22, 2020
@woody-apple woody-apple added the documentation Improvements or additions to documentation label Jan 27, 2021
@woody-apple woody-apple self-assigned this Feb 25, 2021
@franck-apple franck-apple modified the milestones: V1.0, Test Event 1 Feb 26, 2021
@franck-apple franck-apple added TE2 and removed TE1 labels Mar 9, 2021
@woody-apple woody-apple removed the TE2 label Apr 6, 2021
@woody-apple woody-apple modified the milestones: Test Event 2, V1.0 Apr 6, 2021
@woody-apple woody-apple removed the V1.0 label Oct 27, 2021
andy31415 added a commit that referenced this issue Feb 22, 2022
@andy31415
Added an initial index.md for GH pages.
a855317 
This is to start to address #3266 - we should later be able to use gh pages to host the QR code display.
@dhrishi dhrishi mentioned this issue Jun 5, 2022
Closed
This was referenced Jun 23, 2022
Merged
Merged
@woody-apple woody-apple removed their assignment Jun 23, 2022
jrhees-cae added a commit to jrhees-cae/connectedhomeip that referenced this issue Aug 3, 2023
@jrhees-cae
TC-DRLK-2.2/2.3/2.12:
5fca778 
- Modify flow to allow for RequirePINforRemoteOperation to be TRUE or FALSE
- Reorder steps 7-9 to ensure subsequent test of WrongCodeEntryLimit doesn't already have existing failed entry attempts
- Fix incorrect PICS (use .Write PICS) for writing WrongCodeEntryLimit and UserCodeTemporaryDisableTime

Fixes project-chip#3263
Fixes project-chip#3265
Fixes project-chip#3266
@jrhees-cae jrhees-cae mentioned this issue Aug 3, 2023
Closed
jrhees-cae added a commit to jrhees-cae/connectedhomeip that referenced this issue Aug 4, 2023
@jrhees-cae
TC-DRLK-2.2/2.3/2.12: Modify flow to allow for RequirePINforRemoteOpe…
c6a52b1 
…ration to be TRUE or FALSE

- Modify flow to allow for RequirePINforRemoteOperation to be TRUE or FALSE
- Reorder steps 7-9 to ensure subsequent test of WrongCodeEntryLimit doesn't already have existing failed entry attempts
- Fix incorrect PICS (use .Write PICS) for writing WrongCodeEntryLimit and UserCodeTemporaryDisableTime

Fixes project-chip#3263
Fixes project-chip#3265
Fixes project-chip#3266
mergify bot pushed a commit that referenced this issue Aug 10, 2023
@jrhees-cae @restyled-commits
TC-DRLK-2.2/2.3/2.12: Modify flow to allow for RequirePINforRemoteOpe…
1b6ea10 
…ration to be TRUE or FALSE (#28523)

* TC-DRLK-2.2/2.3/2.12: Modify flow to allow for RequirePINforRemoteOperation to be TRUE or FALSE

- Modify flow to allow for RequirePINforRemoteOperation to be TRUE or FALSE
- Reorder steps 7-9 to ensure subsequent test of WrongCodeEntryLimit doesn't already have existing failed entry attempts
- Fix incorrect PICS (use .Write PICS) for writing WrongCodeEntryLimit and UserCodeTemporaryDisableTime

Fixes #3263
Fixes #3265
Fixes #3266

* Restyled by autopep8

* Exclude .yaml tests from CI due to use of EqualityTests pseudo-cluster

* Add zap-regen'd files

* Exclude .yaml tests from CI linux due to use of EqualityTests pseudo-cluster

---------

Co-authored-by: Restyled.io <commits@restyled.io>
abpoth pushed a commit to abpoth/connectedhomeip that referenced this issue Aug 15, 2023
@jrhees-cae @restyled-commits
TC-DRLK-2.2/2.3/2.12: Modify flow to allow for RequirePINforRemoteOpe…
a66c77c 
…ration to be TRUE or FALSE (project-chip#28523)

* TC-DRLK-2.2/2.3/2.12: Modify flow to allow for RequirePINforRemoteOperation to be TRUE or FALSE

- Modify flow to allow for RequirePINforRemoteOperation to be TRUE or FALSE
- Reorder steps 7-9 to ensure subsequent test of WrongCodeEntryLimit doesn't already have existing failed entry attempts
- Fix incorrect PICS (use .Write PICS) for writing WrongCodeEntryLimit and UserCodeTemporaryDisableTime

Fixes project-chip#3263
Fixes project-chip#3265
Fixes project-chip#3266

* Restyled by autopep8

* Exclude .yaml tests from CI due to use of EqualityTests pseudo-cluster

* Add zap-regen'd files

* Exclude .yaml tests from CI linux due to use of EqualityTests pseudo-cluster

---------

Co-authored-by: Restyled.io <commits@restyled.io>
s07641069 pushed a commit to s07641069/connectedhomeip that referenced this issue Aug 22, 2023
@jrhees-cae @restyled-commits @s07641069
TC-DRLK-2.2/2.3/2.12: Modify flow to allow for RequirePINforRemoteOpe…
1d05925 
…ration to be TRUE or FALSE (project-chip#28523)

* TC-DRLK-2.2/2.3/2.12: Modify flow to allow for RequirePINforRemoteOperation to be TRUE or FALSE

- Modify flow to allow for RequirePINforRemoteOperation to be TRUE or FALSE
- Reorder steps 7-9 to ensure subsequent test of WrongCodeEntryLimit doesn't already have existing failed entry attempts
- Fix incorrect PICS (use .Write PICS) for writing WrongCodeEntryLimit and UserCodeTemporaryDisableTime

Fixes project-chip#3263
Fixes project-chip#3265
Fixes project-chip#3266

* Restyled by autopep8

* Exclude .yaml tests from CI due to use of EqualityTests pseudo-cluster

* Add zap-regen'd files

* Exclude .yaml tests from CI linux due to use of EqualityTests pseudo-cluster

---------

Co-authored-by: Restyled.io <commits@restyled.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dhrishi dhrishi

Labels
documentation Improvements or additions to documentation enhancement New feature or request future p3 priority 3 work V1.X
Projects
None yet
Milestone
V1.0
3 participants
@dhrishi @woody-apple @franck-apple