Release Version 2.0 of Micropub plugin #150

dshanske · 2018-08-14T04:04:56Z

Version 2.0 is merely the name I'm using for all the changes thus far. Since I have decided I'm not going to pursue any further enhancements before releasing a new version, using this issue to track anything left to do before releasing this version as stable.

Asked @chrisaldrich , @jgmac1106, and @dougbeal if they would mind testing the current stable version for issues, as my tests with various Micropub clients seem to be working. Tests conducted with and without the IndieAuth plugin activated.

Tests should include posting with at least a client or two. Quill, Micropublish, etc.

Subject to that and any other blockers, want to share this with the world.

jackjamieson2 · 2018-08-14T16:59:51Z

A couple reports from my testing.

1. When reposting from monocle or together, the following php notices occur:

[14-Aug-2018 16:46:50 UTC] Micropub Input: {"type":["h-entry"],"properties":{"repost-of":["https://jackjamieson.net/511-2/"]}}

14-Aug-2018 16:12:27 UTC] PHP Notice: Undefined index: content in/...{path}.../wp-content/plugins/wordpress-micropub-master/includes/class-micropub-endpoint.php on line 616

[14-Aug-2018 16:12:27 UTC] PHP Notice: Undefined index: summary in/...{path}.../wp-content/plugins/wordpress-micropub-master/includes/class-micropub-endpoint.php on line 622

[14-Aug-2018 16:12:27 UTC] PHP Notice: Undefined index: content in/...{path}.../wp-content/plugins/wordpress-micropub-master/includes/class-micropub-render.php on line 79

A repost is created, but it only consists of a link to the post. The content of the repost is otherwise empty. I don't know if I've ever micropubbed a repost before so I'm not sure if this is the expected behaviour.

2. Posting from Quill with attached image
Everything works, including the image, but my log includes an 404 message about the media endpoint

[14-Aug-2018 16:55:14 UTC] REST request: /micropub/1.0/media: {"q":"last"}

[14-Aug-2018 16:55:14 UTC] REST result: : {"code":"rest_no_route","message":"No route was found matching the URL and request method","data":{"status":404}}

dshanske · 2018-08-16T11:39:51Z

Re 2 above, https://indieweb.org/micropub_media_endpoint#Query_the_last_thing_uploaded - Basically, only Quill supports querying an endpoint. But right now, it generates an error because the endpoint doesn't support querying. I will change it and allow it to be queried, but it will return a proper error at this point, as not supporting last in this round.

miklb · 2018-08-16T12:44:31Z

FWIW I’m on master of all of the plugins & I cant log in with Quill or Indigenous

dshanske · 2018-08-16T16:30:06Z

@miklb Can I work with you to troubleshoot? What data do you have to reproduce?

miklb · 2018-08-16T16:41:54Z

I’ll pull the logs & error messages & post here.

dshanske · 2018-08-16T16:47:26Z

Thanks, I want to narrow it down. Are you using the IndieAuth plugin or is it disabled? Does changing that make a difference?

dshanske · 2018-08-16T16:48:37Z

@jackjamieson2 If the client doesn't send the original post as content, the endpoint won't retrieve it, so that repost is technically correct.

miklb · 2018-08-16T16:53:04Z

I updated all plugins to master. Should I be testing a branch?

dshanske · 2018-08-16T16:55:11Z

No, but there is one little IndieAuth fix, but your site URL is the same as your home url?

miklb · 2018-08-16T18:31:01Z

Indigenous

REST result: : {"code":"unauthorized","message":"missing access token","data":{"status":401}}

and Quill just did a weird redirect loop to this https://miklb.com/?me=https%3A%2F%2Fmiklb.com%2F&redirect_uri=https%3A%2F%2Fquill.p3k.io%2Fauth%2Fcallback&client_id=https%3A%2F%2Fquill.p3k.io%2F&state=75e9a54201f754ae&scope=create%20update%20media&response_type=code

dshanske · 2018-08-16T21:27:05Z

So, missing access token suggests server filtering of the header

dshanske · 2018-08-16T21:32:00Z

But that url is missing wp login.php

miklb · 2018-08-16T21:54:49Z

To my knowledge, I haven't changed any header settings in nginx in a long time, and was able to authenticate not that long ago. I've lost track honestly of when things last fully worked together though.

dshanske · 2018-08-16T21:57:26Z

