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 configurable privacy statement link #1437

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

lampajr
Copy link
Member

@lampajr lampajr commented Mar 7, 2024

Fixes Issue

Fixes #1408

Changes proposed

Provide additional privacy statement in the About popup:

image

The link/URL is configurable at installation/build level using property horreum.privacy.

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.

@lampajr
Copy link
Member Author

lampajr commented Mar 7, 2024

There are still a couple of open points:

  • Set a reasonable default value, right now I am using /link/to/privacy/statement. Any suggestion?
  • Is Privacy category name fine on UI?

I kept the new VersionInfo.privacyStatement field optional from OpenAPI perspective in order to be backward compatible, the webapp should manage when the field is undefined.

@lampajr lampajr force-pushed the gh1408_privacy_stmt branch from ae39774 to ca19d38 Compare March 7, 2024 10:05
@lampajr lampajr requested a review from johnaohara March 7, 2024 10:07
Copy link
Member

@johnaohara johnaohara left a comment

Choose a reason for hiding this comment

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

Please can you rebase

@lampajr lampajr requested review from stalep and johnaohara March 7, 2024 10:08
@lampajr lampajr force-pushed the gh1408_privacy_stmt branch from ca19d38 to 691f507 Compare March 7, 2024 10:10
@johnaohara
Copy link
Member

There are still a couple of open points:

* [ ]  Set a reasonable default value, right now I am using `/link/to/privacy/statement`. Any suggestion?

I am not sure if there is a reasonable default, it is probably best to leave this either commented out in application.properties and not display anything by default. Users can see that it is an option that can be set and/or we need an admin guide that covers the setting

* [ ]  Is `Privacy` category name fine on UI?

I kept the new VersionInfo.privacyStatement field optional from OpenAPI perspective in order to be backward compatible, the webapp should manage when the field is undefined.

I think it is fine to be a configuration option for Horreum as it should apply system wide to all users of the installation

@johnaohara johnaohara merged commit e9cc2c8 into Hyperfoil:master Mar 7, 2024
3 checks passed
@johnaohara
Copy link
Member

merged, thank you for your contribution!

@lampajr
Copy link
Member Author

lampajr commented Mar 7, 2024

@johnaohara do we want to address #1437 (comment) in a separate PR? because otherwise by default the webpage will show /link/to/privacy/statement as privacy statement which does not make sense

@lampajr lampajr deleted the gh1408_privacy_stmt branch March 7, 2024 10:26
@johnaohara
Copy link
Member

@johnaohara do we want to address #1437 (comment) in a separate PR? because otherwise by default the webpage will show /link/to/privacy/statement as privacy statement which does not make sense

yes please, i have already merged this PR

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

Successfully merging this pull request may close these issues.

Add functionality to link to privacy statement in the "About" popup
2 participants