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

Code Refactor Update #14

Closed
1 of 4 tasks
patrick-rodgers opened this issue Apr 20, 2020 · 4 comments
Closed
1 of 4 tasks

Code Refactor Update #14

patrick-rodgers opened this issue Apr 20, 2020 · 4 comments
Labels
enhancement New feature or request

Comments

@patrick-rodgers
Copy link
Member

patrick-rodgers commented Apr 20, 2020

Category

  • Enhancement
  • Bug
  • Question
  • Documentation gap/issue

Version

Please specify what version of IdFix you are using: [ latest ]

We are seeing an uptick in usage of IdFix, which is great and we are excited for all the feedback. In reviewing the issues and looking over the code we decided to take the time to do a larger refactor to make it easier to maintain, add comments, and decouple the logic which is currently contained almost exclusively within in "Form1.cs".

This means that fixing the open issues and publishing a new release will take a bit longer than we planned, but future updates should be quicker.

We have no plans to make any changes to the user experience of the tool and do not plan any UI changes with the code refactor. When done we aim to deliver the same tool with hopefully some small performance improvements and fixes to the open issues.

How you can help

If you are interested in helping test beta versions of the refactored code to compare to the existing tool we would appreciate it. While we do our best to test things, nothing can duplicate the nature of different production environments across the world. More testing is always better. If you want to help please comment below and once we are ready we'll update with how to test beta versions. Thanks!

@daesiku
Copy link

daesiku commented Apr 25, 2020

Interested in testing tld and X400 fix.

@pergranath
Copy link

Also interested in testing X400 whitespace error, and all TLD to work as topleveldomain.

@patrick-rodgers
Copy link
Member Author

@disiku and @pergranath - can you drop me an email so I can share a link to try a beta?

@patrick-rodgers
Copy link
Member Author

Have had a few folks test things out and the refactored code is looking good. We have tested as much as we can within the constraints of our dev environment. We will have a new version published soon and will address any new issues ASAP. Thank you all for the help and interest in the project!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants