Split attachment UI for policy advisory groups

Add tests for policy advisory groups to attachments controller, tabs to
the policy advisory groups form, and swap out the attachments fields for
the markdown reference.
Stop accepting nested attributes for attachments

Now that all attachment editing is done via the admin attachments
controller we no longer need to accept nested attributes. This removes
the nested attributes and all tests which was checking that behaviour.