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

Jump to bottom

fix: Transaction Memo hasn't inputMask #10746

Closed
aligh98 opened this issue Jan 6, 2023 · 1 comment · Fixed by #12980
Closed

fix: Transaction Memo hasn't inputMask #10746

aligh98 opened this issue Jan 6, 2023 · 1 comment · Fixed by #12980
Assignees
@shadab-taiko
Labels
area.bridge category.bug

Comments

@aligh98
Copy link

aligh98 commented Jan 6, 2023 •
edited

Describe the bug

I noticed that no restrictions are applied to the Memo field and the user can record any amount of string he wants in it and perform the transaction with it.
The link below is an example of my transaction with a long text.
Tx on l1

Steps to reproduce

Memo input box.

Additional context

It is better to put a limit.

taiko 5
@aligh98 aligh98 closed this as not planned Won't fix, can't repro, duplicate, stale Jan 6, 2023
@aligh98
Copy link
Author

aligh98 commented Jan 6, 2023

UTF-8 transaction preview:
image

@aligh98 aligh98 reopened this Jan 6, 2023
@shadab-taiko shadab-taiko linked a pull request Jan 18, 2023 that will close this issue
feat(bridge-ui): add memo max length check #12980
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shadab-taiko shadab-taiko

Labels
area.bridge category.bug
Projects
Taiko Project Board
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(bridge-ui): add memo max length check taikoxyz/taiko-mono
2 participants
@aligh98 @shadab-taiko