Skip to content
Snippets Groups Projects

Add 'wot' subcmd to display received and sent certifications

Merged Moul requested to merge wot_display_certs into master
silkaj wot moul
Requested default node: <duniter.org:10901>
moul received 18 and sent 21 certifications:
|    received    |       sent       |
|----------------+------------------|
|   gpsqueeek    |   mathieuBize    |
|  jeanferreira  |      Galuel      |
|    gerard94    |   jeanferreira   |
| BenoitLavenier |     gerard94     |
|      Loda      |      smyds       |
|     smyds      | CaizohanFerreira |
|    fbuland     |    gpsqueeek     |
|     SebasC     |       inso       |
|     Thatoo     |       vit        |
|   cuckooland   |      Thatoo      |
|      inso      |     William      |
|    Paulart     |    cuckooland    |
|   vincentux    |     greyzlii     |
|     Alfybe     |      elois       |
|    urodelus    |      cgeek       |
|    greyzlii    |    PierreYves    |
|   Mententon    |     mmu_man      |
|   PierreYves   |   OlivierAuber   |
|     paidge     |      paidge      |
|                |      Alfybe      |
|                |     1000i100     |
|                |       dig        |

I tested with cgeek, inso, Galuel, jytou and Tortue ids. I had to fix:

  • the fact that cgeek gave two results.
  • the fact that the check is done with case unsensitive as Galuel id is written with uppercase.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Moul
  • Moul
  • Moul
  • Moul
  • Moul
  • Moul
  • Moul
  • Author Owner

    I tested with future members IDs and it seem there is there is issues with identities which have where there is many identities. For instance, with dig, I could found two id. The program may found the one with most certifications or display all of them.

    (So, for dig, it's better that the break it's not here).

  • Author Owner

    Furthermore, AlexisLecointe have two certifications from cuckooland. The program should choose the good one.

  • Author Owner

    I merge for now. Bugs will be discovered/reported later and then could be fixed.

  • Please register or sign in to reply
    Loading