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

Include geonode-importer inside the core #12368

Closed
mattiagiupponi opened this issue Jun 27, 2024 · 0 comments · Fixed by #12570
Closed

Include geonode-importer inside the core #12368

mattiagiupponi opened this issue Jun 27, 2024 · 0 comments · Fixed by #12570
Assignees
Milestone

Comments

@mattiagiupponi
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Since the geonode-importer is stable and is a core component in GeoNode, is better to have it along with the core.
This will allow us to:

  • Remove everything that is not required in the old upload flow
  • Remove unused functions/models
  • Include a core component in the vanilla installation without adding extra dependencies in the master

Describe the solution you'd like
To avoid losing the history of the upload app, the importer will be moved inside the upload by adjusting the migrations. This will let us clean up some dead code

Describe alternatives you've considered
None

Additional context
None

@mattiagiupponi mattiagiupponi added this to the 4.4.0 milestone Jun 27, 2024
@mattiagiupponi mattiagiupponi self-assigned this Jun 27, 2024
@mattiagiupponi mattiagiupponi linked a pull request Sep 13, 2024 that will close this issue
13 tasks
@mattiagiupponi mattiagiupponi modified the milestones: 4.4.0, 5.0.0 Sep 13, 2024
mattiagiupponi added a commit that referenced this issue Oct 15, 2024
mattiagiupponi added a commit that referenced this issue Oct 15, 2024
mattiagiupponi added a commit that referenced this issue Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant