search

EAWF / BTC-Toolbox

A collection of simple programming tools to aid entities in the action of accepting BTC payments directly to HD Wallets within their control.
The Unlicense
9 stars 6 forks source link

Proposal to Consolidate to PHP version and Fork Other OS's #64

Closed EAWF closed 4 years ago

EAWF commented 4 years ago

I'm starting to feel more like this repo should become a PHP Only version that gets turned into a PHP Class of its own, and others can fork other versions for different programming languages.

I can see how it would be easier to maintain the repo that way.

Your thoughts: @Septem151 @coinables @PeterNSteinmetz ?

Septem151 commented 4 years ago

I respect your decision to make the project PHP only. Sounds like a good idea.

EAWF commented 4 years ago

Still in proposal stage. While I have my thoughts, I'm still not convinced that it's the right way to go. Looking for reasons why to keep it the way it is. Not like we have a lot of pseudocode describing the processes yet. I'm not sure what impact changes in the pseudocode would have on others who are maintaining the other OS's and programming platforms, not to mention when other dependent resources used here have modifications or forks themselves.

Here's a "for instance" to ponder. I used the php include path to position the data file on the include path so it was out of range of the public, yet easily in range for the PHP for getBTC.conf as well as getBTC.json. I have no clue if that capability is available in other programming languages without having to force a programmer to write a function that accomplishes that feature in that language.

EAWF commented 4 years ago

OK, read this: https://medium.com/@adamhjk/monorepo-please-do-3657e08a4b70 and am convinced that while my proposal would work in the short run, it would be much harder to deal with in the long run, so I'll delete this issue.