-
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
Redundant start-year-month and end-year-month in get-response #13
Comments
Please read the annotations for these four date-related fields. Those contain information that both date pairs are used for different types of mobility. Annotations are an integral part of the specification and the guidelines contained therein should be followed. Instead of the current solution, we considered creating separate fields for the year + month and day of the month. But we found that for full dates it would be better to operate on common XML types than to force systems to parse and merge the date based on two elements. |
I see how my report is misleading. Please allow me to clarify:
are both described as
The same applies to EDIT: This was garbage, the rest still applies. From my understanding, duplicating the start and end date from
For both I would think that:
In either case, I will need only one of them to make a decision - I fail to think of a scenario where having both EDIT: Changed documentation to further clarify. |
These dates are part of the template required by the EC. While from a business point of view, new la versions should not be created after certain dates, the dates sent in the LA API should be treated as an indication for the mobility coordinator, and not as strict boundaries on which IF instructions could be based in the application code. For example, it cannot be that you cannot create a new version of LA retrospectively just because the creation of a new version has been neglected at some point. Do not think of these dates as a duplicate from the Outgoing Mobilities API. Each university decides which EWP API will implement in its system. It is possible to implement the LA API without implementing the Outgoing Mobilities API. |
Forgot to mention: if this is just there, because it is part of the PDF: I don't see why that would be a reason to send it over the wire. The PDF would be created by sending institution anyway, and that already has the date, so I don't see that as a use-case. The date is not part of the approved portion of the LA (i.e. I can change these dates without nullifying any existing approval, right?) For this API, receiving institution should be only interested in "is there something to approve? if so, am I in time to do so?" and for the latter decision it needs a date to show to the user a "sorry, you're to late" or an approve button. |
Sure, but if both APIs are implemented, the dates should match, right?
So are we exposing data here, which is not even meant to be read by the other side? |
As the student is already nominated, and the nomination has been approved, receiving hei already has that information, right? And after reading through your comment very carefully once again:
This confuses me... I thought: What sense does it make to have a So it must be Are you now telling me, that every single piece of information in this whole API, including |
I'm more and more confused about this.
Which template are we even talking about? If you refer to https://wiki.uni-foundation.eu/display/EWP/New+LA+template I only see Nowhere does it state, that day is required for some types and forbidden for others, as the current version of the XSD states. The "guidelines" document only refers to If you have an additional source, could you please state it somewhere, or add it to the repository, as was the old one on EDIT: And it just pops my eyes right now - above screenshot: EDIT 2: What would even be the proposed mechanism for |
ddeff52 solves part of my questions Only thing remaining: how would Alternative (I guess) would be to declare them |
The two fields
start-date
andend-date
carrying the same (and more, as it is a full date) information. Having all of them optional also introduces possibility of weird combinations, like having onlystart-date
andend-year-month
. This needlessly makes client implementations more complex with extensiveif (...) then else
for all 4 cases (none
,one
,other
,both
) for either of the fields.If the 2 actually carry different information it needs to be documented what these differences are.
The text was updated successfully, but these errors were encountered: