Skip to content

Updated Poison dependency#9

Open
tompave wants to merge 1 commit into
hassox:masterfrom
tompave:upgrade_poison
Open

Updated Poison dependency#9
tompave wants to merge 1 commit into
hassox:masterfrom
tompave:upgrade_poison

Conversation

@tompave

@tompave tompave commented Nov 1, 2016

Copy link
Copy Markdown

Hi, I am in the process of integrating server-side react rendering with a Phoenix application, and I couldn't use the latest release on hex.pm because of a dependency version conflict with Phoenix.

I'm following this guide:
http://blog.overstuffedgorilla.com/render-react-with-phoenix/
(and thanks for the great writeup!)

This PR updates the poison dependency to make the library compatible with Phoenix 1.2.1
This also silences all the "unsafe variable assignment" warnings.

… 1.2.1

This also silences all the "unsafe variable assignment" warnings.
@tompave

tompave commented Nov 12, 2016

Copy link
Copy Markdown
Author

Hi @hassox, have you had a chance to look at this?

I've been using my fork that includes these changes the the ones from here and everything seems to work well.

@keown

keown commented Dec 27, 2016

Copy link
Copy Markdown

+1

1 similar comment
@gogocurtis

gogocurtis commented Dec 29, 2016

Copy link
Copy Markdown

+1

@gogocurtis

Copy link
Copy Markdown

It would also be appropriate to bump the minor version.

@tompave

tompave commented Jan 19, 2017

Copy link
Copy Markdown
Author

bump

@tompave

tompave commented Jan 19, 2017

Copy link
Copy Markdown
Author

It would also be appropriate to bump the minor version.

Yes. I think it should happen after the PR is merged though.

@Flaque

Flaque commented Feb 7, 2017

Copy link
Copy Markdown

Bump

@Tamiyadd

Copy link
Copy Markdown

+1

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.

5 participants