We usually use date in ioBroker context for fixed points in time (which sunrise and sunset are).
Also date.sunrise and date.sunset are defined in stateroles.md, time.sunrise and time.sunset are not.
I checked and found only accuweather adapter uses time.sunset and time.sunrise, but it still supplies a complete Date in both states. So the use of time seems to be confusing there, too. So currently adapters reading TIME_SUNRISE should expect a full date in there. The role should reflect that (especially the default role).
Openweathermap adapter already uses the correct role.
We usually use
datein ioBroker context for fixed points in time (which sunrise and sunset are). Alsodate.sunriseanddate.sunsetare defined in stateroles.md,time.sunriseandtime.sunsetare not.I checked and found only accuweather adapter uses
time.sunsetandtime.sunrise, but it still supplies a complete Date in both states. So the use oftimeseems to be confusing there, too. So currently adapters readingTIME_SUNRISEshould expect a full date in there. The role should reflect that (especially the default role).Openweathermap adapter already uses the correct role.