Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add RichEmbed#attachFile #1202

Merged
merged 5 commits into from
Feb 21, 2017
Merged

Add RichEmbed#attachFile #1202

merged 5 commits into from
Feb 21, 2017

Conversation

zajrik
Copy link
Contributor

@zajrik zajrik commented Feb 20, 2017

Mostly for attaching local images that can be accessed within the embed image/author icon/footer icon via attachment//filename.png and the like.

Mostly for attaching local images that can be accessed within the embed image/author icon/footer icon via `attachment//filename.png` and the like.
@Tvde1
Copy link

Tvde1 commented Feb 21, 2017

get this accepted already

@zajrik
Copy link
Contributor Author

zajrik commented Feb 21, 2017

I don't pester about my PRs. It will get merged when it gets merged. Have some patience. 👌

@Gawdl3y Gawdl3y merged commit db5259c into discordjs:master Feb 21, 2017
devsnek pushed a commit to devsnek/discord.js that referenced this pull request May 14, 2017
* Add RichEmbed#attachFile

Mostly for attaching local images that can be accessed within the embed image/author icon/footer icon via `attachment//filename.png` and the like.

* Update docstring to reflect valid param types

* Update TextBasedChannel.js

* Update RichEmbed.js

* Update RichEmbed.js
devsnek pushed a commit to devsnek/discord.js that referenced this pull request May 14, 2017
* Add RichEmbed#attachFile

Mostly for attaching local images that can be accessed within the embed image/author icon/footer icon via `attachment//filename.png` and the like.

* Update docstring to reflect valid param types

* Update TextBasedChannel.js

* Update RichEmbed.js

* Update RichEmbed.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants