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

Serialize lists to comma delimited string in PaymentAccount.toJson() instead of PaymentAccount.toForm() [0.2 XMR] #1549

Open
woodser opened this issue Jan 17, 2025 · 3 comments · May be fixed by #1553
Labels
💰bounty There is a bounty on this issue

Comments

@woodser
Copy link
Contributor

woodser commented Jan 17, 2025

This issue requests serializing lists to a comma delimited string in PaymentAccount.toJson() instead of PaymentAccount.toForm().

The implementation should be in PaymentAccountTypeAdapter?

@woodser woodser changed the title Serialize lists to comma delimited string in PaymentAccount.toJson() instead of PaymentAccount.toForm() Serialize lists to comma delimited string in PaymentAccount.toJson() instead of PaymentAccount.toForm() [0.2 XMR] Jan 17, 2025
@woodser woodser added the 💰bounty There is a bounty on this issue label Jan 17, 2025
Copy link

There is a bounty on this issue. The amount is in the title. The reward will be awarded to the first person or group of people whose solution is accepted and merged.

In some cases, we may assign the issue to specific contributors. We expect contributors to provide a PR in a reasonable time frame or, in case of an extensive work, updates on their progress. We will unassign the issue if we feel the assignee is not responsive or has abandoned the task.

Read the full conditions and details of our bounty system.

@abhishek818
Copy link

Will take an attempt at this.

@niyid
Copy link
Contributor

niyid commented Jan 17, 2025

This requirement seems to be unclear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💰bounty There is a bounty on this issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants