-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
update MessageMedia.fromUrl #811
Conversation
wictornogueira
commented
Aug 5, 2021
- fromUrl will now set filename based on the url (File name is null when using method MessageMedia.fromUrl() #808)
- Content-type is now used as fallback
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Warning: if you download something from an empty path ( https://image.example.com/ ) fileName will be empty. but the chances of this happening seems very low |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
* fix MessageMedia.fromUrl docs * use content-type as fallback * set filename based on url * fix empty filename * wrap expression in parentheses * hotfix * JSDoc fix * fix for jsdog * eslint gets retirement home Co-authored-by: Rajeh Taher <rajeh@reforward.dev> Co-authored-by: Aliyss Snow <33941859+Aliyss@users.noreply.github.com>
* fix MessageMedia.fromUrl docs * use content-type as fallback * set filename based on url * fix empty filename * wrap expression in parentheses * hotfix * JSDoc fix * fix for jsdog * eslint gets retirement home Co-authored-by: Rajeh Taher <rajeh@reforward.dev> Co-authored-by: Aliyss Snow <33941859+Aliyss@users.noreply.github.com>