I wonder what it could be.

dshanske · 2018-08-16T22:01:02Z

Is this working without the IndieAuth plugin?

miklb · 2018-08-16T22:16:52Z

Is this working without the IndieAuth plugin?

I haven't tested that, I thought the idea of refactoring the micropub plugin was better compatibility with WP IndieAuth.

I'll try and test with IndieAuth off later this evening.

dshanske · 2018-08-16T22:23:13Z

Process of elimination

miklb · 2018-08-16T22:35:04Z

A lot of things I’m thinking about eliminating these days…

…

On Aug 16, 2018, 18:23 -0400, David Shanske ***@***.***>, wrote: Process of elimination — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

dshanske · 2018-08-16T22:37:44Z

I am sorry

miklb · 2018-08-17T00:03:13Z

I don’t hold you responsible David for my website woes.

miklb · 2018-08-17T16:48:07Z

@dshanske you should still have a login to the indiewp.com on my server if you want to test my server configuration/mulit-site. Let me know if you need me to resend or activate/install anything. There is a plugin to view debug log from the admin too.

dshanske · 2018-08-18T04:02:51Z

@jackjamieson2 The latest merge should fix the PHP notices.

dshanske · 2018-08-19T04:31:03Z

As of this comment, per my conversation with @snarfed. I'm freezing any further enhancements. Only fixes and tests will be pushed until we release a version. While I'd like it if 100% of people had no issues, some are due to server issues and such, which I am always chasing as a priority to troubleshoot. I probably should settle for it working for the same or more people as it did before all these pushes and continue trying to figure out the remaining people's issues afterward.

miklb · 2018-08-19T04:50:49Z

Can you document “server issues” that can cause the plugins to fail?

chrisaldrich · 2018-08-19T06:29:12Z

Some observations from a fresh install running current IW Stack, but with the test Micropub install:

Using micropublish.net to send a note gives the following error:
There was an error making a request to your Micropub endpoint. The error received was: Warning: in_array() expects parameter 2 to be array, null given in htdocs/site/wp-content/plugins/wordpress-micropub-master/includes/class-micropub-endpoint.php on line 164

followed by:
Warning: Invalid argument supplied for foreach() in /htdocs//wp-includes/rest-api/class-wp-rest-server.php on line 1259 {"error":"insufficient_scope","error_description":"scope insufficient to create posts"}

I was able to log into IndieBookClub.biz and post there, but post didn't micropub to appear on my site. (This is likely a header stripping issue I'd seen with my production server, but isn't fixed on this install. I'll see what happens when I run things on my production site shortly.)

Quill note post with photo worked (and showed up as photo kind).

Bookmark post from quill gave 404 Page Not Found and redirected to https://quill.p3k.io/null on Draft and other modes, but posted mostly properly except that the Bookmark and URL text appeared in the_content instead of within post kinds as I would have suspected.

Teacup worked and set the right post kind, but put all content into the_content instead of into the properties fields I would have expected for Post Kinds. I could have sworn someone said Teacup had a media endpoint if it detected the site had one available, but I never saw a way to upload photos for the media endpoint in Teacup.

While most of this may not seem perfect, it seems solid enough to me that I'll install the test version on production and rerun several of the tests including for OYS which I've got set up pthere and a couple of others. Given a few bumps my test server surely has, I suspect some of the issues I've seen will wash themselves out.

dshanske · 2018-08-19T11:13:16Z

@miklb Trying to.

aaronpk · 2018-08-19T15:28:16Z

I just tried installing indieauth and micropub on a new wordpress from master.

I was able to get an access token, and I tried posting from the command line:

