fabacab / multisite-directory

:card_index::globe_with_meridians: Add a browseable, flexible directory of the sites in a WP Multisite network.
https://wordpress.org/plugins/multisite-directory/
GNU General Public License v2.0
1 stars 6 forks source link

Suggestions #20

Closed hvianna closed 7 years ago

hvianna commented 7 years ago

First of all, I'm sorry for my PR completely without notice. I'm new to contributing code here on GitHub and still learning the ropes (plus, poor social skills).. Anyway, I have a few more suggestions that were just commited and I just realized they would be included in the original PR, so I decided to open this issue and properly present my suggestions.

So here's a summary of changes proposed in #19

fabacab commented 7 years ago

Thank you so much for the pull request. You're 💯% awesome!

For what it's worth, it's perfectly okay with me if you open a pull request instead of an issue. I use issues as a sort of "to do list" while pull requests are more useful to me as a showcase of work in progress. Pull requests are also more appropriate in this case than a single issue is because a pull request can contain more than one chunk of behavior (as long as the commits in the pull request are atomic), whereas issues that represent more than one distinct issue undermines the utility of using an issue tracker in the first place. ;)

So, with that said, I really like your suggestions and will review them in the pull request itself.