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

JBIDE-16982 - Remove old blog entries, support for draft articles #72

Merged
merged 1 commit into from
Apr 7, 2014

Conversation

xcoulon
Copy link
Member

@xcoulon xcoulon commented Apr 4, 2014

Removed old blog entries
Added a banner on articles whose date is post today
Added a link to old blog at bottom of all /blog pages
Added a message on /blog to handle the case where there's no published article yet

@maxandersen
Copy link
Member

created a blog dates 2014-04-10 and it showed up on http://localhost:4242/blog/

@maxandersen
Copy link
Member

also in the rss feed so seems its not filtered out.

.center
Looking for older articles ? See the
%a{:href=>"https://community.jboss.org/en/tools?view=blog"}< Archived entries.
Copy link
Member

Choose a reason for hiding this comment

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

that url seem to not link to blog page.

i thnk the proper url is https://community.jboss.org/en/tools/blog

@maxandersen
Copy link
Member

Draft "marker" looks good - but Latests Post on frontpage does not show them. Is that intended ?

.alert.alert-danger
%button.close{:type=>"button", "data-dismiss"=>"alert"} &times;
%strong Warning!
This article will be published on the next site build after
Copy link
Member

Choose a reason for hiding this comment

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

Minor nit:
"This article publish date is in the future. It will NOT be published before the next site build after"

Removed old blog entries
Added a banner on articles whose date is post today
Added a link to old blog at bottom of all /blog pages
Added a message on /blog to handle the case where there's no published article yet
@xcoulon xcoulon merged commit 03d1769 into jbosstools:master Apr 7, 2014
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

2 participants