Skip to content
This repository has been archived by the owner on Feb 4, 2019. It is now read-only.

Promote ApacheCon 2018 #213

Merged
merged 1 commit into from
Apr 10, 2018
Merged

Promote ApacheCon 2018 #213

merged 1 commit into from
Apr 10, 2018

Conversation

nacx
Copy link
Member

@nacx nacx commented Apr 10, 2018

No description provided.

@jclouds-commentator
Copy link

@andreaturli
Copy link
Contributor

Thanks @nacx, I've started a PR to promote apachacon but you've been quicker. Actually I ended up trying to revamp the entire website, bumping jekyll version and so on ... let's see what I can do

@jclouds-mirror jclouds-mirror merged commit 46788b0 into jclouds:master Apr 10, 2018
@nacx nacx deleted the apachecon2018 branch April 10, 2018 18:12
@nacx
Copy link
Member Author

nacx commented Apr 10, 2018

Cool @andreaturli! I've already merged the PR as it is good and it should not block you on the Jekyll bump.

@@ -59,7 +59,7 @@
<li><a href="http://www.apache.org/licenses/">License</a></li>
<li><a href="http://www.apache.org/foundation/contributing.html">Donations</a></li>
<li><a href="http://www.apache.org/foundation/sponsorship.html">Sponsorship</a></li>
<li><a href="http://www.apache.org/security/">Report a Vulnerability</a></li>
<li><a href="http://www.apache.org/security/">Security</a></li>
Copy link
Member

Choose a reason for hiding this comment

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

@nacx Unrelated change - is this an update to the recommended links?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, it is unrelated but trivial so I just included it to match the navigation link policy: https://www.apache.org/foundation/marks/pmcs#navigation

@@ -6,8 +6,11 @@
<body>
{% include navbar.html %}
<div id="home-content" class="container">


{% comment %}
Copy link
Member

Choose a reason for hiding this comment

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

Is this intended as a TODO to remove after the event?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes. I've preferred to leave it like that in case we want to add it back we already know where in the page it looked ok.

@demobox
Copy link
Member

demobox commented Apr 11, 2018

Thanks for taking care of this, @nacx!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants