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

Mark generated attributes as safe #22

Merged
merged 1 commit into from May 24, 2016
Merged

Mark generated attributes as safe #22

merged 1 commit into from May 24, 2016

Conversation

iamvery
Copy link
Owner

@iamvery iamvery commented May 24, 2016

In order for ratchet to work with Phoenix, generated attributes must be
flagged as "raw". Otherwise they would be sanitized during rendering.
This has the unfortunately effect of coupling ratchet to phoenix... I
think we'll just be OK with this for now given that phoenix is probably
its only use case. Eventually we'll need to provide some sort of hook
that performs the special behavior in the presence of phoenix.

In order for ratchet to work with Phoenix, generated attributes must be
flagged as "raw". Otherwise they would be sanitized during rendering.
This has the unfortunately effect of coupling ratchet to phoenix... I
think we'll just be OK with this for now given that phoenix is probably
its only use case. Eventually we'll need to provide some sort of hook
that performs the special behavior in the presence of phoenix.
@iamvery iamvery merged commit 853963e into master May 24, 2016
@iamvery iamvery deleted the phoenix branch May 24, 2016 21:11
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

1 participant