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.

Sign up for GitHub

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

Jump to bottom

nansat ingestor is not needed anymore since we have localharvester in the harvesting repo #154

Open
opsdep opened this issue Mar 4, 2021 · 1 comment · May be fixed by #157
Open

nansat ingestor is not needed anymore since we have localharvester in the harvesting repo #154

opsdep opened this issue Mar 4, 2021 · 1 comment · May be fixed by #157

Comments

@opsdep
Copy link
Contributor

opsdep commented Mar 4, 2021

No description provided.

opsdep pushed a commit that referenced this issue Mar 4, 2021
#154 removing the whole app of nansat_ingestor
9f6c1d3
opsdep pushed a commit that referenced this issue Mar 4, 2021
#154 modify the travis file
05beb0c
opsdep pushed a commit that referenced this issue Mar 4, 2021
#154 remove nansat_ingestor from settings.py
9c0af81
opsdep pushed a commit that referenced this issue Mar 4, 2021
#154 remove one redundant test
1da35ca
opsdep pushed a commit that referenced this issue Mar 4, 2021
#154 removing redundant functions from utils.py
cd7f32c
opsdep pushed a commit that referenced this issue Mar 4, 2021
#154 update the readme file
ad04ea1
opsdep pushed a commit that referenced this issue Mar 4, 2021
#154 remove one line from readme file because of removal of nansat_in…
516be93 
…gestor
@akorosov
Copy link
Member

akorosov commented Mar 8, 2021

We cannot remove functionality just like that. There are many users who need nansat_ingestor because they cannot start using geospaas-harvesting right away. We should rather add deprecation warning and allow users to prepare for removal of nansat_ingestor.

  • add deprecation warning to nansat_ingestor

opsdep pushed a commit that referenced this issue Mar 10, 2021
#154 deprecation message
b08d57b
opsdep pushed a commit that referenced this issue Mar 10, 2021
#154 remove input line
c69aca7
opsdep pushed a commit that referenced this issue Mar 10, 2021
#154 remove incorrect test
4df0986
opsdep pushed a commit that referenced this issue Mar 10, 2021
#154 deprecation message
d324b87
opsdep pushed a commit that referenced this issue Mar 10, 2021
#154 remove input line
d39a426
opsdep pushed a commit that referenced this issue Mar 10, 2021
rebase #154 remove incorrect test
f1c0229
opsdep pushed a commit that referenced this issue Mar 10, 2021
#154 minors
f95e96b
@aperrin66 aperrin66 linked a pull request Dec 27, 2021 that will close this issue
#154 deprecation message #157
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#154 deprecation message nansencenter/django-geo-spaas
2 participants
@akorosov @opsdep