← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1626858] Re: The explanation of how to use the customization_module may confuse users.

 

Reviewed:  https://review.openstack.org/375275
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=07185eca288095bbf8f0508e68f2ddff43dfb534
Submitter: Jenkins
Branch:    master

commit 07185eca288095bbf8f0508e68f2ddff43dfb534
Author: Kenji Ishii <ken-ishii@xxxxxxxxxxxxx>
Date:   Fri Sep 23 15:52:07 2016 +0900

    Fix the exapmle of how to use customization_module
    
    If a user use current example[1] as is, other attributes that
    HORIZON_CONFIG have are eliminated. As a result, some unexpected
    behavior may occur. So, this patch modify the exapmle.
    
    [1]:http://docs.openstack.org/developer/horizon/topics/customizing.html
    
    Change-Id: I022a73e6da6ad9b8ae1f807838cf29efedf7e1ef
    Closes-Bug: #1626858


** Changed in: horizon
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1626858

Title:
  The explanation of how to use the customization_module may confuse
  users.

Status in OpenStack Dashboard (Horizon):
  Fix Released

Bug description:
  The explanation of how to use costomization_module is writtend by
  http://docs.openstack.org/developer/horizon/topics/customizing.html
  #horizon-customization-module-overrides.

  And this sample code is below.

  HORIZON_CONFIG = {
      "customization_module": "my_project.overrides"
  }

  However, if a user do so, other attributes that HORIZON_CONFIG have are eliminated.
  In latest horizon, it would caues circular redirect loop.
  So, Avoiding confusing users, we should indicate more safe way like below.

  HORIZON_CONFIG["customization_module"] = "my_project.overrides"

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1626858/+subscriptions


References