Replies: 2 comments
-
Sounds like a good idea, especially since the autocomplete will show the user the types of data that can be accepted |
Beta Was this translation helpful? Give feedback.
-
Honestly, I just followed the approach that we had with one general "read" function. It also was a way to automatically support newly added formats: with a separate annotation for each format, you'll have to explicitly support it in KSP plugin, right? It's going to be mostly repetitive code, but it will be something to keep in mind. |
Beta Was this translation helpful? Give feedback.
-
Now, the annotation @file:ImportDataSchema contains a lot of parameters that gives the user ability to cover all the possible scenarios. But probably it was started as a smaller feature but continued to grow and cover more scenarious (depends on the number of the supported data sources).
I suggest to refactor it in 2 steps: create two child annotations like @file:ImportDataSchemaFromCSV and @file:ImportDataSchemaFromJSON and work with them a little bit. If it's ok, we could make the basic annotation deprecated and remove after getting feedback from our users.
What do you think @belovrv @Jolanrensen @koperagen @ermolenkodev ?
Beta Was this translation helpful? Give feedback.
All reactions