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

Django templates validation #501

Merged
merged 3 commits into from
May 28, 2018
Merged

Conversation

sneetsher
Copy link
Member

No description provided.

@assem-ch assem-ch merged commit 69ba152 into Alfanous-team:master May 28, 2018
@assem-ch
Copy link
Contributor

I tested it , its fixed , thank you @sneetsher .

Just for information Django has its own template engine, Jinja is just an inspiration from it

@muslih
Copy link
Member

muslih commented May 28, 2018

i think it's better to create references to this issue: #496 thank you very much @sneetsher for working on this 🙇

@sneetsher sneetsher changed the title Jinja-HTML template validation Django templates validation May 28, 2018
@sneetsher
Copy link
Member Author

@muslih You welcome. Btw I'm the one who broke it in the first place :).
@assem-ch good to know, I was mislead by old doc on the web.

I'm going to do quick clean up of the base templates to get familiar with the code there before jumping into #499 .

One question, Any reason behind such separation of templates, templates/ & wui/templates/wui/?

@assem-ch
Copy link
Contributor

@sneetsher templates should be merged in one place,

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.

3 participants