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
fix(excel2json): don't crash if optional columns are deleted in "properties" Excel file (DEV-2652) #518
fix(excel2json): don't crash if optional columns are deleted in "properties" Excel file (DEV-2652) #518
Conversation
DEV-2652 excel columns optional
should support deleted excel columns whose content is optional src/dsp_tools/utils/excel2json/properties.py |
@Nora-Olivia-Ammann This PR is okay now, but could you improve the title + text of the linear ticket? As it is now, I only understand it because I know the context. But imagine we discuss it in the DSP-TOOLS meeting: no one would understand. BTW: The same applies for some other tickets you created. If you have time, could you reformulate them? |
@Nora-Olivia-Ammann : IMHO, the title is not expressive enough. What do you think about "fix(excel2json): don't crash if optional columns are deleted in "properties" Excel file" --> and the linear ticket accordingly |
No description provided.