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 Egypt TIN #334

Closed
wants to merge 8 commits into from
Closed

Add support for Egypt TIN #334

wants to merge 8 commits into from

Conversation

unho
Copy link
Contributor

@unho unho commented Oct 9, 2022

Closes #225.

@unho unho force-pushed the eg-tin branch 6 times, most recently from 9ebb6b1 to 4bb5d70 Compare October 12, 2022 09:41
@arthurdejong
Copy link
Owner

Sorry it took so long but I merged this with a small fix to the compact() function for Python 2.7 in 6d366e3. It is not ideal because there is no clear description of the number I could find (but that is probably also due to my not being able to grasp Arabic) and the lack of validation of a checksum or check digit but the presence of the conversion still is useful.

@unho
Copy link
Contributor Author

unho commented Jan 2, 2023

For this particular PR there was something that didn't work with the CI that I was unable to figure out. Thank you very much for fixing that glitch and merging this.

@unho unho deleted the eg-tin branch January 9, 2023 13:08
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.

Add support for Egypt TIN
2 participants