Skip to content

LT-22312: Make copyright more accessible in Webonary#611

Merged
aror92 merged 3 commits intorelease/9.3from
LT-22312
Jan 7, 2026
Merged

LT-22312: Make copyright more accessible in Webonary#611
aror92 merged 3 commits intorelease/9.3from
LT-22312

Conversation

@aror92
Copy link
Contributor

@aror92 aror92 commented Jan 7, 2026

Create GeneratePictureLicenseContent method in LcmGenerator that returns a string containing picture copyright and license info.

In GenerateContentForPicture, use this new method to generate the license content and add license as an argument to AddImage.

In "AddImage" in the Xhtml generator, add the license as a title in the image element, so that license will display when mousing over the image.

In "AddImage" in the JSON generator, add a "copyrightLicense" property to the image data so it will be searchable in the database.


This change is Reviewable

aror92 added 2 commits January 7, 2026 13:51
Create GeneratePictureLicenseContent method in LcmGenerator that
returns a string containing picture copyright and license info.

In GenerateContentForPicture, use this new method to generate the
license content and add license as an argument to AddImage.

In "AddImage" in the Xhtml generator, add the license as a title
in the image element, so that license will display when mousing
over the image.

In "AddImage" in the JSON generator, add a "copyrightLicense"
property to the image data so it will be searchable
in the database.

Change-Id: Ice63fbd238e41ec94dc1ebe8178ec727919ec481
Change-Id: Ia5bbd365d1cd7240e8544a916db415716f6b161b
@aror92
Copy link
Contributor Author

aror92 commented Jan 7, 2026

Two JSON generator tests are failing because the expected results need to be updated to include the copyrightLicense property. I'm updating those now.

Change-Id: I3c41e011a89b4a6e4fcd81df6cbb73b0a266ad45
@aror92 aror92 enabled auto-merge (squash) January 7, 2026 19:38
Copy link
Contributor

@jasonleenaylor jasonleenaylor left a comment

Choose a reason for hiding this comment

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

Very nice work!
:lgtm:

@jasonleenaylor partially reviewed 6 files and made 1 comment.
Reviewable status: 5 of 6 files reviewed, all discussions resolved (waiting on @aror92).

@aror92 aror92 merged commit 09de71c into release/9.3 Jan 7, 2026
4 of 5 checks passed
@aror92 aror92 deleted the LT-22312 branch January 7, 2026 21:44
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