In T406950#11269551, @Pepe_piton wrote:

Excellent @Kimondorose! Your solution works well and you understood the logic. Anyway, you might be interested in reading the comment I left for @Lebogang , since there are slightly simpler conditional solutions for adding the vertical bar separator in the template.

In T406950#11269488, @Pepe_piton wrote:

@Oluwatumininu.m Thanks for your contribution! You managed to place the Wikidata link very well. However, with your current solution, the Wikimedia Commons link isn't displayed. This is because the work.html template doesn't receive a variable named "work_commons_link." For a clue on how to resolve this issue, it might be helpful to revisit the task of adding the "inception" field to the work page: T406949

@Nurah_Wakili The same thing happens with your solution as with that of @Oluwatumininu.m. For the Wikimedia Commons link to work, some changes need to be made to some other files.

@Lebogang Good job. You've managed to grasp the logic of the task, finding a way to add the link to Wikimedia Commons. It works well! As a side note, it's possible to simplify the if statements in the work.html template to correctly display the vertical bar symbol ("|"), given that the Wikidata link is present in 100% of works (if there's no Wikidata link, the item doesn't exist). You can see a simpler way of building the wikilink if statements in the author.html template.
By the way, some changes from the terms task were included in the MR diff, but it was easy to identify the changes corresponding to this task, so don't worry.

@Kimondorose The merge request link didn't appear in your message. Can you share it again?

@Pepe_piton Here is a link to my Merge Request for this task: https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/68
Key Changes
pdclasses.py:

• Implemented logic in the Work.init method to safely extract the Wikimedia Commons sitelink URL from the raw Wikidata item data (sitelinks['commonswiki']).

@Pepe_piton @Nat_WDU I have implemented pagination plus added extra functionalities surrounding the feature. Here is a summary:
Changes Made:

• Added page-based navigation to results.
• Users can now view results page by page.
• The template displays Page X of Y at both the top and bottom of the results page.
• If one wants to go to a specific page, they can type it in the browser's search bar.
• Prevents "Page not found" errors when navigating beyond available results.
• Updated Warning Message for Large Result Sets: Your search returned too many results. All results are not being shown. Please refine your search.

Hello @Pepe_piton @Nat_WDU
I have implemented the required changes as described below and here is the link to my Merge Request: https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/60:

• Dynamic Visibility: The "Add information" button is only displayed if the author is missing one or more biographical fields (e.g., 'Date of death', 'Nationality').
• Dropdown Population: Upon clicking the button, it is replaced by a dropdown menu that only lists the fields currently missing from the biography table.
• Dynamic Input Type: When a user selects a field from the dropdown, a text input box is displayed next to it.
• For fields requiring structured data (like 'Date of birth' or 'Date of death'), the input type is dynamically set to type="date".
• For all other fields, the input remains a standard type="text".
• (No Missing Fields): When there are no missing fields, the "Add information" button is completely hidden.

Hello @Pepe_piton @Nat_WDU I have shared a merge request on this task. As per the requirements, there is a pencil icon next to the author's nationality. When a user clicks on the pencil icon, they can edit the field. They can then edit the nationality and when they finish, click the check icon to publish the information. An added feature is the ability to cancel their edit if they no longer want to edit it. Here is the link to the request: https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/57