Project

Profile

Help

Issue #3856

closed

In the bindings, href docstring is ambiguous and sometimes has the wrong type

Added by daviddavis over 6 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
-
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
API Bindings
Sprint:
Sprint 42
Quarter:

Description

Looking at the API bindings:

class RepositoriesApi(object):

    def repositories_versions_create(self, href, data, **kwargs):  # noqa: E501
        """repositories_versions_create  # noqa: E501

        :param str href: A relative URI for the resource. (required)
        :param RepositoryVersion data: (required)
        :return: AsnycOperationResponse
                 If the method is called asynchronously,
                 returns the request thread.
        """
        pass

    def repositories_versions_delete(self, href, **kwargs):  # noqa: E501                               
        """repositories_versions_delete  # noqa: E501                                                   

        :param int href: A relative URI for the resource. (required)                                    
        :return: AsnycOperationResponse
                 If the method is called asynchronously,
                 returns the request thread.
        """

It's not totally clear what href means. Also, for repositories_versions_delete() it says the href type is an int.

Actions #1

Updated by daviddavis over 6 years ago

  • Tags API Bindings added
Actions #2

Updated by daviddavis over 6 years ago

  • Subject changed from In the bindings, href docstring is ambiguous and sometimes wrong has the wrong type to In the bindings, href docstring is ambiguous and sometimes has the wrong type
Actions #3

Updated by CodeHeeler over 6 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 40
Actions #4

Updated by rchan over 6 years ago

  • Sprint changed from Sprint 40 to Sprint 41
Actions #5

Updated by dkliban@redhat.com about 6 years ago

  • Status changed from NEW to POST
  • Assignee set to dkliban@redhat.com
Actions #6

Updated by rchan about 6 years ago

  • Sprint changed from Sprint 41 to Sprint 42

Added by dkliban@redhat.com about 6 years ago

Revision b9399bcd | View on GitHub

Problem: OpenAPI schema doesn't use URIs to identify resources

Solution: Create a custom OpenAPI generator that refers to resources by their URI.

closes #3856 https://pulp.plan.io/issues/3856

closes #3851 https://pulp.plan.io/issues/3851

Added by dkliban@redhat.com about 6 years ago

Revision b9399bcd | View on GitHub

Problem: OpenAPI schema doesn't use URIs to identify resources

Solution: Create a custom OpenAPI generator that refers to resources by their URI.

closes #3856 https://pulp.plan.io/issues/3856

closes #3851 https://pulp.plan.io/issues/3851

Actions #7

Updated by dkliban@redhat.com about 6 years ago

  • Status changed from POST to MODIFIED
Actions #8

Updated by bmbouter over 5 years ago

  • Tags deleted (Pulp 3)
Actions #9

Updated by bmbouter almost 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF