Skip to content

Conversation

@lifeBCE
Copy link

@lifeBCE lifeBCE commented Apr 16, 2015

No description provided.

@mohsen1
Copy link
Contributor

mohsen1 commented Apr 16, 2015

@lifeBCE This looks like a very good feature. Couple of thought:

  • You should make changes in the source code not the files in the dist folder. To start developing, run gulp serve, make your changes and let gulp produces a new build in dist folder.
  • To change the HTML templates change the HTML files not the produced JS file.
  • I believe there are multiple places this input tag is being rendered (based on required, not require). Please add {{type}} to all of them.

@lifeBCE
Copy link
Author

lifeBCE commented Apr 16, 2015

Yeah I spent some time trying to find the actual source but was unsuccessful. I'll take another look when I have some more time.

fehguy added a commit that referenced this pull request May 8, 2015
@fehguy
Copy link
Contributor

fehguy commented May 8, 2015

added logic in b1efe88

@fehguy fehguy closed this May 8, 2015
@tokra
Copy link

tokra commented Oct 21, 2015

Is this also working for @formdata ? in our API using swagger+ui we have formData on login API and password field on swagger-UI has visible characters. can we somehow show *** instead of plain text on formData which are password?

@fehguy
Copy link
Contributor

fehguy commented Oct 21, 2015

It should be supported, yes. Just use the following:

{
  "type": "string",
  "format": "password"
}

@tokra
Copy link

tokra commented Oct 21, 2015

I would like to use it in Java code. Do you have some annotation for it ? thx

@tokra
Copy link

tokra commented Apr 22, 2016

@fehguy Can you please answer my question ? How do i define that some String expected on API method is password, for swagger UI - as Backend we have swagger with JAX-RS. Is there any annotation so swagger will produce something for swagger-ui ? thank you in advance

@fehguy
Copy link
Contributor

fehguy commented Apr 22, 2016

Sorry @to-kra this repo is for swagger-ui. Please open an issue on swagger-core for this--to be honest I don't think it exists, but that's the right place to ask.

@tokra
Copy link

tokra commented Apr 25, 2016

@fehguy I mean this is not just a backend part, I am talking about swagger-ui, to support password fields.

Jonahss pushed a commit to eaze/swagger-ui that referenced this pull request Aug 12, 2016
vincent-zurczak pushed a commit to roboconf/swagger-ui that referenced this pull request Aug 19, 2016
JuanSW18 pushed a commit to Digital-Paw/digital-paw-swagger-ui that referenced this pull request Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants