Skip to content

Pin, Duress Pin, and Custom Messeges#1

Open
micr0-dev wants to merge 4 commits intomainfrom
feature/pin
Open

Pin, Duress Pin, and Custom Messeges#1
micr0-dev wants to merge 4 commits intomainfrom
feature/pin

Conversation

@micr0-dev
Copy link
Owner

Adds those features

Copy link

@andrwq andrwq left a comment

Choose a reason for hiding this comment

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

form validation and sanitation suggested

Copy link
Contributor

@darltrash darltrash left a comment

Choose a reason for hiding this comment

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

This is great, and most of my concerns were already covered by the other reviewer, but I feel like some changes can be done here to improve security and mantainability.

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