Skip to content

Draft: removing deprecated concepts from export

davidnoirman requested to merge fix-export-bug into master

Simple bug fix that addresses #206.

The bug

When trying to export an ssyk-level-4 group that have deprecated skill concepts connected to it to excel, an error occurs. The editor is trying to fetch skill-headline concepts for deprecated skills, without setting include_deprecated to true. The API responds with an empty vector.

The fix

When exporting an ssyk-level-4 group there is no need to include skills that have been deprecated. This fix excludes such skills.

Testing

Pre bug fix state

From the editor, choose the ssyk-level-4 group 7233 Underhållsmekaniker och maskinreparatörer and click export:

Screenshot_from_2024-03-27_14-14-26

After a while, this should generate an error in the console (the error will not show up in the GUI).

From the network pane you can also see which request the editor gets stuck on.

Post bug fix state

Do the same procedure as above and verify that the export goes through.

Merge request reports