Adonit / LAMY-iPadOS-SDK

0 stars 0 forks source link

iPad, swift - unable to use some view controllers #1

Closed Kaven01 closed 4 months ago

Kaven01 commented 4 months ago

I am trying to enable LAMY by set of view controllers provided by this SDK, and have issues when using some of them. I imported the framework according to the wiki ( https://github.com/Adonit/LAMY-iPadOS-SDK/wiki/Getting-Started-Guide ), which works. I then run into issues when trying to instantiate provided view controllers using UIStoryboard.instantiateLamyViewController(withIdentifier:)

I can successfully create these controllers: LamyViewControllerUnifiedStylusConnectionAndSettingsIdentifier, LamyViewControllerDebugStatusIdentifier, LamyViewControllerPressToConnectIdentifier, LamyViewControllerUnifiedStatusButtonAndConnectionAndSettingsIdentifier . These work as expected, can be placed in my views etc.

However if I add some of the following controllers, the code does not compile : LamyViewControllerBatteryIdentifier, LamyViewControllerShortCutsIdentifier, LamyViewControllerWritingStyleIdentifier . The error issued by xcode during compile is (for battery controller):

Undefined symbol: _LamyViewControllerBatteryIdentifier

Strange is that xcode autocomplete sees these identifiers (eg. LamyViewControllerBatteryIdentifier ), and xcode does not show any errors during writing code. Error appears only during compilation.

adonitDevel commented 4 months ago

Hi @Kaven01 thanks for your feedback This issue has been fixed by version 1.0.2 Thanks.

Kaven01 commented 4 months ago

Hi @Kaven01 thanks for your feedback This issue has been fixed by version 1.0.2 Thanks. That is better, I can now use LamyViewControllerBatteryIdentifier and LamyViewControllerShortCutsIdentifier,

However LamyViewControllerWritingStyleIdentifier is still not available. This time error in xcode is "Cannot find 'LamyViewControllerWritingStyleIdentifier' in scope". It appears immediatelly, not later during build as before. The problem is with this one identifier only, all others work.

adonitDevel commented 4 months ago

Hi @Kaven01, The LamyViewControllerWritingStyleIdentifier is a legacy feature, so I removed it. Sorry for your inconvenience.

Kaven01 commented 4 months ago

Oh, I see. OK, no need for this UI then.