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
azamifard 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

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

Comments

@azamifard
Copy link
Contributor

No description provided.

azamifard pushed a commit that referenced this issue Mar 4, 2021
#154 removing the whole app of nansat_ingestor
9f6c1d3
azamifard pushed a commit that referenced this issue Mar 4, 2021
#154 modify the travis file
05beb0c
azamifard pushed a commit that referenced this issue Mar 4, 2021
#154 remove nansat_ingestor from settings.py
9c0af81
azamifard pushed a commit that referenced this issue Mar 4, 2021
#154 remove one redundant test
1da35ca
azamifard pushed a commit that referenced this issue Mar 4, 2021
#154 removing redundant functions from utils.py
cd7f32c
azamifard pushed a commit that referenced this issue Mar 4, 2021
#154 update the readme file
ad04ea1
azamifard 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

azamifard pushed a commit that referenced this issue Mar 10, 2021
#154 deprecation message
b08d57b
azamifard pushed a commit that referenced this issue Mar 10, 2021
#154 remove input line
c69aca7
azamifard pushed a commit that referenced this issue Mar 10, 2021
#154 remove incorrect test
4df0986
azamifard pushed a commit that referenced this issue Mar 10, 2021
#154 deprecation message
d324b87
azamifard pushed a commit that referenced this issue Mar 10, 2021
#154 remove input line
d39a426
azamifard pushed a commit that referenced this issue Mar 10, 2021
rebase #154 remove incorrect test
f1c0229
azamifard 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 @azamifard