curl -H "Authorization: Bearer EmqwS2gHFxB9H28G7mEAnMcu1UXhCbUVEzDfw8KAiPhjKzBd9O6CIf2zpxevtHYM40YSQLJ98xL8CZg6g2fKlZmQNSkeOObd0xLn9gBPwBQuXUuDMgIYvGXGoXF24pPH" -d h=entry -d content=hi https://wordpress.com.dev/index.php?rest_route=/micropub/1.0/endpoint
<br />
<b>Warning</b>:  in_array() expects parameter 2 to be array, null given in <b>/Users/aaronparecki/Code/wordpress/wordpress/wp-content/plugins/micropub/includes/class-micropub-endpoint.php</b> on line <b>164</b><br />
<br />
<b>Warning</b>:  in_array() expects parameter 2 to be array, null given in <b>/Users/aaronparecki/Code/wordpress/wordpress/wp-content/plugins/micropub/includes/class-micropub-endpoint.php</b> on line <b>164</b><br />
<br />
<b>Warning</b>:  in_array() expects parameter 2 to be array, null given in <b>/Users/aaronparecki/Code/wordpress/wordpress/wp-content/plugins/micropub/includes/class-micropub-endpoint.php</b> on line <b>164</b><br />
<br />
<b>Warning</b>:  in_array() expects parameter 2 to be array, null given in <b>/Users/aaronparecki/Code/wordpress/wordpress/wp-content/plugins/micropub/includes/class-micropub-endpoint.php</b> on line <b>164</b><br />
<br />
<b>Warning</b>:  Invalid argument supplied for foreach() in <b>/Users/aaronparecki/Code/wordpress/wordpress/wp-includes/rest-api/class-wp-rest-server.php</b> on line <b>1259</b><br />
{"error":"insufficient_scope","error_description":"scope insufficient to create posts"}

if I remove the authorization header, it fails in the same way. If there is no authorization header, or if the access token is not valid, it should return an error before trying to check for scopes.

chrisaldrich · 2018-08-25T20:11:24Z

Micropub errors for OYS 8-25-18 (using master branch from 2018-08-24 ef76125)
[25-Aug-2018 18:42:04 UTC] Micropub Error: 403 forbidden - Unauthorized [25-Aug-2018 18:42:04 UTC] REST request: /micropub/1.0/endpoint: {"type":["h-entry"],"properties":{"published":["2018-08-24T17:44:13-07:00"],"syndication":["https:\/\/www.swarmapp.com\/user\/11479\/checkin\/5b80a65de0c0c9002c0c8589"],"content":["Being lazy for dinner tonight."],"photo":["https:\/\/igx.4sqi.net\/img\/general\/original\/11479_Z5VSLqOla8M3LhlRo_7QKUXeiGvaufQM1NTUYg32Dp0.jpg"],"checkin":[{"type":["h-card"],"properties":{"name":["Gerrish Grill"],"url":["https:\/\/foursquare.com\/v\/5b66159f2db4a9002ce66d2b"],"latitude":[34.1745],"longitude":[-118.09457],"locality":["Pasadena"],"region":["CA"],"country-name":["United States"],"postal-code":["91107"]},"value":"https:\/\/foursquare.com\/v\/5b66159f2db4a9002ce66d2b"}],"location":[{"type":["h-adr"],"properties":{"latitude":[34.1745],"longitude":[-118.09457],"locality":["Pasadena"],"region":["CA"],"country-name":["United States"],"postal-code":["91107"]}}]}}(Header Absent) [25-Aug-2018 18:42:04 UTC] REST result: /micropub/1.0/endpoint: {"error":"forbidden","error_description":"Unauthorized"}(403) - null(User ID: 0) [25-Aug-2018 18:42:04 UTC] PHP Warning: Invalid argument supplied for foreach() in /htdocs/wp-includes/rest-api/class-wp-rest-server.php on line 1259 [25-Aug-2018 18:42:08 UTC] PHP Notice: Array to string conversion in /htdocs/wp-includes/formatting.php on line 1045 [25-Aug-2018 18:42:08 UTC] PHP Notice: Array to string conversion in /htdocs/wp-includes/formatting.php on line 1045 [25-Aug-2018 18:42:08 UTC] PHP Notice: Array to string conversion in /htdocs/wp-includes/formatting.php on line 1045 [25-Aug-2018 18:42:09 UTC] PHP Notice: Array to string conversion in /htdocs//wp-includes/formatting.php on line 1045 [25-Aug-2018 18:42:12 UTC] PHP Notice: Array to string conversion in /htdocs/wp-includes/formatting.php on line 1045 [25-Aug-2018 18:42:12 UTC] PHP Notice: Array to string conversion in /htdocs/wp-includes/formatting.php on line 1045 [25-Aug-2018 18:42:14 UTC] http://nominatim.openstreetmap.org/reverse?format=json&extratags=1&addressdetails=1&lat=45.53548&lon=-122.621244&zoom=18&accept-language=en-US [25-Aug-2018 18:42:15 UTC] http://nominatim.openstreetmap.org/reverse?format=json&extratags=1&addressdetails=1&lat=45.53548&lon=-122.621244&zoom=18&accept-language=en-US

