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

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
opsdep pushed a commit that referenced this issue Mar 4, 2021
opsdep pushed a commit that referenced this issue Mar 4, 2021
opsdep pushed a commit that referenced this issue Mar 4, 2021
@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
opsdep pushed a commit that referenced this issue Mar 10, 2021
opsdep pushed a commit that referenced this issue Mar 10, 2021
opsdep pushed a commit that referenced this issue Mar 10, 2021
opsdep pushed a commit that referenced this issue Mar 10, 2021
opsdep pushed a commit that referenced this issue Mar 10, 2021
opsdep pushed a commit that referenced this issue Mar 10, 2021
@aperrin66 aperrin66 linked a pull request Dec 27, 2021 that will close this issue
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 a pull request may close this issue.

2 participants