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 support for old safe send link format #122

Merged
merged 4 commits into from
Nov 19, 2023

Conversation

mariofriz
Copy link
Contributor

@mariofriz mariofriz commented Dec 2, 2022

This PR fixes #24

Changes:

  • Add a route watcher, that transforms old format to new format

Tasks:

  • Add support for address + amount
  • Add support for address only

@mariofriz mariofriz marked this pull request as ready for review December 2, 2022 12:52
@sisou
Copy link
Member

sisou commented Mar 14, 2023

I don't know how this happened, but this PR now includes unrelated Portuguese translations. Can you maybe rebase this on top of master with only the relevant commits?

@mariofriz
Copy link
Contributor Author

mariofriz commented Mar 19, 2023

I don't know how this happened, but this PR now includes unrelated Portuguese translations. Can you maybe rebase this on top of master with only the relevant commits?

Indeed something weird happened I had a strange error while rebasing, I ended up skipping the problematic commit. I rebased onto master now and also changed to named regex groups so it's a bit easier to read.

@mariofriz
Copy link
Contributor Author

@sisou Hi, I know you are busy with Nimiq 2.0 but if you get a quick chance to review this PR that would be great! If changes are needed, please let me know.

@sisou sisou merged commit 305f6e4 into nimiq:master Nov 19, 2023
1 check passed
@sisou
Copy link
Member

sisou commented Nov 19, 2023

Thank you very much @mariofriz! 👍

@mariofriz mariofriz deleted the feat/add-safe-link-support branch December 23, 2023 13:59
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.

Support old requestLink generated from the donation app / safe
2 participants