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

Added update function to REST interface #1058

Merged
merged 4 commits into from
Aug 14, 2020

Conversation

lgoltz
Copy link
Contributor

@lgoltz lgoltz commented Apr 9, 2020

This PR enhances the REST interface by an update function. Only changed files are reinitialized.

This PR includes #1057

This PR based on the work of MrSnyder, Thanks!

@stempler
Copy link
Member

stempler commented Apr 9, 2020

If you wanted to honor the authorship of @MrSnyder you could have cherry-picked his commits and done your changes based on his.

@lgoltz
Copy link
Contributor Author

lgoltz commented Apr 9, 2020

If you wanted to honor the authorship of @MrSnyder you could have cherry-picked his commits and done your changes based on his.

We did this but had to merge the commits into one cause of conflicts. We use this code in two other forks and liked to bring back this changes to the community.
But we will try to set the author of the commit to @MrSnyder and lgoltz as co-author, as the commit includes some changes of the original commit.

@dstenger dstenger force-pushed the RESTUpdateFunction-152 branch from e927427 to a692cbe Compare April 9, 2020 13:41
@dstenger
Copy link
Contributor

dstenger commented Apr 9, 2020

Pull request was updated. Hope this status is fine for everyone.

@tfr42 tfr42 added console deegree administration console enhancement enhancement or improvement labels Apr 17, 2020
@copierrj copierrj merged commit 2bdd14a into deegree:master Aug 14, 2020
@tfr42 tfr42 added this to the 3.4.14 milestone Aug 14, 2020
@lgoltz lgoltz deleted the RESTUpdateFunction-152 branch December 8, 2022 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
console deegree administration console enhancement enhancement or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants