some 'url' in "images/urls.py" are repeated and redundant

Bug #1626566 reported by qiaomin032
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Won't Fix
Low
Unassigned

Bug Description

The 'image_urls' and 'snapshot_urls' in 'images/urls.py' are repeated, we can refactor the code and remove some redundant code.

Changed in horizon:
assignee: nobody → qiaomin032 (chen-qiaomin)
status: New → In Progress
Changed in horizon:
importance: Undecided → Low
milestone: none → next
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/410099

Changed in horizon:
assignee: qiaomin032 (chen-qiaomin) → Yakup Adakli (yakupa)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by David Lyle (<email address hidden>) on branch: master
Review: https://review.openstack.org/372312
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Rob Cresswell (<email address hidden>) on branch: master
Review: https://review.openstack.org/410099
Reason: Abandoned due to inactivity

Changed in horizon:
assignee: Yakup Adakli (yakupa) → nobody
status: In Progress → Triaged
tags: added: low-hanging-fruit
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/493484

Changed in horizon:
assignee: nobody → Artem Tiumentcev (darland-maik)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by Artem Tiumentcev (<email address hidden>) on branch: master
Review: https://review.openstack.org/493484

Revision history for this message
Akihiro Motoki (amotoki) wrote :

As discussed in the review above, we decided to keep the current pattern.

Changed in horizon:
milestone: next → none
status: In Progress → Won't Fix
assignee: Artem Tiumentcev (darland-maik) → nobody
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.