Just like in the Drupal 7 version, the following data need to be displayed on the user profile :

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

fmb created an issue. See original summary.

_m’s picture

Assigned: Unassigned » _m
fmb’s picture

Issue tags: +Vienna2025
_m’s picture

I've made progress on the first and third item. However, the "Profile on primary site" link is something that feels like it is configured only on the production site (i.e. not custom code).

I will submit my contribution asap. I still need to format it as requested.

fmb’s picture

@_m how is it going?

dydave’s picture

Assigned: _m » dydave

OK, I've started checking the project's documentation and will take a closer look at this issue.

Thanks for all the info @fmb!

fmb’s picture

profile on primary site

Interestingly enough, this is not related to SSO. We can just retrieve the user ID on drupal.org that way with a GET query on https://www.drupal.org/jsonapi/user/user?filter[name]=USERNAME, and then extract the drupal_internal__uid attribute (USERNAME can be "dydave" or "fmb" for instance). We would then be able to build this kind of URLs: https://www.drupal.org/user/619092.

dydave’s picture

Assigned: dydave » Unassigned
Status: Active » Needs review

OK Felip (@fmb), I've pushed all the changes discussed together for this issue, in particular for the two requirements:
1 - list of translation teams (formerly "group audience"),
3 - Localization contributions: number of approved translations for each translation team.

The merge request: https://git.drupalcode.org/project/l10n_server/-/merge_requests/112
needs to be merged first and add a Views relationship plugin which is used in the other merge request:
https://gitlab.com/drupal-infrastructure/sites/localize/-/merge_requests/53

Which also contains multiple other changes: Upgrade of the code base, installation of additional modules, uninstalled unused modules, etc...

If possible, it would be great if these changes could be merged first, before fixing the second requirement with:
2 - profile on primary site (might be related to #3274544: Review integration of l10n_drupal_rest with drupalorg_crosssite),

We've started working on this last point, with a colleague, as mentioned by email, but it's most likely going to take a little bit more time...
Could probably land next week.

Let me know if you spot anything when testing the merge request, I would surely be glad to make additional changes.

Thanks in advance!

  • fmb committed 7fbd895c on 3.0.x
    Merge branch '3487803-display-localize-related-data' into '3.0.x'
    
    Issue...

  • dydave committed 4a2eb004 on 3.0.x
    Issue #3487803 by dydave: Added Views relationship plugin to relate a...
fmb’s picture

Status: Needs review » Needs work

Thank you so much @dydave for these nice and well-documented MRs! All tested and merged. Now, could you please add the new view in l10n_community/config/install (or l10n_community/config/optional, whichever is the relevant option here)?

dydave’s picture

Status: Needs work » Needs review

Sorry for the delay on this once again Felip (@fmb), but we took a bit more time to do a bit more testing and code clean-up before being able to push these changes.

The comments raised at #12 have been addressed in MR !117, with the missing views added, plus the last requirement for this ticket, with the User primary profile link block.

These changes should be merged before merging the changes containing the configuration changes to display the block from merge request:
https://gitlab.com/drupal-infrastructure/sites/localize/-/merge_requests/57

A few small additional changes were added:
Added PHPMyAdmin ddev plugin for local development purpose, since the project is using MariaDB anyway.

We noticed a block was wrongly displayed on the project releases page, so we updated the visibility settings of the language progress block, thanks to the contrib module block_exclude_pages.
 

At this stage, the user profile page should have all the features it currently has on the D7 site.

We would greatly appreciate if you could please try testing and reviewing this new round of changes and let us know if you spot anything else.
Feel free to let us know if you have any questions, comments or suggestions on any aspects of these merge requests or this ticket in general, we would surely try answering as soon as possible.
Thanks in advance!

donquixote made their first commit to this issue’s fork.

  • fmb committed bb91c6ef on 3.0.x
    Merge branch '3487803-display-localize-related-data' into '3.0.x'
    
    Issue...

  • fmb committed cd6935a8 on 3.0.x authored by donquixote
    Issue #3487803: Add 'l10n_community' in ModuleViewsIntegrityTest.
    

  • fmb committed c3cc6176 on 3.0.x authored by dydave
    Issue #3487803 by dydave: Added 'User Primary Profile' block to display...
fmb’s picture

Status: Needs review » Fixed

Thank you!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

dydave’s picture

Thank you everyone for the great help on this issue! 🙏
Glad we could get this one landed 😊

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.