These are chat archives for inveniosoftware/invenio

6th
Apr 2016
Esteban J. G. Gabancho
@egabancho
Apr 06 2016 09:48

I am getting this weird error on travis

self = <sqlalchemy.dialects.postgresql.psycopg2.PGDialect_psycopg2 object at 0x7f6031291f10>
cursor = <cursor object at 0x7f603132e908; closed: -1>
statement = '\nCREATE TABLE records_metadata (\n\tcreated TIMESTAMP WITHOUT TIME ZONE NOT NULL, \n\tupdated TIMESTAMP WITHOUT TIME ZONE NOT NULL, \n\tid UUID NOT NULL, \n\tjson JSON, \n\tversion_id INTEGER NOT NULL, \n\tPRIMARY KEY (id)\n)\n\n'
parameters = {}
context = <sqlalchemy.dialects.postgresql.psycopg2.PGExecutionContext_psycopg2 object at 0x7f603016d110>
    def do_execute(self, cursor, statement, parameters, context=None):
>       cursor.execute(statement, parameters)
E       ProgrammingError: (psycopg2.ProgrammingError) type "json" does not exist
E       LINE 6:  json JSON, 
E                     ^
E        [SQL: '\nCREATE TABLE records_metadata (\n\tcreated TIMESTAMP WITHOUT TIME ZONE NOT NULL, \n\tupdated TIMESTAMP WITHOUT TIME ZONE NOT NULL, \n\tid UUID NOT NULL, \n\tjson JSON, \n\tversion_id INTEGER NOT NULL, \n\tPRIMARY KEY (id)\n)\n\n']

has anyone seen this before?

Jiri Kuncar
@jirikuncar
Apr 06 2016 10:04
Which version of PG are you running?
Esteban J. G. Gabancho
@egabancho
Apr 06 2016 10:05
doesn’t work with 9.4?
Lars Holm Nielsen
@lnielsen
Apr 06 2016 11:19
9.3 / 9.4 should work…..just 9.2 which doesn’t have the json type
Esteban J. G. Gabancho
@egabancho
Apr 06 2016 11:26
I’m using 9.4 and it’s failing for some reason :(
Javier Martin Montull
@jmartinm
Apr 06 2016 12:57
Hi there, just had a look at https://github.com/inveniosoftware/invenio-deposit - as we will also need in INSPIRE, is there a plan to create separate issues/milestones with features that will be implemented?
Also happy to help from our side
Lars Holm Nielsen
@lnielsen
Apr 06 2016 13:09
@jmartinm We’re sprinting on it this week and next……after that there will surely be leftovers that would be great to have help with…..
Javier Martin Montull
@jmartinm
Apr 06 2016 13:10
and for the sprints you don’t use Github milestones anymore?
Lars Holm Nielsen
@lnielsen
Apr 06 2016 13:12
yes, but this one is a bit more tricky as there’s some prototyping to get done first….and we’re primarily focused on the zenodo deposit
so you’ll have to wait a bit to see more ;-)
Tibor Simko
@tiborsimko
Apr 06 2016 13:18
If you are looking at milestones, you simply have to query both Invenio and Zenodo organisations: https://github.com/issues?q=user%3Ainveniosoftware+user%3Azenodo+milestone%3ASprint-Deposit
Lars Holm Nielsen
@lnielsen
Apr 06 2016 13:25
@tiborsimko it’s not fully updated (or at least doesn’t give the full picture)
on me to fix that
Jiri Kuncar
@jirikuncar
Apr 06 2016 13:29
There is a WIP PR inveniosoftware/invenio-deposit#27 with current prototype. Everything is hardcoded for now, but you can try the example app.
Javier Martin Montull
@jmartinm
Apr 06 2016 13:41
thanks for the info - will give it a try
Lars Holm Nielsen
@lnielsen
Apr 06 2016 13:45
@jmartinm i saw that you’re looking at oauth2server/client…..what would be helpful at this state is if you take the current modules for a spin and see that everything……(also e.g. migration of tokens etc would be useful - i.e. for invenio-migrator)
Javier Martin Montull
@jmartinm
Apr 06 2016 13:55
:+1: will give it a try and see how migrations are being done in invenio-migrator
Tibor Simko
@tiborsimko
Apr 06 2016 14:28
@lnielsen WRT populating the sprint issue list, even some cryptic one-liners will do :) once the exploratory phase is over
Samuele Kaplun
@kaplun
Apr 06 2016 15:33
@lnielsen BTW, supposed that I want the little form that asks for email after a user a authenticated for the first time via ORCID, to be styled as all the other pages of my installation, where should I fiddle with? (in invenio-oauthclient) Also to properly configure where to redirect after authentication... (currently it 404s because it redirects to http://localhost:5000/http://localhost:5000 :) )
Lars Holm Nielsen
@lnielsen
Apr 06 2016 15:35
is the problem that a) it doesn’t ask for email or b) just the styling ?
Samuele Kaplun
@kaplun
Apr 06 2016 15:35
Just b) :)
(and then the redirect, in general, regardless if coming from the form)
which probably should be change to extend a different template
i don’t think {%- extends config.OAUTHCLIENT_SETTINGS_TEMPLATE %} is correct
alternatively, this has to be replaced with a config variable
Samuele Kaplun
@kaplun
Apr 06 2016 15:38
Which indeed in our instance resolves to: invenio_oauthclient/settings/base.html
Lars Holm Nielsen
@lnielsen
Apr 06 2016 15:38
for the redirect, i assume it’s a bug…..could be because the redirect uri is being set in the form….
Samuele Kaplun
@kaplun
Apr 06 2016 15:39
I see. Beside that, authentication into ORCID works great :)
Lars Holm Nielsen
@lnielsen
Apr 06 2016 15:40
probably it should extend COVER_TEMPLATE instead, so that it fits with the rest of the login
for redirect, check value of next
it’s either provided wrong, or not handled correctly
Samuele Kaplun
@kaplun
Apr 06 2016 15:41
Thanks!
Samuele Kaplun
@kaplun
Apr 06 2016 15:54
Mmh, I wonder if ORCID respect at all our next parameter at this point.
Lars Holm Nielsen
@lnielsen
Apr 06 2016 15:55
they ought to...
at least i remember we had an interaction with them about it
Samuele Kaplun
@kaplun
Apr 06 2016 15:56
Yep recently we have asked to allow http://localhost:5000 for our sandbox credentials, and I wonder if they accept only that :) and ignore anything under it...
Lars Holm Nielsen
@lnielsen
Apr 06 2016 16:05
ahh….otherwise you can use http://0.0.0.0:5000 ;-)