Skip to content
This repository has been archived by the owner on Jul 3, 2018. It is now read-only.

Length validations #173

Merged
merged 4 commits into from
Apr 14, 2014
Merged

Length validations #173

merged 4 commits into from
Apr 14, 2014

Conversation

robmckinnon
Copy link
Contributor

  • Additional length validations defined by Kellie

@jenkins-moj
Copy link

Test PASSed.
Refer to this link for build results: http://jenkins.dsd.io/job/civilclaims-accelerated-test-pulls/323/

whoojemaflip
whoojemaflip reviewed Apr 14, 2014
View reviewed changes
spec/models/claimant_contact_spec.rb
@@ -13,13 +13,13 @@ def claimant_contact options={}

describe "company name" do
context "when over 60 characters" do
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

40

robmckinnon pushed a commit that referenced this pull request Apr 14, 2014
Merge pull request #173 from ministryofjustice/length_validations
09e17e4 
Length validations
@robmckinnon robmckinnon merged commit 09e17e4 into master Apr 14, 2014
@robmckinnon robmckinnon deleted the length_validations branch April 14, 2014 14:56
@jenkins-moj
Copy link

Test PASSed.
Refer to this link for build results: http://jenkins.dsd.io/job/civilclaims-accelerated-test-pulls/324/

@jenkins-moj
Copy link

Test PASSed.
Refer to this link for build results: http://jenkins.dsd.io/job/civilclaims-accelerated-test-pulls/325/

robmckinnon pushed a commit that referenced this pull request Jul 15, 2014
Merge pull request #173 from ministryofjustice/length_validations
e11c608 
Length validations
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@robmckinnon @jenkins-moj @whoojemaflip