-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Some notifications include update and canonical link in links array #82
Comments
for reference, the other requirements in Req. 11:
|
Note, this is different than what we are doing with WCMP2 which has these distinct properties instead:
|
@wmo-im/tt-wismd |
@antje-s are you objecting the formulation "at least one" and would prefer to have there "exactly one"? |
The messages should not contain links for canonical and update (or deletion) at the same time. The notification is either for new data OR an update for already published data. Change to "exactly one" would be good, but then someone could still include "exactly one" canonical link AND "exactly one" update link. |
I have also stumbled upon notifications with both "canonical" and "update" links. We should definitely clarify this. The current formulation allows multiple canonical links too. I suggest to change
and add one more requirement:
Of course, suggestions for better formulations are welcomed, but the current formulations with "at least one" are obviously causing confusion. |
TT-WISMD 2024-11-27
ACTION: @amilan17 to investigate update workflow |
Should we add to Requirement 11:
"H": The links array property SHALL provide only one link with a link relation of http://def.wmo.int/def/rel/wnm/-/deletion, http://def.wmo.int/def/rel/wnm/-/update or IANA link relation of canonical"
to be more precise, whether it is new data, an update or a deletion.
The text was updated successfully, but these errors were encountered: