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

WIP: custom splash page for pangeo.pydata.org #141

Merged
merged 3 commits into from
Mar 8, 2018
Merged

WIP: custom splash page for pangeo.pydata.org #141

merged 3 commits into from
Mar 8, 2018

Conversation

jhamman
Copy link
Member

@jhamman jhamman commented Mar 6, 2018

I've put together a few modifications that should alter the splash page for pangeo.pydata.org. This is not working just yet but I suspect I'm close enough that @jacobtomlinson can point me in the right direction.

xref: #78 and https://github.com/pangeo-data/pangeo-custom-jupyterhub-templates

cc @jacobtomlinson and @mrocklin

@jacobtomlinson
Copy link
Member

I wouldn't worry about disabling the prepuller. I only did this in ours because I wanted to do it a custom way. My way has actually made it's way upstream now so I could probably remove this too.

I'm surprised it's not working, your config looks the same as mine.

@jhamman
Copy link
Member Author

jhamman commented Mar 6, 2018

This is working now. We need to update the CSS color scheme and I need to fix a typo in the github issues link but otherwise this should be good to go. This is live now and looks like:
pangeo_splash

@kmpaul
Copy link

kmpaul commented Mar 6, 2018

@jhamman This looks great! Nice!

@rabernat
Copy link
Member

rabernat commented Mar 6, 2018

You rock Joe!

@jhamman
Copy link
Member Author

jhamman commented Mar 6, 2018

To be clear, I stole basically everything here from @jacobtomlinson and Jade.

@mrocklin
Copy link
Member

mrocklin commented Mar 6, 2018 via email

@jacobtomlinson
Copy link
Member

To be clear I stole it from here and made it orange.

But always happy to take the praise 😄.

@@ -26,6 +26,7 @@ rbac:

hub:
extraConfig: |
c.JupyterHub.log_level = 'DEBUG'
Copy link
Member

Choose a reason for hiding this comment

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

Do we want to revert this before merging?

@mrocklin
Copy link
Member

mrocklin commented Mar 8, 2018

Is there anything left to do here or can this be merged?

@jhamman
Copy link
Member Author

jhamman commented Mar 8, 2018

I'll clean this up a bit before merging.

@jhamman jhamman merged commit c3ed69e into pangeo-data:master Mar 8, 2018
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