intellij-swagger icon indicating copy to clipboard operation
intellij-swagger copied to clipboard

Support completion from workspace files

Open gayanper opened this issue 4 years ago • 8 comments

This will show definition in other workspace files in ref completions for definitions.

gayanper avatar May 11 '20 15:05 gayanper

Coverage Status

Coverage increased (+0.1%) to 91.282% when pulling 0cf5a41031f30d4d0a4a40fc57851c51e28dda56 on gayanper:ref_completions into 372e5a26b58212525390a09448be2239a134598a on zalando:master.

coveralls avatar May 11 '20 15:05 coveralls

@zeitlinger and @tkrop Please review my changes and let me know what you think.

Please let me know what do you think about the PR. I’m not really sure its a good idea to start file paths with ./. Let me know your thoughts

gayanper avatar May 11 '20 16:05 gayanper

@zeitlinger is this repo live and accepting PRs ?

gayanper avatar May 12 '20 15:05 gayanper

Can these PRs get some feedback ? I would really like to get this merged into the product rather than starting up my own fork doing builds. @semonte @maxim-tschumak @tkrop please help me to get this changes in. I’m trying hard here because this is great potential tool when it comes swagger spec editing.

gayanper avatar May 13 '20 10:05 gayanper

@gayanper thanks for the PR.

I'm not sure what this feature is about. Can you explain your exact use case and what additional references are made available here?

tkrop avatar May 16 '20 10:05 tkrop

@tkrop When you split swagger spec in the multiple files, you might want to add a reference to a resource in another file. This PR will list resources from all the files inside the current module the current editing file belongs to. Something similar to https://apihandyman.io/writing-openapi-swagger-specification-tutorial-part-8-splitting-specification-file/

gayanper avatar May 16 '20 11:05 gayanper

A year will soon pass, and the problem has not been resolved. The plugin is developing in the best traditions of a bloody enterprise. Good strategy, effective and users are happy.

shaburov avatar Apr 19 '21 15:04 shaburov

@shaburov Sorry, the year. I'm actually only waiting for the initiator to add the security switch to turn this feature on and off as a safeguard. If you are willing to take this over, I can merge it tomorrow.

tkrop avatar Jun 11 '21 08:06 tkrop

Closed, since user test with bigger projects have not been executed to mitigate risk of bad user experience due to autocomplete search delays.

tfrauenstein avatar Sep 06 '22 12:09 tfrauenstein