[nodebb-plugin-github-embed] GitHub Issue Embedding

julian

And maybe you should add some styles that reduce the user-image size within "Recent Posts" widget, there is a giant julian-image laughing about me

LOL you know, I noticed that too, but I wasn't sure whether it was a problem with core (since it auto adds img-responsive class to all images), or whether the problem was the plugin. I know we get around it in our emoji plugins with an !important style, but I often wonder if there's a better way...

As for the issues being the wrong way around, that's a good point (leave it for my debut post a bug to show up, eh?). It's because I use two separate regular expressions to find matches. The shorthand regex is run first (hence why it showed up first). Perhaps they can be merged into one regular expression...

Text completion would be possible... didn't really think of it yet

frissdiegurke

@julian said:

LOL you know, I noticed that too, but I wasn't sure whether it was a problem with core (since it auto adds img-responsive class to all images), or whether the problem was the plugin. I know we get around it in our emoji plugins with an !important style, but I often wonder if there's a better way...

Isn't the reason that the divs .github-issue and .meta are missing and because of this the selector .github-issue .meta img doesn't match any more?
If you add

.author-picture {
  float: left;
  width: 16px;
  margin-right: .5em;
}

it should be better, shouldn't it (just tested temporarily addition within browser)?
And a clear: left; should separate multiple embeds at least a bit

julian

@frissdiegurke Yeah, the surrounding tags are removed because they're not in the whitelist, but <img> is. I'll add the necessary styles to .author-picture

frissdiegurke

@julian lol, got the last 3 notifications (1h ago) delivered with this one ^^ no clue why

frissdiegurke

@julian And I get 502 on accessing https://community.nodebb.org/notifications may there be some kind of bug?

julian

For this wondering, it turns out it was a bug in NodeBB, which has since been patched Thanks @frissdiegurke !

frissdiegurke

@julian You're welcome it's always nice to look into other peoples code, if it's well-structured
And to test your plugin once more: #1413

julian

Released v0.1.1 -- fixed the issue ordering @frissdiegurke

julian

v0.1.2, a bit better styling, two per line now.

julian

v0.1.4, fixed lots of issues that @frissdiegurke found

julian

v0.2.0

Short-form syntax has changed to be gh#{num}, so that it does not conflict with user experience (accidental false positive) and markdown # as header.
- Long-form syntax is unchanged.

Testing new syntax: gh#1413

trevor

gh#1337

Just testing this.

<baris>

Wouldn't it make sense to replace gh#123 with the rendered output instead of adding it to the end like other markdown stuff?

julian

It's mostly a aesthetic choice. I'm not a fan of plugins that replace certain keywords with an inline object (take for example, youtube videos). That example is less of a problem, but I like being able to reference an issue in the middle of a sentence... a la:

Take a look at gh#256, that's where we solved that bug

^ No interruption of flow, and the referenced issue is at the bottom for you to check out if you want.

psychobunny

I'll disagree with both of you and suggest that the output be a popup similar to @Schamper's plugin. On mobile, just links to the actual issue. When I mention gh#123 it should be a link, at the very least. Just some ideas

EDIT: Scratch all of the above, you guys just gave me a great idea for a hook. Will get back to this later

a_5mith

@psychobunny reading the title of the issue you used in your example took 4 attempts to make any sense... kept thinking I was reading words over and over again like a child that reads the same line twice.

julian

Hahah wow @psychobunny actually fixed gh#123? I don't even need it anymore, but it's a nice thing to have

psychobunny

@a_5mith said:

@psychobunny reading the title of the issue you used in your example took 4 attempts to make any sense... kept thinking I was reading words over and over again like a child that reads the same line twice.

And the best part?

I don't even need it anymore

Mandosis

@julian Is it possible to update this plugin to work with private repositories? I have a private forum for a group of developers I am working with to create and indie game and it would great to be able to use this but our repo must be private for obvious reasons.

julian

Hi @Chris-Rabuse -- private repositories are hidden behind secure API endpoints that require a client ID and secret to access. You can generate one tied to your account in the GitHub account settings. I use my personal GitHub account, but I believe an organization works too.

It looks something like this:

If you register for an id/secret pair and put it into the GitHub Embed settings, it should allow the plugin to read/show private issues.

Keep in mind that this also means unprivileged users on your forum could also use the short codes to retrieve private issue data, so caveat emptor