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

[BE] Support assets/ paths #37

Closed
ioggstream opened this issue Feb 15, 2022 · 11 comments
Closed

[BE] Support assets/ paths #37

ioggstream opened this issue Feb 15, 2022 · 11 comments

Comments

@ioggstream
Copy link
Contributor

I expect

To harvest first following paths:

assets/{ontologies,vocabularies,schemas}

Instead

It only procesesses ontopia-like paths Ontologie,VocabolariControllati

Possibili soluzioni

Extend the static class to support both, considering that Ontopia-like paths are a legacy.

A configuration parameter has been considered and excluded.

ioggstream added a commit that referenced this issue Feb 15, 2022
@Clou-dia Clou-dia changed the title Support assets/ paths [BE] Support assets/ paths Nov 2, 2023
gnespolino added a commit to nnzJd/dati-semantic-backend that referenced this issue Nov 3, 2023
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in [schema.gov.it] - issue tracking Nov 3, 2023
@Clou-dia
Copy link
Member

Clou-dia commented Nov 6, 2023

Hi, how can we test it? @gnespolino

@Clou-dia Clou-dia reopened this Nov 6, 2023
@gnespolino
Copy link
Contributor

Hi, how can we test it? @gnespolino

It's released on dev and test environments, so yes. We also added Inail to the repo lists

@Clou-dia
Copy link
Member

Clou-dia commented Nov 6, 2023

from dev or test I cannot see inail assets, I can only see the changes in the PR.
What do you mean by "We also added Inail to the repo lists"?
You changed the possible folder organization and name

@gnespolino
Copy link
Contributor

from dev or test I cannot see inail assets, I can only see the changes in the PR. What do you mean by "We also added Inail to the repo lists"? You changed the possible folder organization and name

we added support for asset/** folder, leaving the legacy folder name support.
So yes, Inail repo should be harvested now, but you probably we need to wait the harvester to run.

@gnespolino
Copy link
Contributor

from dev or test I cannot see inail assets, I can only see the changes in the PR. What do you mean by "We also added Inail to the repo lists"? You changed the possible folder organization and name

I just triggered an harvesting manually both on dev and test, result should be visible in a couple of minutes.

@Clou-dia
Copy link
Member

Clou-dia commented Nov 6, 2023

can you confirm this are the url:
https://ndc-test.apps.cloudpub.testedev.istat.it/
https://ndc-dev.apps.cloudpub.testedev.istat.it/

Still didn't see anything from inail, so far.

@Clou-dia Clou-dia moved this from ✅ Done to 🏗 In progress in [schema.gov.it] - issue tracking Nov 6, 2023
@gnespolino
Copy link
Contributor

can you confirm this are the url: https://ndc-test.apps.cloudpub.testedev.istat.it/ https://ndc-dev.apps.cloudpub.testedev.istat.it/

Still didn't see anything from inail, so far.

Good morning @Clou-dia, let me finish some work on it (there was a naming error in the vocabularies folder). I'll ping you when it's ready.

@Clou-dia
Copy link
Member

Clou-dia commented Nov 8, 2023

#78

@Clou-dia
Copy link
Member

@gnespolino è risolto?

@gnespolino
Copy link
Contributor

@gnespolino è risolto?

yes, legacy paths are still supported as well.
New supported paths are here

@Clou-dia
Copy link
Member

Verified, thanks

@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in [schema.gov.it] - issue tracking Nov 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

5 participants