btcpayserver / woocommerce-plugin

Plugin for connecting Woocommerce with BTCPayServer
https://btcpayserver.org/
Other
51 stars 41 forks source link

Replace BTC logo #27

Closed dalijolijo closed 5 years ago

dalijolijo commented 5 years ago

The BTC logo is irritating when altcoins and no bitcoin are accepted on the running BTCPay server instance. Therefore, I suggest that the Bitcoin logo be replaced by a neutral logo. This logo is a suggestion, which can also be adjusted. Suggestions are welcome.

pavlenex commented 5 years ago

I don't like the approach. You can't just enforce the image you like to everybody. If you want, you can code the image to be changeable, but PR, where you delete something and simply replace it, isn't something I'd like to see merged.

So a NACK from me, because I don't like the approach. If you can make it changeable from the user-end then I'd reconsider. On a side that the proposed image isn't something people would recognize, but I understand your point and I think the default BTC image should be changeable, but not enforced.

rockstardev commented 5 years ago

https://github.com/btcpayserver/btcpayserver#contributing

DM me on Mattermost if you want to talk details.