Hi @Pepe_piton

I have fixed this issue.
Before:

https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/14/diffs?commit_id=969604e8c1a2a14f14b3057bce9c1ab1bd15f11b

HI @Nat_WDU @Pepe_piton

Could you please review this change? https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/21

This was the before:

The update now:

hello, @Pepe_piton
i made a merge request related this issue:-https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/33/diffs?commit_id=76eab43e749393767ccf01a582f37b7fa4439365
Changes made:

• Added JavaScript check to hide rows with empty values dynamically.
• Ensured template fields such as Nationality, Ethnicity, Occupation, and Date of Death are displayed only when data is available.
• Tested locally — the layout now renders cleanly without blank rows.

I'd appreciate it if you could take a look at my work and provide feedback. I want to make sure I'm meeting expectations and understand any areas for improvemen

https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/36

Implemented the changes in the above PR
For displaying values of various fields, I employed a list of dicts (with field label and value) within the jinja template, iterating over them in a for loop. Skipping the ones where the value attribute of the dict is empty.

Kindly review. Thanks!

@Oluwatumininu.m, Thanks for your contribution. You catched the essence of the task: adding if statements with Jinja2 in the author.html page. Simple and readable. Congrats!

@Rolalove, You also did a good job, with a solution similar to Oluwatumininu.m, which is also the first solution I had in mind. You also made some code style improvements to the file, which are very nice, but keep in mind that sometimes if you include the code style improvements in the same PR as the feature, it can be a little tricky to review the feature.

@Dipanshu1223, You really surprised me with your Javascript approach! It works very well. Using the row classes, you managed to hide the empty fields.

@shreya-bhagat, Nice! You went a step further and leveraged Jinja2's features to create a loop for the biography table. The peculiarities of some fields (date of birth and date of death) added some complexity to the loop, but you managed to make it work well. This solution is also very nice because it reduces the number of lines.

Thanks again for your work. With different and clever approaches, you all solved the task.

In T406909#11266034, @Pepe_piton wrote:

@Oluwatumininu.m, Thanks for your contribution. You catched the essence of the task: adding if statements with Jinja2 in the author.html page. Simple and readable. Congrats!

Thank you so very much for the feedback @Pepe_piton

In T406909#11266034, @Pepe_piton wrote:

@Oluwatumininu.m, Thanks for your contribution. You catched the essence of the task: adding if statements with Jinja2 in the author.html page. Simple and readable. Congrats!

@Rolalove, You also did a good job, with a solution similar to Oluwatumininu.m, which is also the first solution I had in mind. You also made some code style improvements to the file, which are very nice, but keep in mind that sometimes if you include the code style improvements in the same PR as the feature, it can be a little tricky to review the feature.

@Dipanshu1223, You really surprised me with your Javascript approach! It works very well. Using the row classes, you managed to hide the empty fields.

@shreya-bhagat, Nice! You went a step further and leveraged Jinja2's features to create a loop for the biography table. The peculiarities of some fields (date of birth and date of death) added some complexity to the loop, but you managed to make it work well. This solution is also very nice because it reduces the number of lines.

Thanks again for your work. With different and clever approaches, you all solved the task.

Thank you so much for the detailed feedback @Pepe_piton .

Good afternoon,

I’m a little late to the party but since I’ve completed this microtask, I would very much appreciate comments and feedback from you @Nat_WDU and @Pepe_piton .

Please find attached my Merge Request.

Hello @Pepe_piton,

I would really love your feedback and comments on my solution.

I must say that I am delighted to finally be able to submit a merge request before the microtask is resolved. It is also a refresher for me as I have been away from Flask for a while.

Here is my Merge Request. Thank you.

Before
{F66751223}

After
{F66751226}

Hi @Nat_WDU @Pepe_piton ,

i have completed this task and submitted merge request : https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/135
Kindly review!

Thank you

@Miiswom @Chumzine @Aradhya05 Good work!

Whether by identifying each field and putting an if statement in each one, as Miiswom and Chumzine did, or by building a loop, as Aradhya05 did, you achieved the expected result.

I merged @Chumzine solution. Thanks to everyone!