tenthe commented on PR #2616: URL: https://github.com/apache/streampipes/pull/2616#issuecomment-2022142363
> > > > Should we also create variables for data type names that are displayed to the user in `DataTypeService`? (e.g. String, Integer, Boolean, ...). If so, how should the variables for the urls and the labels be named? > > > > > > > > > Do we use them at multiple places? I could imagine to also include them in `DataType` as a `displayLabel` or similar > > > > > > There are currently only two places where we use it. That's why I wasn't sure. > > I think then we can move it to the `DataType` class so we have everything in one place Ok, I checked it again and there is only one place where it is used. So I'll leave it at that for now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
