Raised by @sellth and it is a good point. If a user makes an edit which is technically allowed in ISA-Tab, warnings raised by AltamISA are not displayed. They are only shown for sheet import/restore.
The problem is, these warnings are usually verbose and most of them most likely don't even come from the most recent edit, but the initial import. However, it would still make sense for an expert user to be able to see them if needed.
Hence, a reasonable implementation could be as follows:
Return warnings from SheetCellEditAjaxView, SheetRowInsertAjaxView and SheetRowDeleteAjaxView on 200
Display a button near the status indicator for warnings if they have occurred since the last edit
(The whole status display should be vastly improved, but that's a different topic)
Open a modal from the button displaying the most recent parser warnings
TBD: Prioritization and schedule? E.g. should this be done before/after #994? Not much functionality is required in the Vue app side, so if this is considered urgent we don't really need to wait for that blocking issue.
Raised by @sellth and it is a good point. If a user makes an edit which is technically allowed in ISA-Tab, warnings raised by AltamISA are not displayed. They are only shown for sheet import/restore.
The problem is, these warnings are usually verbose and most of them most likely don't even come from the most recent edit, but the initial import. However, it would still make sense for an expert user to be able to see them if needed.
Hence, a reasonable implementation could be as follows:
SheetCellEditAjaxView
,SheetRowInsertAjaxView
andSheetRowDeleteAjaxView
on 200TBD: Prioritization and schedule? E.g. should this be done before/after #994? Not much functionality is required in the Vue app side, so if this is considered urgent we don't really need to wait for that blocking issue.