AtomGraph / LinkedDataHub

The low-code Knowledge Graph application platform. Apache license.
https://atomgraph.github.io/LinkedDataHub/
Apache License 2.0
485 stars 122 forks source link

Type Ahead did not suggest construct query and file on CSV import #138

Closed FNakano closed 1 year ago

FNakano commented 1 year ago

I tried to import CSV data using [LinkedDataHub] User guide - CSV import video and https://atomgraph.github.io/LinkedDataHub/linkeddatahub/docs/reference/imports/csv/#vocabulary-conversion as references.

  1. Created a countries.csv file from https://atomgraph.github.io/LinkedDataHub/linkeddatahub/docs/reference/imports/csv/#vocabulary-conversion fist file listing;
  2. Created a construct query from https://atomgraph.github.io/LinkedDataHub/linkeddatahub/docs/reference/imports/csv/#vocabulary-conversion third file listing
  3. Tried to create a CSV Import: typeahead for file and query did not suggest any file nor query (screenshot below).

Captura de tela de 2022-10-28 22-18-19

Also tried capitalizing first letter, same result.

Curiously, typeahead suggested a constructor (not a construct query)

photo1667008510

namedgraph commented 1 year ago

Thanks for the report, will investigate ASAP. If you could provide the Docker log leading up to the typeahead (docker-compose logs --tail 500 linkeddatahub) it would be helpful.

FNakano commented 1 year ago

Hello! Procedure to get the log:

  1. Removed the CSV import resource created in the previous tries;
  2. Removed the File resource;
  3. Removed the CONSTRUCT queries related to the issue.

From this clean(?) start:

  1. In the Root container, create new File, get countries.csv from the filesystem, Label it countries;
  2. In the Root container, create new CONSTRUCT, fill in the query text with the query from https://atomgraph.github.io/LinkedDataHub/linkeddatahub/docs/reference/imports/csv/, label it countries;
  3. In the Root container, create new CSV Import;
    1. Type 'cou' in File text box (got no suggestion);
    2. Type 'cou' in Query text box (got no suggestion);
  4. Generate log with docker-compose logs --tail 500 linkeddatahub >importCSV-typeahead-issue-138.log

importCSV-typeahead-issue-138.log

:)

namedgraph commented 1 year ago

@FNakano this could be to the broken multiple-type support in the typeahead component which went unnoticed for some time. I fixed this accidentally in the develop branch just a few days ago :) Would you be able to try this with the latest develop code?

namedgraph commented 1 year ago

Now released as 3.2.23 and on master as well.

FNakano commented 1 year ago

@FNakano this could be to the broken multiple-type support in the typeahead component which went unnoticed for some time. I fixed this accidentally in the develop branch just a few days ago :) Would you be able to try this with the latest develop code?

Hello! Tried the develop branch a few minutes ago. Typeahead do suggest. photo1667226399

I'm going to try the latest release...

FNakano commented 1 year ago

Hello! Yes! TypeAhead in master branch showed the same suggestions. I'm going to close the issue... ... it is a somehow related doubt: Which one should I choose - the ArchiveItem or the FileDataObject?

Thank you!