A typo in Horizon and source code in docs.o.o/developer/horizon

Bug #1619847 reported by Ian Y. Choi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Low
tamil vanan

Bug Description

In http://git.openstack.org/cgit/openstack/horizon/tree/openstack_dashboard/dashboards/admin/floating_ips/views.py#n119 ,

Line 119, "msg = _('Unknow resource type for detail API.')"
: "Unknow" is typo. It should be changed to "Unknown".

I am not sure whether the source code in following developer documentation page automatically links to the original source code or not. If not, please consider it.

In http://docs.openstack.org/developer/horizon/_modules/openstack_dashboard/dashboards/admin/floating_ips/views.html ,

" msg = _('Unknow resource type for detail API.') "

tamil vanan (tamilhce)
Changed in horizon:
assignee: nobody → tamil vanan (tamilhce)
status: New → 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/365264

Ian Y. Choi (ianychoi)
summary: - A typo in Horizon and source code in developer.o.o doc
+ A typo in Horizon and source code in docs.o.o
summary: - A typo in Horizon and source code in docs.o.o
+ A typo in Horizon and source code in docs.o.o/developer/horizon
Revision history for this message
tamil vanan (tamilhce) wrote : Re: [Bug 1619847] Re: A typo in Horizon and source code in docs.o.o

Hi Ian,

Regarding the bug 1619847,
As per my understanding we can't change http://docs.openstack.org/
developer/horizon/_modules/openstack_dashboard/
dashboards/admin/floating_ips/views.html directly.

The fix in
 http://git.openstack.org/cgit/openstack/horizon/tree/openstack_dashboard/
dashboards/admin/floating_ips/views.py#n119

will change the http://docs.openstack.org/developer/horizon/_modules/
openstack_dashboard/dashboards/admin/floating_ips/views.html accordingly.

Regards,
Tamil

  ,

On Sun, Sep 4, 2016 at 12:28 PM, Ian Y. Choi <email address hidden> wrote:

> ** Summary changed:
>
> - A typo in Horizon and source code in developer.o.o doc
> + A typo in Horizon and source code in docs.o.o
>
> ** Summary changed:
>
> - A typo in Horizon and source code in docs.o.o
> + A typo in Horizon and source code in docs.o.o/developer/horizon
>
> --
> You received this bug notification because you are a bug assignee.
> https://bugs.launchpad.net/bugs/1619847
>
> Title:
> A typo in Horizon and source code in docs.o.o/developer/horizon
>
> Status in OpenStack Dashboard (Horizon):
> In Progress
>
> Bug description:
> In
> http://git.openstack.org/cgit/openstack/horizon/tree/
> openstack_dashboard/dashboards/admin/floating_ips/views.py#n119
> ,
>
> Line 119, "msg = _('Unknow resource type for detail API.')"
> : "Unknow" is typo. It should be changed to "Unknown".
>
> I am not sure whether the source code in following developer
> documentation page automatically links to the original source code or
> not. If not, please consider it.
>
> In
> http://docs.openstack.org/developer/horizon/_modules/
> openstack_dashboard/dashboards/admin/floating_ips/views.html
> ,
>
> " msg = _('Unknow resource type for detail API.') "
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/horizon/+bug/1619847/+subscriptions
>

Revision history for this message
Ian Y. Choi (ianychoi) wrote :

Hello Tamil,

I think your saying is correct. Thanks for letting me know this.

Just I have noticed that the summary in Launchpad because the document
URL is docs.o.o/developer, not developer.o.o.
That's why I have changed the summary title :)

With many thanks,

/Ian

