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

feat(engine+rest): engine and rest query for deploymenttime #118

Closed
wants to merge 3 commits into from
Closed

feat(engine+rest): engine and rest query for deploymenttime #118

wants to merge 3 commits into from

Conversation

ingorichtsmeier
Copy link
Contributor

No description provided.

@buildhive
Copy link

camunda BPM » camunda-bpm-platform #959 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@buildhive
Copy link

camunda BPM » camunda-bpm-platform #961 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@ingorichtsmeier ingorichtsmeier changed the title feat(engine): query for deployment time now with engine and rest query for deploymenttime Oct 27, 2014
@ingorichtsmeier ingorichtsmeier changed the title now with engine and rest query for deploymenttime feat(engine+rest): engine and rest query for deploymenttime Oct 28, 2014
@menski
Copy link
Contributor

menski commented Oct 29, 2014

Hi Ingo,

thanks for the pull request. I created a JIRA ticket for it CAM-2968.

Cheers,
Sebastian

@menski
Copy link
Contributor

menski commented Oct 30, 2014

Hi Ingo,

I merged your changes (cd8bce6).

I have two notes:

  • first it would be nice if you could synchronize your fork with the current upstream master, your master is 4 month behind
  • second you missed to update the REST docs for the deployment count query, mostly we have every query two times and sometimes even four times if the query supports POST requests

But again thank you for your contribution.

Cheers,
Sebastian

@menski menski closed this Oct 30, 2014
tmetzke pushed a commit that referenced this pull request Feb 13, 2020
@osteinhauer
Merge branch 'master' into release200
451f82d 
* master:
  added configuration migration hint
  complete authorization configuration #118 as mentioned in #35
  added property for default serialization format closes #127
  added property for authorization enabling (defaults to true). #118
  add documentation for camunda-bpm-spring-boot-starter-rest closes #110
  changed configuration docs to meet #86
  to get a first feeling for ProcessEnginePlugin as configuration replacement #56
  introduce properties for metrics (#124)
  use of constants for commit 9198b67
  added dependencies for webapp-classes (required in 7.6)
  removed usage of internal implementation classes (one at least changes with 7.6)
  add metrics configuration to context, fixes #122
  add enabled comment, fixes #46
  introduce properties for metrics (#124)
  add Roadmap
  separate compile for extensions and examples
  typo
  to get a first feeling for ProcessEnginePlugin as configuration replacement #56
  typo
  to get a first feeling for ProcessEnginePlugin as configuration replacement #56

# Conflicts:
#	README.md
#	examples/pom.xml
#	extension/starter/src/main/java/org/camunda/bpm/spring/boot/starter/configuration/CamundaConfiguration.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ingorichtsmeier @buildhive @menski