search

yawik / SimpleImport

Simple Job Import Module. Imports job openings into YAWIK
MIT License
0 stars 1 forks source link

add-crawler does not check for existing crawlers #15

Closed mbo-s closed 4 years ago

mbo-s commented 6 years ago

The add-crawler command does not check for an existing crawler and will add the crawler twice

cbleek commented 4 years ago

@kilip

the following cli command adds a crawler to the db. The command should exit, if the name of the crawler already exists.

$ ./vendor/bin/yawik simpleimport add-crawler --name=mycrawler --feed-uri=http://example.com --organization=<valid organization id>

do you have time do do this?

kilip commented 4 years ago

Yes, I will work on this issue now

kilip commented 4 years ago

@cbleek If necessary, I can make crawler to check an existing feed-uri too.

cbleek commented 4 years ago

Males more Sense, than to check the Spider Name. Yes, please Check The Feed-url.

Cross Solution | Carsten Bleek Diemelstraße 2-4 | Tel: 069-71910361 60486 Frankfurt am Main | Fax: 069-71910369 http://cross-solution.de | bleek@cross-solution.de

Am 02.07.2020 um 04:59 schrieb Anthonius Munthi notifications@github.com:

 @cbleek If necessary, I can make crawler to check an existing feed-uri too.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.