Contributing to iOS app

[Alessio Nossa]

I am aware of the bug reported by Houman. To get around this problem I
have implemented a workaround: when the action needs to make changes
to the configuration, the main app is launched and the user can then
specify in Shortcut's action a URL to open when the action is
completed (default will be "shortcuts://"). This workaround has been
used by other apps like Working Copy [1] (in "Clone repository from"
action) for actions that need to be executed by the main app.
With this solution you cannot return results of the operation to
Shortcuts, but you can successfully get the work done and if something
went wrong, there is Wireguard's log to check.

One thing I had to change in the app to accomplish this is to move
TunnelsManager initialization from MainViewController to AppDelegate.
It will not slow down app startup since it is an asynchronous action
and I think it is fine to initialize the main component of the app's
logic, outside the view hierarchy.

Hopefully, one day the bug will be fixed and moving the action logic
to Intents Extension will be straightforward.

Regards,
Alessio

[1]: https://workingcopyapp.com/

Il giorno mar 25 gen 2022 alle ore 15:52 Houman <houmie at gmail.com> ha scritto:
>
> Shortcut integration for Wireguard on iOS would be amazing. But did you actually get it working?
>
> I think due to this Apple bug that has been open for years: https://developer.apple.com/forums/thread/96020, it is not possible to work with Siri shortcuts on the network extension.
> But if your solution works, it would be amazing!!
>
> Regards,
> Houman
>
> On Tue, 25 Jan 2022 at 14:20, Alessio Nossa <alessio.nossa+list at gmail.com> wrote:
>>
>> Hello,
>> I’d like to contribute to wireguard-apple with Shortcuts integration
>> for the iOS app (this is a feature in the todo-list too).
>> The implementation is almost complete, I’d like to be allowed to merge
>> my contribution with the main app (or to have information on how I can
>> submit my contributions).
>>
>> Kind regards,
>> Alessio Nossa