Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • A administrate
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 96
    • Issues 96
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 32
    • Merge requests 32
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • thoughtbot, inc.
  • administrate
  • Merge requests
  • !364

Generate SHOW_PAGE_ATTRIBUTES as an explicit array

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge gw-show-array into master Dec 22, 2015
  • Overview 3
  • Commits 1
  • Pipelines 0
  • Changes 3

Created by: gracewashere

Closes #130 (closed)

Problem:

The dashboard generator specifies:

SHOW_PAGE_ATTRIBUTES = ATTRIBUTE_TYPES.keys

This means that the order of the show page attributes is dependent on the order of keys in the ATTRIBUTE_TYPES hash.

Ruby hashes happen to be ordered, but we shouldn't make an assumption that hashes are ordered because that is not a property that hashes are designed to have.

Solution:

Because we assume that users will be editing SHOW_PAGE_ATTRIBUTES to customize the contents and order of show page attributes, we'll generate an explicit array for them to modify.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: gw-show-array