Skip to content
This repository was archived by the owner on Feb 6, 2019. It is now read-only.

Productionization #12

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Productionization #12

wants to merge 7 commits into from

Conversation

kizzx2
Copy link

@kizzx2 kizzx2 commented Jan 20, 2019 •
edited
Loading

  • Configurable owner API URL instead of hardcoded localhost:13420
  • Fix NaN in output-listing table
  • Fix flickering tx-listing table due to reverse() keeps being called in view
  • Dockerfile to allow people to easily host their wallet. The WALLET_READONLY will have nginx proxy only readonly calls to owner_api
  • npm update

kizzx2 added 7 commits January 20, 2019 15:28
@kizzx2
misc: npm update
bf1caeb
@kizzx2
feat: configurable owner api endpoints / readonly wallet / dockerfile
7232abe
@kizzx2
fix: output-listing not shown correctly
0681e9b
@kizzx2
ci: fix build syntax
9595ede
@kizzx2
fix: flickering tx-listing
772f814 
Don't mutate the list in template since the component might rerender
without `txs` being updated, leading to a tx-listing that keeps
reversing itself.
@kizzx2
fix: only do substitution on first run
8e994c8 
So that this survives container restart
@kizzx2
fix: build file was missing assets
20a9e17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kizzx2