Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ixv
Copy link
Contributor

@ixv ixv commented May 26, 2021

Here is the bitcoin wallet

Beyond being just a bitcoin wallet, its also Tlon's first attempt at distributing multiple frontend packages. Includes 2 new hoon apps, a new pkg/btc-wallet, some changes to pkg/interface to make custom tiles work with images, and changes to glob to make it work with multiple packages.

Thanks to @timlucmiptev for writing the bulk of the backend code, and thanks to @pkova for contributing to both frontend and backend.

This is still somewhat of an alpha, so we are releasing this in an unstarted state. People who feel savvy enough to use this can |start %btc-wallet

@liam-fitzgerald
Copy link
Contributor

I'd argue that this can and should go out to next-userspace before hitting master

@matildepark
Copy link
Contributor

I'd argue that this can and should go out to next-userspace before hitting master

Idk if it can with the glob changes — janeway isn't updated to handle the new glob logic. But I like the intention.

@tacryt-socryp
Copy link
Contributor

Could the glob changes be separately targeted to hit master?

Copy link
Contributor

@tacryt-socryp tacryt-socryp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving the %glob and %file-server changes.

@ixv ixv force-pushed the release/bitcoin-wallet branch from 3883eac to b8ec17f Compare May 27, 2021 01:30
@ixv ixv changed the base branch from master to release/next-userspace May 27, 2021 01:31
Copy link
Contributor

@liam-fitzgerald liam-fitzgerald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(I'm only reviewing the build process and integration with the rest of landscape)
Discussed OOB, everything LGTM!

@jtobin
Copy link
Contributor

jtobin commented May 27, 2021

I'd argue that this can and should go out to next-userspace before hitting master

Idk if it can with the glob changes — janeway isn't updated to handle the new glob logic. But I like the intention.

Anything I need to do here? Can't remember if I was supposed to update something and forgot.. 😝

@jtobin
Copy link
Contributor

jtobin commented May 27, 2021

changes to glob to make it work with multiple packages.

(ah, nm)

@liam-fitzgerald
Copy link
Contributor

I've done the glob changes, PRing now

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.

10 participants