Some comments. Also, I committed #3077598: Add a service to get user informat...

Contribution Date
Technology
Contribution Project
Contribution Details

Some comments. Also, I committed #3077598: Add a service to get user information from d.o with a different method name, so that needs to be changed as well.

  1. +++ b/src/DOComputedFields.php @@ -0,0 +1,51 @@ + switch ($field_type) {

    Let's add at least these fields:
    - Bio
    - IRC Nick
    - Websites
    - Country
    - Languages
    - First Name
    - Last Name
    - URL
    - Created

    We can even do some of these in other issues. For example, "Industries" requires another API request, so it might make sense to do that in a different issue.

  2. +++ b/src/DOComputedFields.php @@ -0,0 +1,51 @@ + return $userInformation->field_industries_worked_in; + + break;

    `break` is not required here. Nor in the next `case`.

Issue Status
Needs work
Contribution Author
Files count
0
Patches count
0