Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wot command refactoring #66

Merged
merged 2 commits into from
Jul 20, 2017
Merged

Wot command refactoring #66

merged 2 commits into from
Jul 20, 2017

Conversation

M5oul
Copy link
Member

@M5oul M5oul commented Jul 10, 2017

News

  • display charts for identities with same uid
  • display the pubkey and their blockstamps
  • count number of certifications is now done with len() method

Example

It is useful for identities published many times:

silkaj wot roodinux

roodinux (VxCoq…) from block #32606-0000077F8…
received 2 and sent 0 certifications:
|  received  |  sent  |
|------------+--------|
|  pafzedog  |        |
|    moul    |        |

roodinux (VxCoq…) from block #32777-00000CF74…
received 6 and sent 0 certifications:
|  received  |  sent  |
|------------+--------|
|    moul    |        |
|   RavanH   |        |
|  pafzedog  |        |
|  juandre   |        |
| PierreYves |        |
|   RavanH   |        |

Tests, reviews

  • test if it doesn't crash with other identities
  • take a look at the code

- display charts for identities with same uid
- display their blockstamps
- count number of certifications is now done with len()
@M5oul M5oul added this to the 0.4.0 milestone Jul 10, 2017
@M5oul M5oul requested a review from a team July 10, 2017 19:26
@M5oul M5oul requested review from jytou and Tortue95 July 11, 2017 10:40
@M5oul M5oul merged commit 5529921 into master Jul 20, 2017
@M5oul M5oul deleted the wot_refactor branch July 20, 2017 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant