stampchain-io / BTCStampsExplorer

Bitcoin Stamps API / Explorer
https://bitcoinstamps.xyz
GNU Affero General Public License v3.0
0 stars 2 forks source link

image upload #33

Closed dev4pear closed 2 months ago

reinamora137 commented 3 months ago

is this fixing the exposure of the quicknode api key discussed in https://github.com/stampchain-io/BTCStampsExplorer/issues/31 ?

Perhaps this is solely re: https://github.com/stampchain-io/BTCStampsExplorer/issues/7

dev4pear commented 3 months ago

31 I think, this issue is caused by wrong api call and already fixed here

https://github.com/stampchain-io/BTCStampsExplorer/commit/4f42cd7809bea461ef03fbb59ce5c4fdf1e01d53

And, actually we don't get this error.

reinamora137 commented 3 months ago

Seem to be getting this error when trying to connect leather wallet. Associated with : https://github.com/stampchain-io/BTCStampsExplorer/issues/34

image

dev4pear commented 3 months ago

image It seems we can connect leather wallet successfully in bitcoinstamps.xyz

image But, got stucks about crypto.randomUUID when we try to connect leather wallet in dev.bitcoinstamps.xyz

In this PR, I've tested about leather wallet connection locally, and it works fine same as bitcoinstamps.xyz I think, we can merge this PR for now.

Let me know your idea about this.