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

CeramicClient 0.10.0 is incompatible with ThreeIDProvider.CeramicAPI. #1152

Open
RobertoSnap opened this issue Mar 24, 2021 · 5 comments
Open

Comments

@RobertoSnap
Copy link

Describe the bug
With "3id-did-provider": "^0.5.1" and "@ceramicnetwork/http-client": "^0.10.0", it seems that CeramicClient is incompatible with ThreeIDProvider.CeramicAPI.

To Reproduce
Steps to reproduce the behavior:

  1. yarn add 3id-did-provider @ceramicnetwork/http-client
  2. Write function
 const API_URL = 'https://ceramic-clay.3boxlabs.com';
            const ceramic = new CeramicClient(API_URL);
            const threeIdProvider = await ThreeIdProvider.create({ ceramic: ceramic, getPermission: async () => ['/'], seed: new Uint8Array(randomBytes(32)) });
           
  1. Should see type error for ThreeIdProvider.create
  2. yarn add "@ceramicnetwork/http-client": "^0.9.6",
  3. Type error resolves

Expected behavior
Expected no type error :)

Screenshots
image

Ceramic versions
"@ceramicnetwork/http-client": "^0.10.0"
"3id-did-provider": "^0.5.1"

Machine, OS, browser information (please complete the following information):
MACOS

Additional context
Add any other context about the problem here.

@oed
Copy link
Member

oed commented Apr 7, 2021

Is this still happening with the latest versions of these packages?

@oed oed added the need/triage label Apr 7, 2021
@tahpot
Copy link

tahpot commented Sep 5, 2021

@oed

There are still issues here with setDid:

Screen Shot 2021-09-05 at 12 05 59 pm

Using:

  • "@ceramicnetwork/3id-did-resolver": "^1.4.0",
  • "@ceramicnetwork/http-client": "^1.2.0",

@tahpot
Copy link

tahpot commented Sep 5, 2021

I'm also getting the same error as @RobertoSnap when initializing IDX:

const idx = new IDX({ ceramic })

@PaulLeCam
Copy link
Contributor

It is likely that you have different versions of these packages in your dependency tree, please make sure only one version of each package is present.

@tahpot
Copy link

tahpot commented Sep 6, 2021

Thanks @PaulLeCam, you're correct.

I am using a monorepo and another package had an older dependency.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants