Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[1.9.2] Export doesn't handle lists with no cards #4902

Closed
magikmw opened this issue Jul 10, 2023 · 0 comments 路 Fixed by #5096
Closed

[1.9.2] Export doesn't handle lists with no cards #4902

magikmw opened this issue Jul 10, 2023 · 0 comments 路 Fixed by #5096

Comments

@magikmw
Copy link
Contributor

magikmw commented Jul 10, 2023

How to use GitHub

  • Please use the 馃憤 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Describe the bug
Export function doesn't output anything if the board being exported contains a list without any cards assigned.
Issue seems to be similar to the one solved here: #4566

To Reproduce
Steps to reproduce the behavior:

  1. Go to Deck
  2. Create a new deck containing at least two lists, and leave one of them empty
  3. Try exporting via left panel 3 dot menu -> Export board
  4. Nothing, except for an error in browser console:
Uncaught (in promise) TypeError: e.cards is undefined
    ht BoardApi.js:156
    ht BoardApi.js:156
    promise callback*value BoardApi.js:150
    actionExport AppNavigationBoard.vue:320
    VueJS 4
    onClick index.module.js:2
    VueJS 33

Expected behavior
Should export a board into CSV file.

Screenshots
n/a

Client details:

  • OS: Windows 10 22H2
  • Browser Firefox
  • Version 114.02
  • Device: desktop

I've left out server details, because I'm fairly sure it's JS related. Log in reproduction steps.

Using NC 26.0.3 and Deck 1.9.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants