Opened 3 years ago

Closed 3 years ago

#24375 closed enhancement (fixed)

Make (effective_family) url that points to list of all family members

Reported by: cypherpunks Owned by: irl
Priority: Medium Milestone:
Component: Metrics/Relay Search Version:
Severity: Normal Keywords:
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Thank you for implementing #21368
https://lists.torproject.org/pipermail/tor-commits/2017-November/134774.html

what do you think about making that new number a URL to the list of eff. family members: (untested)

@@ -80,6 +80,8 @@
 			<td>
             <a href="#details/<%= relay.get('fingerprint') %>">
             <%= _.escape(relay.get('nickname')) %>
+	    </a>
+	    <a href="#search/family:<%= relay.get('fingerprint') %>">
             <% if (!relay.get('is_bridge')) { %>(<%= relay.get('effective_family') == null ? 1 : relay.get('effective_family').length + 1 %>)<% } %>
             </a>
             </td>

Child Tickets

Change History (3)

comment:1 Changed 3 years ago by cypherpunks

Status: assignedneeds_review

comment:2 Changed 3 years ago by cypherpunks

this make also clear what the new number is about

comment:3 Changed 3 years ago by irl

Resolution: fixed
Status: needs_reviewclosed

I've merged a slightly modified version of this patch in 145e223.

Note: See TracTickets for help on using tickets.