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

support buying xmr without deposit or fee using passphrase #1463

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

woodser
Copy link
Contributor

@woodser woodser commented Dec 11, 2024

No description provided.

@woodser woodser marked this pull request as draft December 11, 2024 17:27
@woodser woodser force-pushed the deployment branch 2 times, most recently from 4bca41c to e6023c0 Compare December 11, 2024 18:18
@shortwavesurfer2009
Copy link
Contributor

shortwavesurfer2009 commented Dec 11, 2024

Are you planning to set a 0.11xmr cap on this so its only use is nocoiners?

Edit: if so

  • purchase 1: 0.11, no sec deposit
  • purchase 2: <=0.5, 0.1 sec deposit (20%)
  • purchase 3: <= 2.5, 0.5 sec deposit (20%)

@monerobull
Copy link

cap should be 0.5 XMR or a bit lower, maybe 0.25. 0.11 is already annoyingly low if you have no xmr and want to put for example $300 into it (because you need multiple trades to just get the minimum deposit amount).

@woodser woodser force-pushed the deployment branch 2 times, most recently from e14a174 to 2b02824 Compare December 12, 2024 23:19
@woodser woodser changed the title support passphrase-protected offers with no deposit (wip) support no deposit offers with passphrase Dec 12, 2024
@woodser woodser marked this pull request as ready for review December 12, 2024 23:19
@woodser
Copy link
Contributor Author

woodser commented Dec 12, 2024

Limit set to 0.5 XMR for no deposit offers with passphrase.

@woodser woodser force-pushed the deployment branch 4 times, most recently from 8cdf725 to 5d2e157 Compare December 14, 2024 17:40
@shortwavesurfer2009
Copy link
Contributor

Hey, the version that implements this will be a mandatory upgrade correct. It seems like it would be a change to the protocol and therefore require all clients to upgrade.

@woodser
Copy link
Contributor Author

woodser commented Dec 15, 2024

Yeah it's best to be a mandatory update, otherwise trades will fail if non updated clients try taking the new offer type.

@woodser woodser force-pushed the deployment branch 2 times, most recently from a01cf73 to f8a8407 Compare December 15, 2024 16:53
@woodser woodser changed the title support no deposit offers with passphrase support buying xmr without deposit or fee using passphrase Dec 16, 2024
@woodser
Copy link
Contributor Author

woodser commented Dec 16, 2024

Limit bumped to 1 XMR from this conversation: #1464 (comment)

@woodser woodser merged commit 775fbc4 into haveno-dex:master Dec 16, 2024
5 checks passed
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.

3 participants