Nico Zigouras

Results 20 comments of Nico Zigouras

Cool, I like the look and feel of rapidoc, although the Try it out feature is disabled in Apicurio: https://github.com/Apicurio/apicurio-studio/blob/dfc277b03032562e4a3d37f16c851159340495b0/front-end/servlet/src/main/resources/io/apicurio/studio/fe/servlet/servlets/preview_rapidoc.template#L8 To enable it set `allow-try=true` or remove the attribute as...

I may be interested in contributing, I will be in touch!

@stephanegigandet I understand the risk of touching old code. We might therefore consider these changes first for new code only, e.g. if we are adding new functionality, try to implement...

There are many string literals everywhere in the code, e.g. tag names. If we could formalize the management of those in a data structure we could eliminate the chance of...

Just add some explanatory text. I am not exactly sure how it is calculated but I think it is "* Calculated as (this product value/average category value - 1) x...

@brandonfcohen1 the tests are commented, indicating what each one does. Some of them are POCs, just running code to see the output. Some have assertions. I would not run them...

What does the `vacant_properties_end` table hold, is it essentially a union of the data in all of the other tables? If so seems like it could be the most important...

@brandonfcohen1 you are probably looking at an earlier commit in this PR. Look at the latest commit. The backend setup doc describes the new backup and diff process.

> > What does the `vacant_properties_end` table hold, is it essentially a union of the data in all of the other tables? If so seems like it could be the...

> got this running the new script for the first time: This happens the first time you run it because there are null opa_ids in the vacant_properties table. To rerun,...