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

Added Server operations to License Generator #696

Merged
merged 17 commits into from
Jul 6, 2023
Merged

Conversation

robertsonbrinker
Copy link
Contributor

@robertsonbrinker robertsonbrinker commented Jul 3, 2023

All these changes should be able to work with old licenses too.

Along with testing the LicenseGenerator, making sure the Registration works would be good.

I noticed that after successfully registering there are a few seconds on the ProjectPicker where the user is told they don't own paratext. This is not unique to this branch, the same thing happens for me on main as well.

The validation isn't perfect. I changed the EmailValidator to not accept # because that symbol was acting weird in the database.

I got rid of the LicenseKey property in the User class because it doesn't make sense to be there. How can an object have a property where the contents of that property is an encrypted version of that object? It can't. Instead LicenseKey was moved to the new DashboardUser class. Any places where User was switched to DashboardUser is where LicenseKey was being used.

image

@robertsonbrinker robertsonbrinker requested a review from GeoDirk July 3, 2023 14:51
@GeoDirk GeoDirk merged commit eaf1df0 into main Jul 6, 2023
@GeoDirk GeoDirk deleted the Sending-License-to-Cloud branch July 6, 2023 15:29
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.

2 participants