[BUG]: Add Field from Data Source don't work when text

narimanshariat commented 2 months ago

SDK Version

1.6.4

Client Framework

React

Server Platform

Java

Operating System

Linux

Description

Uncaught (in promise) TypeError: Cannot read properties of null (reading 'dataSourceId') at Class.createDataSourcesWithDatabase (infragistics.reveal.js:154:1371040) at Class.getExistingDatasources (infragistics.reveal.js:154:1374345) at c.init (infragistics.reveal.js:154:1219048) at c.init (infragistics.reveal.js:168:112374) at new c (infragistics.reveal.js:56:695) at c (infragistics.reveal.js:168:286774) at infragistics.reveal.js:168:287377 at o.current.onDataSourcesRequested (RevealContainer.tsx:48:17) at SdkApp.showSelectDatasourcesScreen (infragistics.reveal.js:168:287319) at c.showBlendEditor (infragistics.reveal.js:154:1653585)

This error occurs in the following condition.

Create a visual with a data source
Add a text box
Edit the existing visual.
Add a field from a data source to the visual
Error occurs
Exit the edit mode for the visual
Delete the text box
Repeat steps 3 & 4 and no error occurs

gkaveckis commented 2 months ago

Hi Could you send us some quick video about how to reproduce it? Because we are trying here without success,

Thanks!

narimanshariat commented 2 months ago

Here you go: https://www.loom.com/share/bc71c6f9f5f846d8a3e761b1dafc6dc9

dpazos-infragistics commented 2 months ago

Hi. We tried to reproduce the error but have been unable to do so. Have you been able to reproduce it in a simpler dashboard that might help us reproduce it locally?

narimanshariat commented 2 months ago

Now with the console panel in view. https://www.loom.com/share/ba501bf9fdc442fb8f74fface3b62b77

dpazos-infragistics commented 2 months ago

I tried to reproduce the issue with SDK 1.6.4, using Postgres for the visualization and a textbox with no title, but the issue didn't occur. Let's try to simplify the scenario. Try deleting all of the visualizations except the one from which you're accessing the calculated field UI. Does the problem still happen? If not, try going back from there and see which visualization is the one causing the issue.

narimanshariat commented 2 months ago

I started with a completely blank dashboard and created a visual from scratch. Same issue.

https://www.loom.com/share/318a4d25e2ca45c287c34debdcc4205e

dpazos-infragistics commented 1 month ago

I followed your video as closely as I could and still couldn't reproduce the error. Here's what I tried:

Using SDK 1.6.4, create a new dashboard
Add a visualization based on a view on a Postgres DS. Keep the default of a Columns visualization
Add a text field to the Label section and another text field to the Values section
Add a field from another data source. Make it linked to a different view in the same DS, linked by a matched numeric field, and include a text field from it. Do not include this field in the visualization.
Save the visualization.
Add a textbox to the dashboard and set a text for it, leaving title empty.
Save the dashboard.
Edit the dashboard again
Edit the visualization
Select the option to add a field from another data source. No error was encountered.

Can you think of anything else that I missed and might be causing it?

narimanshariat commented 1 month ago

If I start a blank dashboard and add a text box. Then try to add a visualization I get the same error.

To me it seems that it is trying to fetch the datasource but it is hitting an error. Uncaught (in promise) TypeError: Cannot read properties of null (reading 'dataSourceId')

Is it possible that we don't have something configured correctly for textboxes to work? There is something unique about the textbox that is interrupting the normal flow.

brianlagunas commented 1 month ago

Can you try to create a small stand-alone application that reproduces the issue? You can start by modifying an existing sample to save you some time. Maybe start with one of the data source samples:

https://github.com/RevealBi/sdk-samples-javascript/tree/main/DataSources

You can also try sharing the dashboard RDASH file with us via a DM on Discord.

brianlagunas commented 3 weeks ago

There hasn't been a reply to this issue in a month. I am closing this issue since we have been unable to reproduce it. If you can provide a sample, or more information, that can help us reproduce this I can reopen it.

RevealBi / Reveal.Sdk