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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nextcloud Tables name with "-" #380

Closed
hartsberger opened this issue Dec 7, 2023 · 4 comments
Closed

Nextcloud Tables name with "-" #380

hartsberger opened this issue Dec 7, 2023 · 4 comments
Assignees
Labels
in progress development in progress needs info feedback from requester required
Milestone

Comments

@hartsberger
Copy link

Hi,

there is an issue when renaming the Nextcloud table used as data source after setting up a background load job.

To reproduce:

  1. Create a Nextcloud table. For example "Gewicht".
  2. Create an Analytics report with a dataset.
  3. Under "automation" choose the Nextcloud table as the source.
  4. Configure a hourly background load.
  5. Wait until the background job runs the first time and verify that everything works as expected.
  6. Rename the Nextcloud table and let it contain a "-". For example "Gewicht - Stefan".
  7. Wait for the next run of the background job and receive an error message via notifcation.

I guess it has to do with the second issue that I noticed:

In the automation area:
If you want to choose a Nextcloud table from the dropdown list, it truncates the table names at the point where a "-" exists in the name.
For example: The table name "Gewicht - Stefan" appears as "Gewicht " in the dropdown list.

I think if the "-" problem is solved, then the issue with the failing background job is solved, too.

@Rello Rello changed the title Error with background dataload when Nextcloud table gets renamed and contains a "-" Nextcloud Tables name with "-" Dec 10, 2023
Rello added a commit that referenced this issue Dec 10, 2023
@Rello Rello self-assigned this Dec 10, 2023
@Rello Rello added in progress development in progress needs info feedback from requester required labels Dec 10, 2023
@Rello
Copy link
Owner

Rello commented Dec 10, 2023

Hi,

the '-' caused just a display issue. it is corrected in the next version.
But all data loads are via IDs, so that can not be the issue.
is there anything on the nextcloud.log?

@Rello Rello added this to the 4.12.0 milestone Dec 10, 2023
@hartsberger
Copy link
Author

I fixed the load errors now. Had been strange values in the data source. Now it's working.
Thanks for fixing the display issue.

@Rello
Copy link
Owner

Rello commented Dec 11, 2023

Hi @hartsberger can you comment on the "strange values"?
I am always looking into making the app more error resistant...

@hartsberger
Copy link
Author

hartsberger commented Dec 12, 2023

Hi @Rello ,
sure. I have a Nextcloud table with some rows not having values in it (Null values).
These null values get displayed as "NaN" in the tables app.
When I try to load them, I get an error message, that the load has failed. But it does not tell me why.

It would be nice, if the system tells me why it has failed to load the data.
Or if it would just load these null values in the table.
Or if it would ask me if I would like to include null values.

Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in progress development in progress needs info feedback from requester required
Projects
None yet
Development

No branches or pull requests

2 participants