<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hi Mojca,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Apr 12, 2019 at 3:03 AM Mojca Miklavec <<a href="mailto:mojca@macports.org">mojca@macports.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Awesome!</blockquote><div> </div><div>Thank You. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">One thing that "urgently" needs to be done is to obfuscate the email<br></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
(if written at all). I'm not even sure whether we actually want the<br>
email being displayed there. We need it to send automated emails from<br>
the buildbot in case some failure happens, or to occasionally contact<br>
the maintainer directly. But exposing that info on the website might<br>
be too much.<br></blockquote><div> </div><div>I had fixed it as soon as I saw the email, but couldn't reply that time.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">As far as accessing the information is concerned: at the moment you<br>
use email as unique identifier. I would probably use github handle by<br>
default / as the main entry point. I think that '@' is an allowed<br>
character in URL. If so, we could use<br>
    <url>/maintainer/@ryandesign<br>
to access the same page</blockquote><div> </div><div>There was a big error in my parsing script due to which the GitHub handles of a lot of maintainers were not being parsed. And hence, I went with email. But still, some 300 maintainers have not provided GitHub handles.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Alternatively we could allow macports handles (for those with<br>
@<a href="http://macports.org" rel="noreferrer" target="_blank">macports.org</a> email addresses), so<br>
    <url>/maintainer/ryandesign<br>
could work just as well.<br></blockquote><div> </div><div>Yes, this would be good. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">For maintainers with a super long list of ports, or, for<br>
non-maintained ports in particular, we might eventually need a way to<br>
shorten that list (have multiple pages) or provide a similar search<br>
functionality as for global ports, except that here it would be<br>
limited to the ports by that particular maintainer.</blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">I would put that list of ports in a table and add version & short description.<br></blockquote><div> </div><div>Yes, I will add pagination and show more details for both list of ports on maintainer's page and on category page. The filter would also be amazing. I will do this.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">I still need to check the code: what's your current strategy for<br>
showing links to the tickets?<br>
At some point we could differentiate different types of tickets (for<br>
example mark bugs separately).<br></blockquote><div> </div><div>Sorry to disappoint you here, but right now I am using web scrapping to do this. I am looking for a plugin that could add public api feature to track tickets, but maybe it doesn't exist right now. We can make our own for sure.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">One minor suggestion. I really like the "search for port" field. Could<br>
this be added to every page? There is "ports" in the top right corner,<br>
but that one is a lot less useful in itself (not saying that it should<br>
go, just suggesting search on each page).<br></blockquote><div> </div><div>I have added it to maintainer-detail and port-detail. But it still needs some work on its position on the page. </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Here's what I would do, but feel free to propose an alternative and/or<br>
discuss further. (Actually, I have two slightly different ideas for<br>
implementation in my mind, I'll describe one of them first.)<br>
<br>
For the maintainers you could declare a unique keyword over the<br>
combination of github handle + email. Every maintainer of every port<br>
has a uniquely specified pair (github + email) when you import it to<br>
the database (neither github handles not emails would be unique on its<br>
own). Note that you still have index specified on both columns<br>
(separately on each, but the uniqueness is only defined on combination<br>
of the two).<br>
<br>
When you read the port, you check the pair (@github, email). If the<br>
pair already exists in the database, you enter the (port, author) pair<br>
into the database of maintainerships. If it doesn't exist, you create<br>
it first, and then assign the maintainership to the port. (Note that<br>
whenever you are updating the port, you also need to check if you need<br>
to remove some maintainers from that port.)<br>
<br>
When you display a particular maintainer, say @somerandomgithubhandle,<br>
you run a query and if you hit more that one entry with that github<br>
handle in the database:<br></blockquote><div><br></div><div>This would solve the problem of multiple emails with same GitHub handle. But there are cases when the maintainer has provided 'GitHub and email' both for one port and just 'email' for other port. Sorry If I am missing something.</div><div><br></div><div>Example:</div><div>for libsmf {ryandesign openmaintainer}</div><div>for penal-soft {{ryandesign @ryandesign} openmaintainer}</div><div><br></div><div>And while many haven't provided GitHub handles, some haven't provided emails.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">You could also have a separate page which runs different queries and<br>
looks for all maintainers with inconsistencies (that's for later). It<br>
would generally be helpful to have a collection of such pages for<br>
different things, like: all broken builds on buildbot, all outdated<br>
ports, ...<br></blockquote><div> </div><div>This for sure would be very good to add into the app once everything is ready.</div><div><br></div><div>Thank You</div></div></div></div></div></div>