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

Bug #956760 reported by Lukas Ocilka on 2012-03-16
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Gabriel Hurley

Bug Description

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

Andy Chong (andycjw) wrote :

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

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.

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

Lukas Ocilka (lukas-ocilka) wrote :

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

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
Devin Carlen (devcamcar) on 2012-03-19
Changed in horizon:
milestone: none → essex-rc1

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) on 2012-03-20
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2012-04-05
Changed in horizon:
milestone: essex-rc1 → 2012.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers