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.

Sign up for GitHub

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

Multiple-URI support in parse_uri and make_uri #9756

Open
wants to merge 58 commits into
base: master
Choose a base branch
from
+615 −64
Open

Multiple-URI support in parse_uri and make_uri #9756

Changes from 1 commit
Commits
Show all changes
58 commits
Select commit Hold shift + click to select a range
7469d69
Multiple url functionality.
U65535F Jan 28, 2025
d1f69b9
Remove commented line.
U65535F Jan 28, 2025
1b1e930
Fix error by removing repeated code block.
U65535F Jan 28, 2025
168668a
fix no member named errors
U65535F Jan 28, 2025
ab0c39e
Add function overloads for make_uri to make it compatible with older …
U65535F Jan 28, 2025
25b3ef2
Fix tests
U65535F Jan 29, 2025
a55f1f1
Fix function overloads
U65535F Jan 29, 2025
246926a
Fix errors of parse_uri saying that m_wallet was not found.
U65535F Jan 29, 2025
bfa2c35
Add semi-colon
U65535F Jan 29, 2025
cd922c6
Update parse_uri to support single tx
U65535F Jan 29, 2025
0fe95f9
fix parse_uri
U65535F Jan 29, 2025
a5750c5
fix errors
U65535F Jan 29, 2025
4a83b22
fix make_uri
U65535F Jan 30, 2025
6ca8ace
Slight imporvements and fix always failing test code.
U65535F Jan 30, 2025
dab6fe2
Fix tests that were using atomic units in URL. Add parameter consiste…
U65535F Jan 30, 2025
20e6095
Fix failing python functional tests
U65535F Jan 31, 2025
546fb29
fix improper size calc
U65535F Jan 31, 2025
993ebfc
Move overloads to wallet2.cpp. Introduce backwards compatibility in w…
U65535F Feb 1, 2025
577b7ee
Fix function declarations
U65535F Feb 1, 2025
faf7306
fix uri_data not found by replacing it with tools::wallet2::uri_data …
U65535F Feb 1, 2025
48fe7eb
remove override keyword from wallet.h: make_uri and parse_uri
U65535F Feb 1, 2025
9383f36
fix override issue
U65535F Feb 1, 2025
ca7b72e
add semi-colon.
U65535F Feb 2, 2025
a7fbf60
warning fix: loop variable 'entry' creates a copy.
U65535F Feb 2, 2025
df0021e
Merge branch 'master' of https://github.com/U65535F/monero
U65535F Feb 2, 2025
400d812
add const keyword
U65535F Feb 2, 2025
d678eee
allow standalone payment ids (yet not recommended; fallback)
U65535F Feb 2, 2025
bd357c5
Exclude core_tests
U65535F Feb 2, 2025
f0de5c8
fix failing uri.py by properly calling make_uri and parse_uri. restor…
U65535F Feb 2, 2025
c489d7d
introduce a custom convert to url func instead of modifying existing …
U65535F Feb 2, 2025
5911114
fix conver_to_url_format not declared by changing call to epee::net_u…
U65535F Feb 2, 2025
d9a56b8
Remove out of scope payment id check
U65535F Feb 10, 2025
cd59dfa
Introduce v2 versions of make_uri and parse_uri to ensure proper back…
U65535F Feb 11, 2025
6f27307
add new-lines
U65535F Feb 11, 2025
01d3a94
fix occurrences of uri funcs
U65535F Feb 11, 2025
5b1f082
remove v2 uri funcs as C++ differs them through function overloading …
U65535F Feb 11, 2025
0d7cb1a
fix make_uri_v2 and parse_uri_v2 calling make_uri and parse_uri in py…
U65535F Feb 11, 2025
3562a50
fix uri.py not providing payment_id to one of the tests
U65535F Feb 11, 2025
33ad3f0
remove extra new lines in wallet2_api.h
U65535F Feb 11, 2025
1ebba43
Merge branch 'monero-project:master' into master
U65535F Feb 12, 2025
71aa95e
only accept standard types as parameters
U65535F Feb 12, 2025
de28957
Merge branch 'master' of https://github.com/U65535F/monero
U65535F Feb 12, 2025
44b70c1
update wallet2
U65535F Feb 12, 2025
eeb3051
fix typos and overlooked references to old variables
U65535F Feb 12, 2025
bbcc043
fix typos
U65535F Feb 12, 2025
a647610
add const keyword
U65535F Feb 12, 2025
c1964b1
fix const keyword dropping error
U65535F Feb 12, 2025
707700d
remove override keyword
U65535F Feb 12, 2025
952486e
add const keyword to wallet2 interface
U65535F Feb 12, 2025
cc6deca
add const keyword
U65535F Feb 12, 2025
0d8698f
restore unnecessary edits and address sneedlewoods' comments
U65535F Feb 12, 2025
c6f16a6
remove some unnecessary edits and old references to removed variables…
U65535F Feb 12, 2025
1f14a22
fix silly typos
U65535F Feb 12, 2025
9158feb
Merge branch 'monero-project:master' into master
U65535F Feb 14, 2025
e4c0e63
Merge branch 'monero-project:master' into master
U65535F Feb 16, 2025
dedfcec
Merge branch 'monero-project:master' into master
U65535F Feb 17, 2025
b59fc59
Merge branch 'monero-project:master' into master
U65535F Feb 18, 2025
e74a7d4
Merge branch 'monero-project:master' into master
U65535F Feb 25, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix no member named errors
@U65535F
U65535F committed Jan 28, 2025
commit 168668a4019b5bec41b4dffc303d6edcfb252f2e
3 changes: 0 additions & 3 deletions src/wallet/wallet_rpc_server_commands_defs.h
View file
Original file line number Diff line number Diff line change
@@ -1833,9 +1833,7 @@ namespace wallet_rpc

BEGIN_KV_SERIALIZE_MAP()
KV_SERIALIZE(address)
KV_SERIALIZE(payment_id)
KV_SERIALIZE(amount)
KV_SERIALIZE(tx_description)
KV_SERIALIZE(recipient_name)
END_KV_SERIALIZE_MAP()
};
@@ -1853,7 +1851,6 @@ namespace wallet_rpc
END_KV_SERIALIZE_MAP()
};


struct COMMAND_RPC_MAKE_URI
Copy link
Collaborator

@selsta selsta Feb 9, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WALLET_RPC_VERSION_MINOR has to be bumped. Generally I'm unsure about this change since it seems incompatible. Wouldn't it make more sense to add a separate COMMAND_RPC_MAKE_MULTI_URI command instead of changing the existing one? I assume that would also reduce the diff in the tests.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. It would reduce the difference. So the original tests will work as usual. But if there is a need for multi-URI, then the user has to call the separate RPC commands which accept multiple recipient parameters for URI.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But if there is a need for multi-URI, then the user has to call the separate RPC commands which accept multiple recipient parameters for URI.

This is preferable over breaking all existing clients in my opinion.

{
struct request_t