Skip to content

Commit 49a99d8

Browse files
committed
use link for qr code
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
1 parent 449d1c3 commit 49a99d8

1 file changed

Lines changed: 14 additions & 12 deletions

File tree

.github/workflows/qa.yml

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -75,17 +75,22 @@ jobs:
7575
archive: false
7676

7777
- name: Create QR code
78-
id: qr
7978
if: ${{ steps.check-secrets.outputs.ok == 'true' }}
8079
env:
8180
ARTIFACT_URL: ${{ steps.upload-apk.outputs.artifact-url }}
8281
run: |
8382
sudo apt-get -y install qrencode
84-
{
85-
echo 'text<<QR_EOF'
86-
qrencode -t ASCII -m 2 "$ARTIFACT_URL" | sed 's/#/█/g'
87-
echo 'QR_EOF'
88-
} >> "$GITHUB_OUTPUT"
83+
qrencode -s 8 -o qr.png "$ARTIFACT_URL"
84+
85+
- name: Upload QR code
86+
id: upload-qr
87+
if: ${{ steps.check-secrets.outputs.ok == 'true' }}
88+
uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f
89+
with:
90+
path: qr.png
91+
if-no-files-found: error
92+
retention-days: 7
93+
archive: false
8994

9095
- name: Comment PR
9196
if: ${{ steps.check-secrets.outputs.ok == 'true' }}
@@ -99,16 +104,13 @@ jobs:
99104
| | |
100105
|---|---|
101106
| **Download** | [${{ steps.build-qa.outputs.apk-name }}](${{ steps.upload-apk.outputs.artifact-url }}) (${{ steps.build-qa.outputs.apk-size }}) |
107+
| **QR code** | [Open the QR code for this download](${{ steps.upload-qr.outputs.artifact-url }}) |
102108
| **Commit** | ${{ github.event.pull_request.head.sha }} |
103109
| **Version** | `${{ github.event.number }}` |
104110
| **Available until** | 7 days after this build |
105111
106-
```
107-
${{ steps.qr.outputs.text }}
108-
```
109-
110112
The QA build installs alongside a released Nextcloud app, so you can keep
111113
using your existing install while testing.
112114
113-
Downloading the file requires a GitHub account, so scan the QR code on a
114-
device that is signed in, or transfer the APK to it.
115+
Downloading the file requires a GitHub account, so open this link on the
116+
device you want to test on, or transfer the APK to it.

0 commit comments

Comments
 (0)