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

Issue #840 - Sort building detail fields alphabetically by default. #864

Merged
merged 1 commit into from
Mar 22, 2016

Conversation

nicholasserra
Copy link
Contributor

What's this PR do?

Sort the building detail fields alphabetically if the user hasn't set any field settings.

What are the relevant tickets?

Refs #840

Definition of Done:

  • Is there appropriate test coverage? (e.g. ChefSpec, Mocha/Chai, Python, etc.)
  • Does this PR require a Selenium test? (e.g. Browser-specific bugs or complicated UI bugs)
  • Does this PR require a regression test? All fixes require a regression test.
  • Does this add new dependencies? If so, does PIP, npm, bower requirements need to be updated?

@nicholasserra nicholasserra changed the title Sort building detail fields alphabetically by default. Issue #840 - Sort building detail fields alphabetically by default. Mar 21, 2016
mmclark added a commit that referenced this pull request Mar 22, 2016
…order

Issue #840 - Sort building detail fields alphabetically by default.
@mmclark mmclark merged commit 4a9016a into develop Mar 22, 2016
@mmclark mmclark deleted the 840-building-detail-field-order branch March 22, 2016 14:46
@mmclark
Copy link
Contributor

mmclark commented Mar 22, 2016

Loaded on seedtest

@RDmitchell RDmitchell added this to the 1.5 Release (May 31, 2016) milestone May 11, 2016
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.

3 participants