After plugin #84

Merged
merged 2 commits into from Jan 29, 2018

Conversation

Projects
None yet
5 participants
@harold
Contributor

harold commented Mar 29, 2013

I made a simplest-possible 'After' plugin.

It exposes an after method in controllers so you can run some code after everything's generated.

It let me write this in my controller.rb, which brought joy:

after do
    File.open( "./public/sitemap.txt", "w" ) do |f|
        Dir["public/**/*.html"].each do |n|
            f.puts "http://danceliquid.com/#{n}"
        end
    end
end

Take it, if you'd like. (:

@alem0lars

This comment has been minimized.

Show comment Hide comment
@alem0lars

alem0lars Jul 18, 2013

It would be great to have this 👍

It would be great to have this 👍

@alexdean

This comment has been minimized.

Show comment Hide comment
@harold

This comment has been minimized.

Show comment Hide comment
@harold

harold Oct 19, 2013

Contributor

@alexdean (!) Too right, friend!

emot-psyduck

I have a very similar edit in my local stasis.rb ... I must have flubbed the git add on the original commit.

Artfully done, sir.

Contributor

harold commented Oct 19, 2013

@alexdean (!) Too right, friend!

emot-psyduck

I have a very similar edit in my local stasis.rb ... I must have flubbed the git add on the original commit.

Artfully done, sir.

@alexdean

This comment has been minimized.

Show comment Hide comment
@alexdean

alexdean Oct 20, 2013

@harold ✌️

@harold ✌️

@winton

This comment has been minimized.

Show comment Hide comment
@winton

winton Nov 25, 2013

Owner

Hey @harold, sorry for the late reply on this. Could you add @alexdean's change to your PR?

I generally don't merge without specs but I could write some up for this. Thanks.

Owner

winton commented Nov 25, 2013

Hey @harold, sorry for the late reply on this. Could you add @alexdean's change to your PR?

I generally don't merge without specs but I could write some up for this. Thanks.

@harold

This comment has been minimized.

Show comment Hide comment
@harold

harold Nov 26, 2013

Contributor

@winton, no problem on the late reply. Thanks for stasis, I enjoy it thoroughly.

I added the require line at the beginning to keep them in alphabetical order. (not because my plugin is the most important) 😵

I briefly wondered about how you were doing the specs. I'll have a look at what you do so I'll know what to do if I contribute more in the future.

Take care, friend.

Contributor

harold commented Nov 26, 2013

@winton, no problem on the late reply. Thanks for stasis, I enjoy it thoroughly.

I added the require line at the beginning to keep them in alphabetical order. (not because my plugin is the most important) 😵

I briefly wondered about how you were doing the specs. I'll have a look at what you do so I'll know what to do if I contribute more in the future.

Take care, friend.

@jedemenge-it

This comment has been minimized.

Show comment Hide comment
@jedemenge-it

jedemenge-it Jul 9, 2014

Hi,

why is this feature not merged? It would be nice to have it by default (without tweaking the local installation).

Cheers

Hi,

why is this feature not merged? It would be nice to have it by default (without tweaking the local installation).

Cheers

@winton winton merged commit 7f856dd into winton:master Jan 29, 2018

1 check passed

default The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment