alphagov / specialist-publisher

Publishes specialist documents on GOV.UK
https://docs.publishing.service.gov.uk/apps/specialist-publisher.html
MIT License
10 stars 7 forks source link

[CLOSED] Add International Development Funds #184

Closed benilovj closed 8 years ago

benilovj commented 8 years ago

Issue by tommyp Thursday 24 July 2014 at 15:30 GMT Originally opened as https://github.com/alphagov/specialist-publisher/pull/184


This PR adds International Development Funds to Specialist Publisher for DFID Editors. There's a lot going on in this, but it boils doing to

At this point, you can save IDFs, but to publish them the next commit includes the observers with the rest being a reshuffle of a step and then testing creating, editing and publishing of International Development Funds.

Ticket


tommyp included the following code: https://github.com/alphagov/specialist-publisher/pull/184/commits

benilovj commented 8 years ago

Comment by rboulton Friday 25 July 2014 at 07:46 GMT


There was some discussion in slack about changing "idf_sector" to "development_sector" or similar. It seems like this might be a good time to do that, if it's going to be done.

benilovj commented 8 years ago

Comment by Themitchell Friday 25 July 2014 at 09:53 GMT


Added a couple of comments regarding code duplication but otherwise LGTM +1

benilovj commented 8 years ago

Comment by Themitchell Friday 25 July 2014 at 12:42 GMT


:+1: Good work!