Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Edit supplier declaration #135

Merged
merged 8 commits into from
Oct 14, 2015
Merged

Edit supplier declaration #135

merged 8 commits into from
Oct 14, 2015

Conversation

robyoung
Copy link
Contributor

@robyoung robyoung commented Oct 9, 2015

This adds the ability for a CCS Sourcing team member to view and edit G-Cloud 7 supplier declarations. In the supplier listing the only link they will see alongside a supplier is 'G-Cloud 7 declaration'. This will
take them to the G-Cloud 7 supplier declaration overview where they can edit each section. This uses the new ContentLoader but hard codes the G-Cloud 7 link in the supplier list as it is the only framework they can edit at the moment.

Note that this does no validation, it is assumed that the CCS Sourcing team know the rules.

screen shot 2015-10-09 at 16 45 54

screen shot 2015-10-09 at 16 46 02

Relates to

Signoff

Depends on

@robyoung robyoung force-pushed the edit-supplier-declaration branch 2 times, most recently from c303bec to 5ee5aa2 Compare October 12, 2015 17:27
Only change required is around the ContentLoader
This adds the ability for a CCS Sourcing team member to view and edit
G-Cloud 7 supplier declarations. In the supplier listing the only link
they will see alongside a supplier is 'G-Cloud 7 declaration'. This will
take them to the G-Cloud 7 supplier declaration overview where they can
edit each section. This uses the new ContentLoader but hard codes the
G-Cloud 7 link in the supplier list as it is the only framework they can
edit at the moment.

Note that this does no validation, it is assumed that the CCS Sourcing
team know the rules.
Only show the search fields that a user can interact with on the index
page. For example, admin-ccs-sourcing can only search for suppliers by
name so that is the only form they can see.
It's just not _that_ special.
@quis
Copy link
Contributor

quis commented Oct 14, 2015

Could do with some basic tests for the routes I think.

@quis
Copy link
Contributor

quis commented Oct 14, 2015

👍

robyoung added a commit that referenced this pull request Oct 14, 2015
@robyoung robyoung merged commit 614cc06 into master Oct 14, 2015
@robyoung robyoung deleted the edit-supplier-declaration branch October 14, 2015 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants