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 Dragonfly 1.1 and above #1314

Merged
merged 1 commit into from
Oct 16, 2017
Merged

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Oct 16, 2017

No description provided.

@robinboening
Copy link
Contributor

Does this dragonfly update fix anything, or adds a feature, or is there any other noticeable change introduced?

@tvdeyen
Copy link
Member Author

tvdeyen commented Oct 16, 2017

The maximum version we allowed (1.0.12) is from 2015. There are some fixes made since then

https://github.com/markevans/dragonfly/blob/master/History.md

But you are right. I think it is a good idea to not force everybody to use 1.1. I will lower the constraint to 1.0 which will also allow 1.1 but does not force everybody to use 1.1

Thanks for asking

We want to allow Dragonfly 1.1 - not only 1.0.x - and still want to be sure
to use at least 1.0.7 since this includes a security fix.
@tvdeyen
Copy link
Member Author

tvdeyen commented Oct 16, 2017

@robinboening updated it so it allows 1.1 but makes sure that we still use at least 1.0.7 - since this includes an important security fix.

@tvdeyen tvdeyen requested a review from robinboening October 16, 2017 16:05
@robinboening
Copy link
Contributor

Nice. Yeah, I think its good to allow the transition of the versions as soft as possible, just because we can :)

@tvdeyen tvdeyen merged commit 321a948 into AlchemyCMS:master Oct 16, 2017
@tvdeyen tvdeyen deleted the update-dragonfly branch October 16, 2017 16:19
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