Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add RawTransactionModal for TRX processes #168

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

itailiors
Copy link
Collaborator

added RawTransactionModal as a component,
add for classes spend,invest,wallet.

added max money to invest (5 btc)

@itailiors itailiors marked this pull request as ready for review November 6, 2024 23:11
@itailiors
Copy link
Collaborator Author

Hey @dangershony, I just noticed your note in the task:

"We should limit this on the invest page.
The cap should be 0.1 bitcoin?"

Currently, we’ve set a minimum of 2 BTC on testnet 😅

It seems like we have a couple of options (or let me know if you think of a smarter approach):

  1. Add fields during project creation for setting both minimum and maximum investments. This way, we could set a cap at 0.1 BTC. However, this might not be ideal if founders later need to adjust these values.

  2. Separate logic for TBTC and BTC on the invest page, though this could lead to hardcoding.

Option 1 (with some variation) seems like the better one, allowing founders to define their own minimum and maximum.

(in the meanwhile i'll just set the minimum for 0.01 and max for 0.1 :) )

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.

1 participant