New to OpenAlchemy...not sure why I'm getting an Every property requires a type with this OpenAPI yaml? Also...

[tduval-unifylogic]

Hi, not sure how else to ask this question. Couldn't find a questions or support link. I like what you have put together!

I tried following the documentation and I keep getting 'Every property requires a type' error with the first set of yaml. I've combed through it and every property has a type? That's my first issue, my second is how to handle properties whose types are enums? I searched documentation (at https://openapi-sqlalchemy.readthedocs.io/) and did not get any results. There's another yaml below the first one that the entire file including enums properties prior to coding it for OpenAlchemy. Thanks so much!

The second is the full OpenAPI yaml prior to tagging it for OpenAlchemy x-tablename, etc. where you can see how we refer to enums (which would be essentially a lookup table)

OpenAPI_OpenAlchemyCoded.txt
OpenAPI_FullAPI_noOpenAlchemyCoding.txt

[jdkandersson]

Hi @MartyStache, I'm glad you find it useful!

Regarding your first question, I have released an experimental online editor that should help you here: https://editor.openalchemy.io. It is in alpha stages, I'm keen to know how you went! It should point out exactly what is missing in your schema. In fact, I just tried it and it found a few problems:

1. The Status schema is missing a type definition (would you like it to be an object?)
2. There are some interesting formats you are using that I hadn't considered, uuid and url. They are currently not supported but I'll add them to the backlog.

Other than the above 2, you are good to go! I'll try to update the package over the next few days to add support for uuid and url formats.

On enum, you can add them to the schema but currently there is no effect. I think it is a good use case, I'll also add them in on the backlog. I think that one is slightly more complicated which is why I don't want to promise I can deliver that over the next few days.

[tduval-unifylogic]

Hey thanks! I will try it out and let you know how it goes! This is what I’m looking for for generation for enum (creates a look up table that I can foreign key: https://docs.sqlalchemy.org/en/13/dialects/postgresql.html?highlight=enum#sqlalchemy.dialects.postgresql.ENUM

…

On Thu, Aug 20, 2020 at 7:42 PM jdkandersson ***@***.***> wrote: Hi @MartyStache <https://github.com/MartyStache>, I'm glad you find it useful! Regarding your first question, I have released an experimental online editor that should help you here: https://editor.openalchemy.io. It is in alpha stages, I'm keen to know how you went! It should point out exactly what is missing in your schema. In fact, I just tried it and it found a few problems: 1. The Status schema is missing a type definition (would you like it to be an object?) 2. There are some interesting formats you are using that I hadn't considered, uuid and url. They are currently not supported but I'll add them to the backlog. Other than the above 2, you are good to go! I'll try to update the package over the next few days to add support for uuid and url formats. On enum, you can add them to the schema but currently there is no effect. I think it is a good use case, I'll also add them in on the backlog. I think that one is slightly more complicated which is why I don't want to promise I can deliver that over the next few days. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#166 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AO3YXTGGL3SKZ2TSOZXW5OLSBWYIDANCNFSM4QGVNQ6A> .

[tduval-unifylogic]

I am getting error. The yaml validates in your online editor? Help?

api.py", line 11, in list
return list(ElectricAsset_dicts)
TypeError: list() takes 0 positional arguments but 1 was given
127.0.0.1 - - [21/Aug/2020 20:19:39] "�[35m�[1mGET /electricasset HTTP/1.1�[0m" 500 -

api .yaml.txt
api.py.txt
models.py.txt

[jdkandersson]

The problem is that on line 6 you have defined the function name list which overrides the Python internal list constructor. If you re-name the def list(): function on line 6 to something else, you should be fine.

[jdkandersson]

Regarding alembic, take a look at this example: https://github.com/jdkandersson/OpenAlchemy/tree/master/examples/alembic

[tduval-unifylogic]

Omg...I’m an idiot...lol...got it...thanks for the alembic link!

[jdkandersson]

All good, let me know if anything else comes up.

FYI, support for generic string formats is coming in the next release (already merged into master and it would include uuid and url although OpenAlchemy won't do anything special for them). The Enum will come in a future release, not sure when yet.