OpenAPITools / openapi-generator

OpenAPI Generator allows generation of API client libraries (SDK generation), server stubs, documentation and configuration automatically given an OpenAPI Spec (v2, v3)
https://openapi-generator.tech
Apache License 2.0
21.87k stars 6.59k forks source link

[Elm] import 'Data.List_' not used, but causing an error #1378

Closed Stealthmate closed 5 years ago

Stealthmate commented 6 years ago
Description

I'm generating an API from haskell's servant-swagger, where an endpoint returns a type QueryResponse which is an object with two array fields. The output file for this a module Data.QueryResponse, which has the import line

import Data.List_ as List_ exposing (List_)

however, List_ is not used anywhere in the file. On top of that, Elm doesn't recognize any module like that, and from what I've found there isn't one.

openapi-generator version

3.3.2, built from source from current master

OpenAPI declaration file content or url

gist

Command line used for generation

openapi-generator-cli.sh generate -i "http://localhost:8080/swagger.json" -l elm -o ./api

Steps to reproduce

Just execute the above command on a host that serves the file

Related issues/PRs

I tried looking for similar issues, but didn't find anything.

Suggest a fix/enhancement

I'm not really sure, but I think just simply not importing List_ would do the trick.

Stealthmate commented 6 years ago

I noticed some other problems.

Here's a gist of the code as well.

I think nested list generation may not be working properly. I'll try to do some more research during the week if I have time.

eriktim commented 5 years ago

Thanks for your report @Stealthmate. List_ is indeed a bug as nested Lists/Dicts were not yet supported. See the PR for a fix for both.

Stealthmate commented 5 years ago

@trenneman Thank you! I'll take a look!