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

Use webfactory/phumbor instead of 99designs/phumbor #12

Merged
merged 1 commit into from
Nov 1, 2019

Conversation

mpdude
Copy link
Contributor

@mpdude mpdude commented Oct 31, 2019

The 99designs/phumbor library has been unmaintained for a long time. In particular, there is a long-standing issue regarding application of rawurlencode() to image URLs (99designs/phumbor#20) which has not been addressed. Also the direct question whether the code is still maintained or not has not been answered for almost a year.

In order to address the URL encoding issue, I forked the original repo:

With this change I would like to suggest switching this bundle over to the forked library.

The `99designs/phumbor` library has been unmaintained for a long time. In particular, there is a long-standing issue regarding application of `rawurlencode()` to image URLs (99designs/phumbor#20) which has not been addressed. Also the (direct question whether the code is still maintained or not)[99designs/phumbor#30] has not been answered for almost a year.

In order to address the URL encoding issue, I forked the original repo:

* https:/webfactory/phumbor/
* https://packagist.org/packages/webfactory/phumbor

With this change I would like to suggest switching this bundle over to the forked library.
@mpdude
Copy link
Contributor Author

mpdude commented Oct 31, 2019

/notify @MalteWunsch

@jbouzekri
Copy link
Owner

Strange because https:/99designs seems to still have a lot of activities but you are right, nothing for the past years on this library. I am going to merge it and revert in case they merge the url encoding part later.

@jbouzekri jbouzekri merged commit c7aac0b into jbouzekri:master Nov 1, 2019
@mpdude
Copy link
Contributor Author

mpdude commented Nov 1, 2019

Thank you @jbouzekri

@mpdude mpdude deleted the patch-1 branch November 1, 2019 07:59
@jbouzekri
Copy link
Owner

Released 2.0.0

@mpdude
Copy link
Contributor Author

mpdude commented Nov 1, 2019

Oh, by the way, what is your release policy? Do you tag new releases regularly?

Disregard. You just tagged it :-) 🎉

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.

2 participants