Skip to content

Conversation

m1guelpf
Copy link
Contributor

@m1guelpf m1guelpf commented Feb 25, 2019

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

I use Blogcast to generate audio versions of my articles, and as Dev.to doesn't allow raw embeds, I figured out I'd try to add it.

(DISCLAIMER: Blogcast is not a free product ($0.50/article), and I'm the creator. I'd totally understand if any of those things made you reject the contribution. Just wanted to contribute something, no hard feelings 😄)

Related Tickets & Documents

  • N/A

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

  • N/A

Added to documentation?

  • docs.dev.to

  • readme

  • no documentation needed

  •  liquid tags documentation page

[optional] What gif best describes this PR or how it makes you feel?

First contribution to DEV!

@pr-triage pr-triage bot added the PR: unreviewed bot applied label for PR's with no review label Feb 25, 2019
@CLAassistant
Copy link

CLAassistant commented Feb 25, 2019

CLA assistant check
All committers have signed the CLA.

@benhalpern benhalpern requested a review from Zhao-Andy February 25, 2019 21:32
@benhalpern
Copy link
Contributor

Oh cool, we'd definitely love to offer this. @Zhao-Andy wanna review this?

Copy link
Contributor

@Zhao-Andy Zhao-Andy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left some comments for the issues that need to be addressed. Everything else looks good.

@pr-triage pr-triage bot added PR: reviewed-changes-requested bot applied label for PR's where reviewer requests changes and removed PR: unreviewed bot applied label for PR's with no review labels Feb 27, 2019
Co-Authored-By: m1guelpf <github@miguelpiedrafita.com>
@pr-triage pr-triage bot added PR: unreviewed bot applied label for PR's with no review and removed PR: reviewed-changes-requested bot applied label for PR's where reviewer requests changes labels Feb 28, 2019
Copy link
Contributor

@Zhao-Andy Zhao-Andy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@pr-triage pr-triage bot added PR: reviewed-approved bot applied label for PR's where reviewer approves changes and removed PR: unreviewed bot applied label for PR's with no review labels Feb 28, 2019
@maestromac maestromac merged commit eb37d75 into forem:master Feb 28, 2019
@pr-triage pr-triage bot added PR: merged bot applied label for PR's that are merged and removed PR: reviewed-approved bot applied label for PR's where reviewer approves changes labels Feb 28, 2019
@m1guelpf m1guelpf deleted the blogcast branch February 28, 2019 22:11
@Zhao-Andy
Copy link
Contributor

Hey @m1guelpf, I meant to say this when it was merged, but feel free to write up a changelog post for this if you'd like. :) Here's an example: https://dev.to/link2twenty/changelog-jsfiddle-liquid-tags-now-live-1d1b

@m1guelpf
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: merged bot applied label for PR's that are merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants