Skip to content

simplify opening privacy policy link #97

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rogin
Copy link
Contributor

@rogin rogin commented May 21, 2025

Fixes issue #86

Compiled using Oracle JDK 8. Link click worked on Windows 10.

I see several classes using Desktop.isDesktopSupported() which may warrant a code cleanup ticket.

@rogin rogin force-pushed the fix-policy-link-issue branch from 4717c5f to 65132fc Compare July 14, 2025 21:53
@rogin
Copy link
Contributor Author

rogin commented Jul 14, 2025

New test step requires flipping BrandingConstants.CENTRAL_USER_REGISTRATION to true for the privacy link to display. Confirmed clicking the privacy link on my Windows 10 machine opens my default browser (chrome) and took me to https://github.com/openintegrationengine

@kpalang kpalang requested review from a team, kayyagari, gibson9583, kpalang and jonbartels and removed request for a team, kayyagari, gibson9583, kpalang and jonbartels August 1, 2025 12:07
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.

1 participant