Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Repeatable headers are rendered wrong #81

Closed
monikma opened this issue Nov 12, 2014 · 3 comments
Closed

Repeatable headers are rendered wrong #81

monikma opened this issue Nov 12, 2014 · 3 comments

Comments

@monikma
Copy link

monikma commented Nov 12, 2014

This is a minimal example of the problem. I have the following RAML file:

#%RAML 0.8
title: sample
baseUri: http://localhost/
version: v1
mediaType:  application/json

/pages:
  get:
    headers:
      link:
        repeat: true
        type: string

After I try to render it in a "Try it" tab, the "link" header is rendered with a little "plus" sign next to it, so that it can be multiplied, to provide multiple values.

However, as soon as I add second header by pressing this "plus" sign, the second header gets the same value as the first header, and whichever input field I modify, the other gets modified as well. This makes the feature completely unusable.

@dmartinezg
Copy link

Hi @monami555, I spent last week looking into this. It is a bug in the code that appeared after the update to angular 1.2.16 (Angular fixed a lot of issues in the ng-transclude directive, which apparently we were relying on).

Given that: fixing the issue requires making very big changes to the code, and we are working in a new UI for the api-console (coming in the near future), we thought it is one bug that can wait a little.

@a-thaler
Copy link

Hi @dmartinezg,

I tried "repeat"-able headers in the newest api-console. Now it is not showing the plus sign anymore, so it is rendered as a single header.. but even worse, it seems to be required always. I need to enter a value always to perform a request.

That's my test header:

Accept-Language: 
  type: string
  required: false
  repeat: true
  example: pl

@jarrodek
Copy link
Contributor

This was fixed in version 5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants