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

LiveActivity.swift: widget URL string "Trio://" #197

Merged
merged 1 commit into from
May 18, 2024
Merged

Conversation

bjornoleh
Copy link
Contributor

Replaces widgetURL(URL(string: "freeaps-x://"))

Related to #136

@bjornoleh bjornoleh mentioned this pull request May 15, 2024
Copy link

sonarcloud bot commented May 16, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor

@marionbarker marionbarker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure how the widgetURL is used.
The modification looks reasonable from code review.
Does this mean a shortcut on the phone requires an update?

@bjornoleh
Copy link
Contributor Author

Tapping the Live activity widget was unaffected by the change (app opened before and after). But Trio:// is the correct url for whatever it's used for.

@MikePlante1
Copy link
Contributor

I concur with bjornoleh. Tested opening the app from both the lockscreen widget and the Dynamic Island both before and after applying this PR on a real phone and it worked as usual. Not really sure what (if anything) it affects.

Should it maybe use APP_URL_SCHEME from Config, though? Just a thought. Hardcoding to Trio seems to be fine, though.

@bjornoleh
Copy link
Contributor Author

I didn't quickly figure out how to use APP_URL_SCHEME here, so went with the hardcoded way instead. Shone more than good enough for this anyways.

@MikePlante1
Copy link
Contributor

Actually... I even just deleted that line completely and it still seems to work fine. 🤷 So not sure what it's even for.

@MikePlante1
Copy link
Contributor

Merging this with several approvals and it seems it could even be deleted.

@MikePlante1 MikePlante1 merged commit 5e51344 into dev May 18, 2024
1 check passed
@MikePlante1 MikePlante1 mentioned this pull request May 20, 2024
@dnzxy dnzxy deleted the widgetURLstring branch May 31, 2024 20:36
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.

4 participants