Skip to content

Issue 6 first login required fields #68

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

Conversation

jonbartels
Copy link
Contributor

Resolves #6

First login and user edit only require username and password

user-edit first-login

@jonbartels jonbartels force-pushed the issue-6-first-login-required-fields branch 3 times, most recently from 4eb8c66 to 2691365 Compare April 30, 2025 18:59
Copy link
Contributor

@tonygermano tonygermano left a comment

Choose a reason for hiding this comment

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

I'd like to take a look at rolling back this class to what it looked like in 4.0 before they started making the changes that required all of the additional fields. There may be a few things we need to keep, because I think a few extra fields were added, but I know a lot of GUI elements were added or hidden which we don't need or want. We likely don't need the changes that pre-populate the form with values saved in the local java user profile if we aren't requiring the fields, anymore. I think comparing what is there currently to a known desired state is probably easier than trying to slowly undo things piece by piece and likely overlooking something.

My apologies for not including this in the detail of the issue.

Copy link
Contributor

@tonygermano tonygermano left a comment

Choose a reason for hiding this comment

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

I'll open a new issue for the rollback.

@jonbartels jonbartels force-pushed the issue-6-first-login-required-fields branch from 2691365 to 1bd8e95 Compare May 2, 2025 18:39
@jonbartels jonbartels merged commit 1bd8e95 into OpenIntegrationEngine:main May 2, 2025
2 checks passed
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.

Only username and password should be required fields for first time login
4 participants