Skip to content

Theme by microsite - #4107

Closed
akktis wants to merge 1 commit into
openedx:masterfrom
akktis:master
Closed

Theme by microsite#4107
akktis wants to merge 1 commit into
openedx:masterfrom
akktis:master

Conversation

@akktis

@akktis akktis commented Jun 16, 2014

Copy link
Copy Markdown

Possibility to have one theme per microsite
Please review carefully i'm quit new in the edx code

Possibility to have one theme per microsite
@singingwolfboy

Copy link
Copy Markdown
Contributor

@papipsycho Thanks for the contribution! Before it can be reviewed, though, we need you to submit a signed contributor's agreement, as documented in the CONTRIBUTING file in the repository.

@akktis

akktis commented Jun 16, 2014

Copy link
Copy Markdown
Author

Ok i'll send it tomorow :)

@mhoeber

mhoeber commented Jun 25, 2014

Copy link
Copy Markdown
Contributor

backlogged DOC-628

@jtauber

jtauber commented Jun 26, 2014

Copy link
Copy Markdown
Contributor

@felipemontoya I'd love your input on this too, given your work on #3495

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably this is a duplicated line

@felipemontoya

Copy link
Copy Markdown
Member

@papipsycho I am not sure what you are trying to accomplish here. You can already override the templates in the microsite, using the edxmako.shortcuts.render_to_response.

Let me know if I'm missing something

@akktis

akktis commented Jun 29, 2014

Copy link
Copy Markdown
Author

Actually i'm trying to find a way to have one theme per subdomain,

like comp1.edx.org = theme1,
comp2.edx.org = theme2
comp3.edx.org = theme1

but actually my modification doens't work but if you have any idea your welcome

@jtauber

jtauber commented Jul 10, 2014

Copy link
Copy Markdown
Contributor

@papipsycho what you are trying to do is exactly what microsites already achieves so I'm going to close this pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

open-source-contribution PR author is not from Axim or 2U

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants