Tags on Draft Posts #351

Closed
patrickdavey opened this Issue Oct 16, 2013 · 4 comments

Comments

Projects
None yet
3 participants
Contributor

patrickdavey commented Oct 16, 2013

Hi there,

I was just listing out some tags on our blog and realised that the all_tags method doesn't restrict to live posts.

https://github.com/refinery/refinerycms-blog/blob/master/app/helpers/refinery/blog/controller_helper.rb#l21-23

I'm just wondering if this method should be changed to only show tags for published posts. I'm happy to submit a pull request depending on what you think the functionality should be.

Owner

parndt commented Oct 16, 2013

you mean find_tags?

Contributor

patrickdavey commented Oct 16, 2013

Yes - sorry - typo - I do mean the find_tags method yes.

Owner

parndt commented Oct 16, 2013

It probably should restrict to live tags.

Owner

ugisozols commented Oct 16, 2013

👍 for live tags only.

ugisozols closed this in #352 Oct 18, 2013

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