(Originally published at https://boffosocko.com/2018/08/25/reply-to-release-version-2-0-of-micropub-plugin-%c2%b7-issue-150-%c2%b7-snarfed-wordpress-micropub/)

asuh · 2018-08-25T20:18:54Z

Installed tree ef76125 from this repo, updated WP Indieauth to 3.1.5, logged into Quill to post a Note, received a 403 error when clicking on Post button.

Error log produces the following:
Invalid argument supplied for foreach() in [...]/wp-includes/rest-api/class-wp-rest-server.php on line 1259

It shows the REST request I made from Quill with everything I filled out and ends with (Header Absent)

Similar to @chrisaldrich above

miklb · 2018-08-25T20:22:10Z

for complete reporting I am seeing this from Omnibear

**this is fixed in latest master of IndieAuth**
25-Aug-2018 20:02:28 UTC] PHP Notice:  Undefined variable: params in plugins/wordpress-indieauth/includes/class-indieauth-authorize.php on line 118 

[25-Aug-2018 20:02:28 UTC] Micropub Error: 403 forbidden - Unauthorized 
[25-Aug-2018 20:02:28 UTC] REST request: /micropub/1.0/endpoint: {"q":"syndicate-to"}(Header Present) 
[25-Aug-2018 20:02:28 UTC] REST result: /micropub/1.0/endpoint: {"error":"forbidden","error_description":"Unauthorized"}(403) - null(User ID: 0) 
[25-Aug-2018 20:02:28 UTC] PHP Warning:  Invalid argument supplied for foreach() in /wp-includes/rest-api/class-wp-rest-server.php on line 1259

chrisaldrich · 2018-09-06T18:01:11Z

With the most recent version 261c782 I'm still getting errors posting with OwnYourSwarm:

Is this indicating it isn't finding the correct user account?

[06-Sep-2018 17:48:12 UTC] PHP Notice:  Array to string conversion in /htdocs/wp-includes/formatting.php on line 1045
[06-Sep-2018 17:48:12 UTC] PHP Notice:  Array to string conversion in /htdocs/wp-includes/formatting.php on line 1045
[06-Sep-2018 17:48:12 UTC] PHP Notice:  Array to string conversion in /htdocs/wp-includes/formatting.php on line 1045
[06-Sep-2018 17:48:13 UTC] PHP Notice:  Array to string conversion in /htdocs/wp-includes/formatting.php on line 1045
[06-Sep-2018 17:48:20 UTC] PHP Notice:  Trying to get property of non-object in /htdocs/wp-includes/post-template.php on line 298
[06-Sep-2018 17:48:20 UTC] PHP Notice:  Array to string conversion in /htdocs/wp-includes/formatting.php on line 1045
[06-Sep-2018 17:48:21 UTC] PHP Notice:  Array to string conversion in /htdocs/wp-includes/formatting.php on line 1045
 
[06-Sep-2018 17:48:30 UTC] Micropub Error: 403 forbidden - Unauthorized
 
[06-Sep-2018 17:48:30 UTC] REST request: /micropub/1.0/endpoint: {"type":["h-entry"],"properties":{"published":["2018-09-05T17:05:41-07:00"],"syndication":["https:\/\/www.swarmapp.com\/user\/11479\/checkin\/5b906f556cd68e00258e6ec6"],"content":["Afternoon soccer"],"photo":["https:\/\/igx.4sqi.net\/img\/general\/original\/11479_J-cY4WxP6tHKCHQj1P52Fe3Q6Aa3XBQNSUmXXlhDaFc.jpg"],"checkin":[{"type":["h-card"],"properties":{"name":["Jefferson Park"],"url":["https:\/\/foursquare.com\/v\/4bfe831c8992a593b49cadb0"],"latitude":[34.154797926121],"longitude":[-118.11971716429],"locality":["Pasadena"],"region":["CA"],"country-name":["United States"],"postal-code":["91106"]},"value":"https:\/\/foursquare.com\/v\/4bfe831c8992a593b49cadb0"}],"location":[{"type":["h-adr"],"properties":{"latitude":[34.154797926121],"longitude":[-118.11971716429],"locality":["Pasadena"],"region":["CA"],"country-name":["United States"],"postal-code":["91106"]}}]}}(Header Absent)
 
