-
Notifications
You must be signed in to change notification settings - Fork 148
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
Open the Leather extension programmatically (API needed) #5800
Comments
@bilalanees98 would be able to provide more context re timeline and specifications |
Hey team We tried hitting the chrome-extension url and opening it from there, but that ends up giving us a Wondering if we can get an API that will help us open up the wallet extension. The goal is to navigate users directly from the dotlocker dashboard to either the leather extension popup or to the maximized version in a new tab (preferred). We're on a bit of a tight deadline with the launch, would be great if this could be ready by mid next-week (28th August). Happy to work towards finding a better date as well. |
Hi @bilalanees98 , Thanks for providing this information. I will discuss it with the team and report back shortly. |
Let's look into whether we can add a generic API for opening the extension to any page / route. |
Hi @bilalanees98 , I have got to work on this and have a draft PR ready where I have added a new RPC call - If you wanted, you could download and install this build of Leather wallet from my draft PR to check if it's working for you. |
Thanks @pete-watters I'll get someone to try this out and report back. |
hey @pete-watters tried this on my end and its working 👍 |
Thanks for confirming @Imamah-Zafar 👍 FYI we have decided to rename the API as I have created a release that includes this work and will get it out ASAP |
## [6.47.0](v6.46.0...v6.47.0) (2024-08-28) ### Features * add link and unlink mono script ([29f2174](29f2174)) * add mono link ui script ([717737c](717737c)) * mock hiro ft requests ([6987609](6987609)) * mock nfts and txs hiro requests ([318378d](318378d)) ### Bug Fixes * disable brc-20 sends, ref leather-io/issues[#5773](#5773) ([aafd992](aafd992)) * rename dialog as sheet, closes ref leather-io/issues[#268](#268) ([ec64755](ec64755)) ### Internal * add open API, ref leather-io/issues[#5800](#5800) ([3df62f7](3df62f7)) * icons using ui lib svgs, closes leather-io/issues[#123](#123) ([3e8314e](3e8314e)) * post-release merge back ([56d021b](56d021b)) * show BRC-20 tokens on ledger, ref [#5797](#5797) ([bcc40f7](bcc40f7))
@bilalanees98 @Imamah-Zafar : I just published a release of Leather that includes this API update - @markmhendrickson: I opened this new task for extending the API. I just wanted to get this out to meet the deadline |
I can confirm this update is now live. Let me know if you experience any issues. Good luck with the launch 🚀 |
Sorry for the late follow up. @renashah @bilalanees98 Curious what the .locker user flow is. What is the user expected to do after you've opened the wallet? What page do you open on? I wonder if the API should be more narrowly scoped to the task the user needs to perform, rather than just opening it. @pete-watters I wonder what the promise does in your implementation? If the wallet opens, and no return value is expected, I'd imagine the promise to return immediately, rather than hang indefinitely. |
Thanks @kyranjamie , let me improve it. Do you have any thoughts on this task - #5821. Mark suggested extending this to allow the page to be opened to be passed but I'm unsure if that has other security implications |
## [6.47.1](v6.47.0...v6.47.1) (2024-09-03) ### Bug Fixes * adjust BigTitle padding and close, ref leather-io/extension[#5802](#5802) ([6b9566a](6b9566a)) * don't show send in header ([d8ed76a](d8ed76a)) * only show Send title above small view, ref [#5802](#5802) ([f45fedf](f45fedf)) * remove px for all page headers above sm, ref [#5802](#5802) ([dae0f47](dae0f47)) * send inscription formatting and InfoCardButton icon colour, ref leather-io/extension[#5828](#5828) ([afc50d5](afc50d5)) * update padding of onboarding header to align buttons, ref [#5802](#5802) ([203629d](203629d)) ### Internal * fix padding of asset list items in activity, ref [#5802](#5802) ([3e4d8a2](3e4d8a2)) * improve open API and add tests, ref leather-io/issues[#5800](#5800) ([9d52d9e](9d52d9e)) * post-release merge back ([e554376](e554376)) * update link to docs on open, ref [#5800](#5800) ([003a6ad](003a6ad))
Hey Leather eng team! We need a way to open the Leather extension programmatically. Is there a way for Leather to allow our app to do this? Right now, Chrome will not allow it.
We'll need it to launch .locker TLD.
The text was updated successfully, but these errors were encountered: