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

Fix import issue with WalletConnectUtils #1341

Merged
merged 1 commit into from
Apr 12, 2024

Conversation

defnotpaul
Copy link
Contributor

@defnotpaul defnotpaul commented Apr 9, 2024

Thank you for considering this change, and please let me know if there's any additional context I can provide. I have not tested this using SPM, and wonder if there may be some conflicts with that installation method. Perhaps we can wrap this in some additional flags to make it only available for SPM, if that's the case?

It's worth noting this issue is blocking installation of the more recent versions of this library via cocoapods.

Description

The following compilation error occurs when installed via cocoapods with DEBUG enabled.

  37 |         )
  38 |         let signature = CacaoSignature(
> 39 |             t: WalletConnectUtils.CacaoSignatureType.eip191,
     |                ^ cannot find 'WalletConnectUtils' in scope
  40 |             s: "invalid_signature",
  41 |             m: nil
  42 |         )

Related: #1340

How Has This Been Tested?

Run a local install + build.

Due Dilligence

  • Breaking change
  • Requires a documentation update

The following compilation error occurs when installed via cocoapods with
DEBUG enabled.

```
  37 |         )
  38 |         let signature = CacaoSignature(
> 39 |             t: WalletConnectUtils.CacaoSignatureType.eip191,
     |                ^ cannot find 'WalletConnectUtils' in scope
  40 |             s: "invalid_signature",
  41 |             m: nil
  42 |         )
```
Copy link

sonarcloud bot commented Apr 9, 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

@andrew54068
Copy link

Faced this issue, too.

@bkrem bkrem requested a review from llbartekll April 11, 2024 14:50
@llbartekll llbartekll merged commit 261ed0a into WalletConnect:develop Apr 12, 2024
4 of 6 checks passed
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.

3 participants