[06-Sep-2018 17:48:30 UTC] REST result: /micropub/1.0/endpoint: {"error":"forbidden","error_description":"Unauthorized"}(403) - null(User ID: 0)

[06-Sep-2018 17:48:30 UTC] PHP Warning:  Invalid argument supplied for foreach() in /htdocs/wp-includes/rest-api/class-wp-rest-server.php on line 1259

miklb · 2018-09-06T18:36:55Z

I asked in IRC/chat but my curiosity about the error message regarding User: 0 and @dshanske clarifying that means "no credentials", is this an indieauth issue? I realize the 2 are intrinsically connected at this point, but how to get to the source of the error has me questioning where it is actually coming from, despite the error message.

miklb · 2018-09-07T03:57:13Z

my latest debug.log for the error

Micropub Error: 403 forbidden - Unauthorized 
REST request: /micropub/1.0/endpoint: {"q":"syndicate-to"}(Header Present) 
REST result: /micropub/1.0/endpoint: {"error":"forbidden","error_description":"Unauthorized"}(403) - null(User ID: 0) 
PHP Warning:  Invalid argument supplied for foreach() in /wp-includes/rest-api/class-wp-rest-server.php on line 1259

miklb · 2018-09-07T04:59:14Z

@chrisaldrich is your site a stand alone WP install or a network install?

chrisaldrich · 2018-09-07T22:29:44Z

@miklb That one is from a simple instance on a shared server.

dshanske · 2018-09-14T12:29:39Z

I am trying to figure out that error notice in the rest server.

dshanske · 2018-09-16T06:10:16Z

The invalid argument foreach is coming from a line in the REST server that sends headers.

https://github.com/WordPress/WordPress/blob/master/wp-includes/rest-api/class-wp-rest-server.php#1269 (In the older versions 1259).

It is called earlier as part of serve_request. I'm still stumped. The only possible thing I can think of that might explain this is the debug handling class. I'm curious if the error goes away if WP_DEBUG is false. But, that's the only thing in IndieAuth or Micropub that modifies the response coming back in

ngm · 2018-09-22T17:49:25Z

Testing this version, installed via .zip download of master branch.

These are my installed plugins, if it's helpful:

A simple note post from Quill posted successfully after the update.
Also simple note works OK from micropublish.

ngm · 2018-09-22T17:57:49Z

When I Refresh Configuration from Indigenous for Android, I get the following errors:

Error getting syndication targets: Value <!DOCTYPE of type java.lang.String cannot be converted to JSONObject

Error getting media endpoint: Value <!DOCTYPE of type java.lang.String cannot be converted to JSONObject

Error getting post-types: Value <!DOCTYPE of type java.lang.String cannot be converted to JSONObject

Indigenous was previously configured OK, but I'll just doublecheck and revert to previous WP Micropub to confirm what happens.

UPDATE: Refresh Configuration is fine with v1.4.3 of WP Micropub plugin.

dshanske · 2018-09-22T18:54:36Z

I use Indigenous for Android. @swentel Thoughts?

dshanske · 2018-09-22T18:56:23Z

@ngm By the way, Webmention for Comments was merged into Webmentions

swentel · 2018-09-22T19:04:27Z

@dshanske difference in endpoints maybe? I vaguely kind of remember seeing something like ?endpoint=micropub vs /micropub/1.0/endpoint (or something alike)

If that's the case, Indigenous doesn't refresh endpoints yet :/ So you'll have to delete the account and add it again to get the new endpoints.

dshanske · 2018-09-22T19:06:01Z

That would be it.

swentel · 2018-09-22T19:12:51Z

I'll prioritize IndiePass/indiepass-android#94 :)

ngm · 2018-09-22T19:15:15Z

Marvellous, thanks @swentel

@dshanske - can confirm now then that posting via the new Micropub endpoint is working for me from Indigenous also.

ngm · 2018-09-23T08:43:17Z

