Remove or expand the useless docstrings in tuskar_ui.api

Bug #1342156 reported by Radomir Dopieralski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tuskar-ui
Fix Committed
Undecided
Tzu-Mainn Chen

Bug Description

The docstrings on the api methods don't really do much more than repeat what the function signature already says. Remove the obvious ones, and expand the descriptions on ones that are not obvious.

We will do it gradually.

Changed in tuskar-ui:
assignee: nobody → Radomir Dopieralski (thesheep)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tuskar-ui (master)

Reviewed: https://review.openstack.org/107052
Committed: https://git.openstack.org/cgit/openstack/tuskar-ui/commit/?id=1865cf987a0db83f73fbd5873460c614af879e0a
Submitter: Jenkins
Branch: master

commit 1865cf987a0db83f73fbd5873460c614af879e0a
Author: Radomir Dopieralski <email address hidden>
Date: Tue Jul 15 15:34:45 2014 +0200

    Add a "driver" field to the Register Nodes dialog

    The choice of the driver determines the visibility of the
    driver-dependent fields.

    Partial-bug: #1342156
    Change-Id: I7f68b47c46c12e7f2bf91e475a62e4740ecc3946

Changed in tuskar-ui:
assignee: Radomir Dopieralski (thesheep) → Tzu-Mainn Chen (tzumainn)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/107883
Committed: https://git.openstack.org/cgit/openstack/tuskar-ui/commit/?id=a6ea6bb4bfec47d5b24d35b57b5e66d253d56263
Submitter: Jenkins
Branch: master

commit a6ea6bb4bfec47d5b24d35b57b5e66d253d56263
Author: Tzu-Mainn Chen <email address hidden>
Date: Fri Jul 18 04:45:17 2014 +0200

    Update ironic properties to match REST API doc

    The referenced doc is here:

    http://docs.openstack.org/developer/ironic/webapi/v1.html

    This fix should make it far more explicit what units are needed
    for various node properties. Note that Ironic and Nova BareMetal
    now use the exact same property names.

    Closes-bug: #1342156
    Change-Id: I024c29d0ab648ed5db000ddb6d0ffaa24e8805b8

Changed in tuskar-ui:
status: In Progress → Fix Committed
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.