Skip to content

Conversation

@LukasReschke
Copy link
Member

@jospoortvliet Thoughts? We talked about that in the S-Bahn in Stuttgart on Friday :-)

If okay I'll merge this also on other repositories. But make sure the text is fine first :)

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
@LukasReschke LukasReschke added the 3. to review Waiting for reviews label Apr 2, 2017
@mention-bot
Copy link

@LukasReschke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @nickvergessen, @Finkregh and @MariusBluem to be potential reviewers.

Copy link
Member

@MariusBluem MariusBluem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See my comment ... Rest seems to be fine 👍

Insert your config.php content here
(Without the database password, passwordsalt and secret)
Insert your config.php content here.
Make sure to remove all sensitive contents such as passwords. (e.g. database password, passwordsalt, secret, smtp password, …)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

content is uncountable. 😁

@codecov-io
Copy link

Codecov Report

Merging #4189 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #4189   +/-   ##
=========================================
  Coverage     53.96%   53.96%           
  Complexity    21266    21266           
=========================================
  Files          1259     1259           
  Lines         74136    74136           
=========================================
  Hits          40009    40009           
  Misses        34127    34127

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f31dafe...7909d4e. Read the comment docs.

@jospoortvliet jospoortvliet merged commit 13e7a93 into master Apr 3, 2017
@jospoortvliet jospoortvliet deleted the add-issue-template-app branch April 3, 2017 10:12
@jospoortvliet
Copy link
Member

TODO: port to the other repo's. Will work on that later today.

jospoortvliet pushed a commit to nextcloud/calendar that referenced this pull request Apr 4, 2017
Privacy is a thing and so is users forgetting about it ;-)

ref nextcloud/server#4189
jospoortvliet pushed a commit to nextcloud/notifications that referenced this pull request Apr 4, 2017
Privacy is a thing and so is users forgetting about it ;-)

ref nextcloud/server#4189
@jospoortvliet
Copy link
Member

All reviewed and merged. I pronounce this action DONE ;-)

DeepDiver1975 pushed a commit to owncloud/calendar that referenced this pull request Aug 22, 2017
Privacy is a thing and so is users forgetting about it ;-)

ref nextcloud/server#4189
DeepDiver1975 pushed a commit to owncloud/calendar that referenced this pull request Aug 25, 2017
Privacy is a thing and so is users forgetting about it ;-)

ref nextcloud/server#4189
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants