search

MaRDI4NFDI / portal-compose

docker-composer repo for mardi
https://portal.mardi4nfdi.de
GNU General Public License v3.0
3 stars 1 forks source link

Test new quickstatements version #472

Closed physikerwelt closed 2 weeks ago

physikerwelt commented 5 months ago

We still use our forked image for quickstatements (https://github.com/MaRDI4NFDI/docker-quickstatements) and I don't know how much work it would be to adapt that. I would open another issue if we also want to pull the latest quickstatements image.

@physikerwelt suggests @Daniel-Mietchen tests the new official version in staging, and we see where we go from there. Does that sound like a good plan?

Follow-up from #451

Daniel-Mietchen commented 5 months ago

@physikerwelt Federation with Wikidata needs to be enabled:

Unknown error: Service URI https://query.wikidata.org/sparql is not allowed

physikerwelt commented 5 months ago

@physikerwelt Federation with Wikidata needs to be enabled:

Unknown error: Service URI https://query.wikidata.org/sparql is not allowed

Is this related to quick statements? If not a new issue might be the better option. Or is that a regression from #451

Daniel-Mietchen commented 5 months ago

@physikerwelt Federation with Wikidata needs to be enabled:

Unknown error: Service URI https://query.wikidata.org/sparql is not allowed

Is this related to quick statements? If not a new issue might be the better option. Or is that a regression from #451

Much of my QuickStatements use involves federated queries to Wikidata, so in a way, this is related. But yes, will open a new ticket.

In the meantime, I made a sample edit via QuickStatements using the command Q3441669|P1454|"swh:1:snp:0f4038e01e822bdfa64ce3ee5a709fe609d18912": https://portal.mardi4nfdi.de/w/index.php?title=Item%3AQ3441669&diff=11891802&oldid=10348728

However, removing that edit via -Q3441669|P1454|"swh:1:snp:0f4038e01e822bdfa64ce3ee5a709fe609d18912" did not work. Perhaps an issue with permissions?

physikerwelt commented 5 months ago

No probably a issue with our quickstatements fork, it is only half finished. I suggest we try the new upstream version?

physikerwelt commented 5 months ago

@Daniel-Mietchen can you test https://quickstatements.staging.mardi4nfdi.org/#/ and report if it works better or worse than the version deployed in production

eloiferrer commented 1 month ago

What is the status here? @Daniel-Mietchen can we close this?

eloiferrer commented 2 weeks ago

I consider this solved for now, it can be reopened if any issues arise.