yahoo-eng-team team mailing list archive

[OpenStack Infra <1625566@xxxxxxxxxxxxxxxxxx>]

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

commit 44afeac9d4f14c7c9b0b2685bbdedbf545101562
Author: Kenji Ishii <ken-ishii@xxxxxxxxxxxxx>
Date:   Wed Sep 21 14:10:10 2016 +0900

    [Trivial]remove unnecessary commna
    
    Change-Id: Ie174e707adbc9ad668a2ad516e2cda11a8646b49
    Closes-Bug: #1625566


** 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/1625566

Title:
  Tutorial: Adding a complex action to a table

Status in OpenStack Dashboard (Horizon):
  Fix Released

Bug description:
  urlpatterns = [,
      url(r'^$',
          views.IndexView.as_view(), name='index'),
      url(r'^(?P<instance_id>[^/]+)/create_snapshot/$',
          views.CreateSnapshotView.as_view(),
          name='create_snapshot'),
  ]

  Still gives an error for me? I removed the comma as such:

  urlpatterns = [
      url(r'^$',
          views.IndexView.as_view(), name='index'),
      url(r'^(?P<instance_id>[^/]+)/create_snapshot/$',
          views.CreateSnapshotView.as_view(),
          name='create_snapshot'),
  ]

  And it worked!

  PS. This is a double thought it may as well be it's own report because
  where I posted the double wasn't precisely solid imo, feel free to
  correct me otherwise DS.

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