UX Improvements: Security Groups: Add a rule: UI issues

Bug #956760 reported by Lukas Ocilka
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Low
Gabriel Hurley

Bug Description

* "Cidr" should be "CIDR"
* "Ip protocol" should be "IP Protocol"

Revision history for this message
Andy Chong (andycjw) wrote :

can you show where you saw "Cidr" and "Ip protocol" ?

Revision history for this message
Lukas Ocilka (lukas-ocilka) wrote :

At https://cloud.suse.de/dash/32/security_groups/26/edit_rules

I'll ask our developers for their comment, maybe we just have some older version there.

Revision history for this message
Andy Chong (andycjw) wrote :

i can't use the url you gave
but from the url structure i would guess the version is not the latest one
the naming you showed is not found in the latest UI, it should have been already fixed

Revision history for this message
Lukas Ocilka (lukas-ocilka) wrote :

OK, they say it's Diablo. The latest stable release.

Revision history for this message
Andy Chong (andycjw) wrote :

oh, it's already fixed in the essex horizon

Changed in horizon:
importance: Undecided → Low
status: New → Confirmed
tags: added: essex-rc-potential
Changed in horizon:
assignee: nobody → Gabriel Hurley (gabriel-hurley)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/5495

Devin Carlen (devcamcar)
Changed in horizon:
milestone: none → essex-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/5495
Committed: http://github.com/openstack/horizon/commit/fccfacb6b5e9d136766a9e0947ec3fa596835cfa
Submitter: Jenkins
Branch: master

commit fccfacb6b5e9d136766a9e0947ec3fa596835cfa
Author: Gabriel Hurley <email address hidden>
Date: Sun Mar 18 20:57:12 2012 -0700

    Security Group Rule enhancements.

      * Corrects inconcsistent capitalization. Fixes bug 956760.
      * Adds a default value for CIDR and updates handling code. Fixes bug 956771.
      * Corrects the way SelfHandlingForm access the cleaned data. Fixes bug 958971.

    Change-Id: I66afeb4b530be350f33f63c8f9a60bd4a20e01bf

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: essex-rc1 → 2012.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.