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

oai config #3

Merged
merged 1 commit into from
Aug 22, 2018
Merged

oai config #3

merged 1 commit into from
Aug 22, 2018

Conversation

rerowep
Copy link
Contributor

@rerowep rerowep commented Aug 21, 2018

configs could be added with:

invenio oaiharvester initconfig data/oaisources.yml

or with

invenio oaiharvester addsource VS https://mediatheque-valais.cantookstation.eu/oai.xml

Celery must be started for asynchrone harvesting:
celery worker --app invenio_app --loglevel INFO -B -E

Harvesting could be started with (-k asynchrone, -q quite):
invenio oaiharvester harvest -n VS -k -q
invenio oaiharvester harvest -n NJ -k -q

@rerowep rerowep force-pushed the wep-#614-oai-config branch 3 times, most recently from 998d5f2 to 8946786 Compare August 21, 2018 13:52
@@ -0,0 +1,18 @@

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing copyright

@@ -165,3 +167,12 @@ def _(x):

#: Switches off incept of redirects by Flask-DebugToolbar.
DEBUG_TB_INTERCEPT_REDIRECTS = False

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to be removed?

click.echo('record uuid: ' + str(record.id) + ' | ' + status)


@click.group()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can be imported from invenio-oaiharvester?

@@ -0,0 +1,10 @@
VS:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

copyright?

@jma
Copy link
Contributor

jma commented Aug 22, 2018

3 commits can be merged into one.

@rerowep rerowep force-pushed the wep-#614-oai-config branch 2 times, most recently from 9ae1801 to 3cf6665 Compare August 22, 2018 06:27
Signed-off-by: Peter Weber <Peter.Weber@rero.ch>
Co-authored-by: Bertrand Zuchuat <Bertrand.Zuchuat@rero.ch>
@rerowep rerowep force-pushed the wep-#614-oai-config branch from 3cf6665 to df566a0 Compare August 22, 2018 06:55
from flask.cli import with_appcontext
from invenio_oaiharvester.cli import harvest
from invenio_records.cli import records

from rero_ebooks.api import Ebook
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

relative import

from invenio_records.cli import records

from rero_ebooks.api import Ebook
from rero_ebooks.utils import add_oai_source
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

relative import

reindex=True
)
click.echo('record uuid: ' + str(rec.id) + ' | ' + status)
# TODO bulk update and reindexing
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to be removed?

dbcommit=True,
reindex=True
)
click.echo('record uuid: ' + str(rec.id) + ' | ' + status)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

click does not makes sense when it is called from receiver!

@jma jma merged commit 2b39cce into rero:master Aug 22, 2018
@rerowep rerowep deleted the wep-#614-oai-config branch November 26, 2018 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants