feat: add validator for trx addresses by msamsami · Pull Request #384 · python-validators/validators

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@msamsami

Copy link Copy Markdown

Contributor

@msamsami msamsami commented

Jul 2, 2024

  • A new validator is added for the Tron (TRC20) address

@msamsami

Copy link Copy Markdown

Contributor Author

msamsami commented

Jul 3, 2024

@yozachar All done.

Copy link Copy Markdown

Collaborator

@nandgator nandgator left a comment

Choose a reason for hiding this comment

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

LGTM!

@nandgator nandgator merged commit 860ca46 into python-validators:master

Jul 4, 2024

@nandgator nandgator self-assigned this

Jul 4, 2024

@nandgator nandgator added the enhancement

Issue/PR: A new feature

label

Jul 4, 2024

@nandgator

Copy link Copy Markdown

Collaborator

@msamsami, there's is something odd about the Table of Contents in the crypto_address documentation see: https://yozachar.github.io/pyvalidators/stable/api/crypto_addresses/, any idea why?

@msamsami

Copy link Copy Markdown

Contributor Author

msamsami commented

Jul 4, 2024

@msamsami, there's is something odd about the Table of Contents in the crypto_address documentation see: https://yozachar.github.io/pyvalidators/stable/api/crypto_addresses/, any idea why?

The table of contents in the right sidebar? I noticed it just now. I ran the python package/export TYPE commands once again. No changes to the files! And it's the same on local—no idea why!

nandgator reacted with confused emoji

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nandgator nandgator nandgator approved these changes

Assignees

@nandgator nandgator

Labels

enhancement

Issue/PR: A new feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@msamsami @nandgator