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

Disable all fields on published editions #303

Merged
merged 6 commits into from Nov 21, 2014
Merged

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Nov 20, 2014

  • Test for fields that remain editable on published editions
  • Apply test to integration tests for each edition type
  • Fix all test failures (Videos, Campaigns, Business support finder and summary fields)

https://www.agileplannerapp.com/boards/173808/cards/8723

Example failing test

screen shot 2014-11-20 at 15 41 35

fofr added 6 commits Nov 20, 2014
Match the rest of the form
* When adding new fields it’s common to forget to disable them in
archive and preview, create a helper to apply a test to each edition
format
* Test all form fields for disabled-ness
* Fix business areas which weren’t disabled
* Fix missing disabled fields on licences and videos
vinayvinay added a commit that referenced this pull request Nov 21, 2014
Disable all fields on published editions
@vinayvinay vinayvinay merged commit a1cbd93 into master Nov 21, 2014
1 check passed
1 check passed
default "Build #449 succeeded on Jenkins"
Details
@vinayvinay vinayvinay deleted the fix-disabled-summary-fields branch Nov 21, 2014
@vinayvinay
Copy link
Contributor

@vinayvinay vinayvinay commented Nov 21, 2014

🍓

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.