Skip to content

Created flake for project#40

Open
inkpot-monkey wants to merge 16 commits intoAyms:masterfrom
ngi-nix:ngi-nix
Open

Created flake for project#40
inkpot-monkey wants to merge 16 commits intoAyms:masterfrom
ngi-nix:ngi-nix

Conversation

@inkpot-monkey
Copy link

Pretty simple project to package as everything is already contained within the repository.

There are some apps that load the browser version and some addition checks that make use of the general tests in the project

@inkpot-monkey inkpot-monkey deleted the ngi-nix branch August 18, 2021 16:46
@inkpot-monkey
Copy link
Author

Sorry this was too eager, need to get this check by the ngi review team

@inkpot-monkey inkpot-monkey restored the ngi-nix branch August 18, 2021 16:54
@Ayms
Copy link
Owner

Ayms commented Aug 18, 2021

Yes super simple to package, the only external thing is the third party check for the code loaded https://github.com/Ayms/bitcoin-transactions/blob/master/hash.txt (not so external right now because it is myself)

inkpot-monkey and others added 15 commits August 19, 2021 08:45
1. src now equals lib.cleanSource ./.
2. supportedSystems expanded to any system that can run a browser
3. combined apps into single script with flags
4. created nixosModule which creates a .desktop file
5. removed unused version var
6. enclose bash vars in ""
Co-authored-by: Aaron Janse <aaron@ajanse.me>
@inkpot-monkey inkpot-monkey reopened this Oct 8, 2021
@inkpot-monkey
Copy link
Author

Hi,

This flake has been completed and integrates this project into the Nix ecosystem :0

It builds a package and exposes a shell command
nix run github:Ayms/bitcoin-transactions which opens the project with xdg-open

it also exposes a flag (clear || c) which runs the walletclear.html

and makes it installable as a desktop item on NixOS through a nixosModule (which is why the png has been added to the repo as well)

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.

2 participants