I'd like work on this @Pepe_piton

@Nat_WDU @Pepe_piton I'd like to work on this task.

@Nat_WDU @Pepe_piton I'm done with this task.

This is the PR link: https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/11 @Pepe_piton @Nat_WDU

@Pepe_piton @Nat_WDU I'd really love to get a feedback on this PR: https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/11

Thank you

hi @Pepe_piton @Nat_WDU
i made a merge request related this issue:-https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/66
changes made:-

• Added a Wikidata Query Service (WQS) link to each author's works list.
• Users can view the SPARQL query and reuse it.
• Tested locally — link works and page renders correctly.

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

Hi @Nat_WDU @Pepe_piton

I worked on this issue.
Link to MR: https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/71

Hello @Sopzy, @Oluwatumininu.m and @Dipanshu1223 , the three of you managed to make a good working implementation of this, congrats!

@Dipanshu1223 I really like that your implementation is so simple! However, the query is not formatted, so the user ends up seeing a query all in one line, which is hard to read

This task is still open if you want to submit improvements for your MRs, and for new applicants to submit their own. Thankseveryone!

In T397554#11269589, @Nat_WDU wrote:

Hello @Sopzy, @Oluwatumininu.m and @Dipanshu1223 , the three of you managed to make a good working implementation of this, congrats!

@Dipanshu1223 I really like that your implementation is so simple! However, the query is not formatted, so the user ends up seeing a query all in one line, which is hard to read

Also, I think its better for the links’ wording to read “View this query at the Wikidata Query Service” since WQS is not a familiar term for all Paulina’s users

This task is still open if you want to submit improvements for your MRs, and for new applicants to submit their own. Thankseveryone!

Thank you for the feedback @Nat_WDU

I have made the changes and updated my MR. https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/71/commits

hi @Nat_WDU
thanks for feedback
i made some changes related this issue:-https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/66
changes made:-
add text:- View this query at the Wikidata Query Service
the query is formatted now
image
{F66748932}

Hello @Nat_WDU
I made a merge request concerning this issue: https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/81
Please kindly review, thank you!

Hello @System625 ! I don't think it's a good idea to import requests in the middle of the definition. Let me know if you have any specific reasons for doing so

My bad @Nat_WDU ,
I have fixed it! I've moved the requests import to the top of the file. There was no specific reason for the inline import, I just didn't notice it before pushing. Thanks for the review!

Good job, @System625 . I appreciate that you are using the language selected by the user for the query, and that you've updated i18n files.

@Dipanshu1223 just realized you are hardcoding "en" when we could pass the user's languade to the query. Good job reformatting the query!

Good job, @System625 . I appreciate that you are using the language selected by the user for the query, and that you've updated i18n files.

Thank you very much for the feedback, it means a lot :)

hey @Nat_WDU ,
thanks for feedback
i forgot test other language
i made some changes related this issue::-https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/66/diffs
Changes made:

• Added support for all interface languages (replaced hardcoded “en” with the user’s selected language).

Please kindly review, thank you!

In T397554#11269589, @Nat_WDU wrote:

Hello @Sopzy, @Oluwatumininu.m and @Dipanshu1223 , the three of you managed to make a good working implementation of this, congrats!

@Dipanshu1223 I really like that your implementation is so simple! However, the query is not formatted, so the user ends up seeing a query all in one line, which is hard to read

Also, I think its better for the links’ wording to read “View this query at the Wikidata Query Service” since WQS is not a familiar term for all Paulina’s users

This task is still open if you want to submit improvements for your MRs, and for new applicants to submit their own. Thankseveryone!

@Nat_WDU Thank you for this feedback.

I'll update the hyperlink text to show "View this query at Wikidata Query Service"

Hi @Pepe_piton, @Nat_WDU,

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

Thank you

In T397554#11288085, @Sopzy wrote:

In T397554#11269589, @Nat_WDU wrote:

Hello @Sopzy, @Oluwatumininu.m and @Dipanshu1223 , the three of you managed to make a good working implementation of this, congrats!

@Dipanshu1223 I really like that your implementation is so simple! However, the query is not formatted, so the user ends up seeing a query all in one line, which is hard to read

Also, I think its better for the links’ wording to read “View this query at the Wikidata Query Service” since WQS is not a familiar term for all Paulina’s users

This task is still open if you want to submit improvements for your MRs, and for new applicants to submit their own. Thankseveryone!

@Nat_WDU Thank you for this feedback.

I'll update the hyperlink text to show "View this query at Wikidata Query Service"

I made the change @Nat_WDU

Thanks!

https://gitlab.wikimedia.org/toolforge-repos/paulina/-/merge_requests/11/diffs?commit_id=b9e8a52b9a4f46f3ed8cefcb3930729cd13dceb1