Remove template that appears leftover from namespacing updates. #1928

Merged
merged 1 commit into from Sep 6, 2012

Projects

None yet

3 participants

@JDutil
JDutil commented Sep 5, 2012

Getting to know the Refinery source better, and noticed what appears to be a leftover file from before namespacing. Not 100% sure if this is unused or done intentionally. After removing it the specs still pass, and it appears that if it were to actually be used there would be a missing template error trying to render the page partial that doesnt exist.

@JDutil
JDutil commented Sep 5, 2012

That would render the partial that is under the namespace located here: https://github.com/refinery/refinerycms/blob/master/pages/app/views/refinery/admin/pages/_sortable_list.html.erb which I believe is the only instance of this template required.

@parndt
Member
parndt commented Sep 5, 2012

Oh good stuff I totally missed that the /refinery/ was missing. I'll wait for Travis to report OK and then merge :-)

Thanks a lot! Keen eyes!

@ugisozols ugisozols merged commit d32d6c6 into refinery:master Sep 6, 2012

1 check passed

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