tamil vanan wrote on 9/4/2016 4:26 PM:
> Hi Ian,
>
> Regarding the bug 1619847,
> As per my understanding we can't change
> http://docs.openstack.org/developer/horizon/_modules/openstack_dashboard/dashboards/admin/floating_ips/views.html
> <http://docs.openstack.org/developer/horizon/_modules/openstack_dashboard/dashboards/admin/floating_ips/views.html>
> directly.
>
> The fix in
> http://git.openstack.org/cgit/openstack/horizon/tree/openstack_dashboard/dashboards/admin/floating_ips/views.py#n119
> <http://git.openstack.org/cgit/openstack/horizon/tree/openstack_dashboard/dashboards/admin/floating_ips/views.py#n119>
>
> will change the
> http://docs.openstack.org/developer/horizon/_modules/openstack_dashboard/dashboards/admin/floating_ips/views.html
> <http://docs.openstack.org/developer/horizon/_modules/openstack_dashboard/dashboards/admin/floating_ips/views.html>
> accordingly.
>
> Regards,
> Tamil
>
> ,
>
> On Sun, Sep 4, 2016 at 12:28 PM, Ian Y. Choi <<email address hidden>
> <mailto:<email address hidden>>> wrote:
>
> ** Summary changed:
>
> - A typo in Horizon and source code in developer.o.o doc
> + A typo in Horizon and source code in docs.o.o
>
> ** Summary changed:
>
> - A typo in Horizon and source code in docs.o.o
> + A typo in Horizon and source code in docs.o.o/developer/horizon
>
> --
> You received this bug notification because you are a bug assignee.
> https://bugs.launchpad.net/bugs/1619847
> <https://bugs.launchpad.net/bugs/1619847>
>
> Title:
> A typo in Horizon and source code in docs.o.o/developer/horizon
>
> Status in OpenStack Dashboard (Horizon):
> In Progress
>
> Bug description:
> In
> http://git.openstack.org/cgit/openstack/horizon/tree/openstack_dashboard/dashboards/admin/floating_ips/views.py#n119
> <http://git.openstack.org/cgit/openstack/horizon/tree/openstack_dashboard/dashboards/admin/floating_ips/views.py#n119>
> ,
>
> Line 119, "msg = _('Unknow resource type for detail API.')"
> : "Unknow" is typo. It should be changed to "Unknown".
>
> I am not sure whether the source code in following developer
> documentation page automatically links to the original source
> code or
> not. If not, please consider it.
>
> In
> http://docs.openstack.org/developer/horizon/_modules/openstack_dashboard/dashboards/admin/floating_ips/views.html
> <http://docs.openstack.org/developer/horizon/_modules/openstack_dashboard/dashboards/admin/floating_ips/views.html>
> ,
>
> " msg = _('Unknow resource type for detail API.') "
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/horizon/+bug/1619847/+subscriptions
> <https://bugs.launchpad.net/horizon/+bug/1619847/+subscriptions>
>
>

Changed in horizon:
milestone: none → newton-rc1
importance: Undecided → Low
Revision history for this message
tamil vanan (tamilhce) wrote : Re: [Bug 1619847] Re: A typo in Horizon and source code in docs.o.o/developer/horizon

Regarding the bug 1619847,
As per my understanding we can't change http://docs.
openstack.org/developer/horizon/_modules/openstack_
dashboard/dashboards/admin/floating_ips/views.html directly.

The fix in
 http://git.openstack.org/cgit/openstack/horizon/tree/op
enstack_dashboard/dashboards/admin/floating_ips/views.py#n119

will change the http://docs.openstack.org/de
veloper/horizon/_modules/openstack_dashboard/dashboards/
admin/floating_ips/views.html accordingly.

Regards,
Tamil

On Mon, Sep 5, 2016 at 2:40 PM, Rob Cresswell <email address hidden> wrote:

> ** Changed in: horizon
> Milestone: None => newton-rc1
>
> ** Changed in: horizon
> Importance: Undecided => Low
>
> --
> You received this bug notification because you are a bug assignee.
> https://bugs.launchpad.net/bugs/1619847
>
> Title:
> A typo in Horizon and source code in docs.o.o/developer/horizon
>
> Status in OpenStack Dashboard (Horizon):
> In Progress
>
> Bug description:
> In
> http://git.openstack.org/cgit/openstack/horizon/tree/
> openstack_dashboard/dashboards/admin/floating_ips/views.py#n119
> ,
>
> Line 119, "msg = _('Unknow resource type for detail API.')"
> : "Unknow" is typo. It should be changed to "Unknown".
>
> I am not sure whether the source code in following developer
> documentation page automatically links to the original source code or
> not. If not, please consider it.
>
> In
> http://docs.openstack.org/developer/horizon/_modules/
> openstack_dashboard/dashboards/admin/floating_ips/views.html
> ,
>
> " msg = _('Unknow resource type for detail API.') "
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/horizon/+bug/1619847/+subscriptions
>

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

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

commit ed3a382dee6806bac375861d272fc6499ca72b79
Author: tamilhce <email address hidden>
Date: Sat Sep 3 15:33:45 2016 +0000

    Fix for typo in Horizon

    The horizon views.py had a typo in error msg
    Closes-Bug: #1619847

    Change-Id: I5d41cb3318ca7a54ec973404cb03afdc3453c021

Changed in horizon:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/horizon 10.0.0.0rc1

This issue was fixed in the openstack/horizon 10.0.0.0rc1 release candidate.

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.