On 2.0, when I syndicate my posts to Mastodon (via all micropub clients that I've tried), the content is wrapped in <div id="e-content">...</div>. It doesn't happen on 1.4.3.

(Noting that it's very possible that it's a problem with my theme or my syndication target code, but any pointers as to why it's not happening in 1.4.3?)

dshanske · 2018-09-23T12:53:22Z

It's a Post Kinds issue I need to push to stable. It doesn't know where to disable that extra code because of the Micropub changes.

dshanske · 2018-10-05T23:15:15Z

I fear I will never get enough data on issues regarding auth

dshanske · 2018-10-07T22:08:52Z

indieweb/wordpress-indieauth#113 - Auth issue @asuh reported has been resolved. Documentation to address in various plugins.

The Micropub auth, which maps URL to author and is enabled when the IndieAuth plugin isn't active, would suffer the same issue.

asuh · 2018-10-07T22:44:18Z

As @dshanske reported above, issue is resolved.

My problem had to do with using www and non-www in the hostname. My site eventually forced www on the domain name so that Let's Encrypt wouldn't cause a redirect loop from the web host and WordPress competing with different hostnames.

Considering my site forces www, I used the full host with www with Quill and it worked when I posted a note.

dshanske · 2018-10-10T22:34:12Z

Being as many of the issues seem to be with an Indieauth plugin Micropub interaction and I think I fixed another Indieauth issue, want to retest this.

dshanske · 2018-10-18T23:06:04Z

I think, with 3.1.9 of Indieauth, we may be ready to release.

dshanske · 2018-10-22T00:16:56Z

Released.

dshanske added this to the 2.0.0 milestone Aug 14, 2018

dshanske changed the title ~~Release Version 2.0 of Micrpub plugin~~ Release Version 2.0 of Micropub plugin Aug 14, 2018

dshanske mentioned this issue Aug 17, 2018

Notices and Query #152

Merged

dshanske closed this as completed Oct 22, 2018

ivanvetoshkin mentioned this issue Jul 19, 2020

403 forbidden - Unauthorized #228

Closed

Release Version 2.0 of Micropub plugin #150

Release Version 2.0 of Micropub plugin #150

Comments

dshanske commented Aug 14, 2018 • edited

jackjamieson2 commented Aug 14, 2018

dshanske commented Aug 16, 2018

miklb commented Aug 16, 2018

dshanske commented Aug 16, 2018

miklb commented Aug 16, 2018

dshanske commented Aug 16, 2018

dshanske commented Aug 16, 2018

miklb commented Aug 16, 2018

dshanske commented Aug 16, 2018

miklb commented Aug 16, 2018

dshanske commented Aug 16, 2018

dshanske commented Aug 16, 2018

miklb commented Aug 16, 2018

dshanske commented Aug 16, 2018

dshanske commented Aug 16, 2018

miklb commented Aug 16, 2018

dshanske commented Aug 16, 2018

miklb commented Aug 16, 2018 via email

dshanske commented Aug 16, 2018

miklb commented Aug 17, 2018

miklb commented Aug 17, 2018

dshanske commented Aug 18, 2018

dshanske commented Aug 19, 2018

miklb commented Aug 19, 2018

chrisaldrich commented Aug 19, 2018

dshanske commented Aug 19, 2018

aaronpk commented Aug 19, 2018

chrisaldrich commented Aug 25, 2018

asuh commented Aug 25, 2018 • edited

miklb commented Aug 25, 2018 • edited

chrisaldrich commented Sep 6, 2018

miklb commented Sep 6, 2018

miklb commented Sep 7, 2018

miklb commented Sep 7, 2018

chrisaldrich commented Sep 7, 2018

dshanske commented Sep 14, 2018

dshanske commented Sep 16, 2018

ngm commented Sep 22, 2018 • edited

ngm commented Sep 22, 2018 • edited

dshanske commented Sep 22, 2018

dshanske commented Sep 22, 2018

swentel commented Sep 22, 2018

dshanske commented Sep 22, 2018

swentel commented Sep 22, 2018

ngm commented Sep 22, 2018

ngm commented Sep 23, 2018

dshanske commented Sep 23, 2018

dshanske commented Oct 5, 2018

dshanske commented Oct 7, 2018

asuh commented Oct 7, 2018

dshanske commented Oct 10, 2018

dshanske commented Oct 18, 2018

dshanske commented Oct 22, 2018

dshanske commented Aug 14, 2018 •

edited

asuh commented Aug 25, 2018 •

edited

miklb commented Aug 25, 2018 •

edited

ngm commented Sep 22, 2018 •

edited

ngm commented Sep 22, 2018 •

edited