Skip to content

Chore: create removeWebdavAccount endpoint - #26393

Merged
kodiakhq[bot] merged 25 commits into
developfrom
chore/create-webdav-endpoint
Aug 19, 2022
Merged

kodiakhq[bot] merged 25 commits into
developfrom
chore/create-webdav-endpoint

Conversation

@felipe-rod123

Copy link
Copy Markdown
Contributor

Proposed changes (including videos or screenshots)

Created the '/v1/webdav.removeWebdavAccount' endpoint for the apps/meteor/client/views/account/integrations/AccountIntegrationsPage.tsx file, and added Ajv validations.

Issue(s)

Steps to test or reproduce

Further comments

@dougfabris dougfabris self-assigned this Aug 10, 2022
@felipe-rod123
felipe-rod123 marked this pull request as ready for review August 11, 2022 19:21
@felipe-rod123
felipe-rod123 requested review from a team as code owners August 11, 2022 19:21
@codecov

codecov Bot commented Aug 11, 2022 •

Copy link
Copy Markdown

Codecov Report

Merging #26393 (c8d628c) into develop (bfbade5) will increase coverage by 0.14%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #26393      +/-   ##
===========================================
+ Coverage    38.57%   38.72%   +0.14%     
===========================================
  Files          758      758              
  Lines        18783    18783              
  Branches      1453     1453              
===========================================
+ Hits          7246     7273      +27     
+ Misses       11322    11293      -29     
- Partials       215      217       +2     
Flag Coverage Δ
e2e 38.72% <ø> (+0.14%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Comment thread apps/meteor/tests/end-to-end/api/17-webdav.js Outdated
@ggazzo ggazzo added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Aug 18, 2022
@kodiakhq
kodiakhq Bot merged commit f40006c into develop Aug 19, 2022
@kodiakhq
kodiakhq Bot deleted the chore/create-webdav-endpoint branch August 19, 2022 21:10
csuadev pushed a commit that referenced this pull request Aug 26, 2022
Co-authored-by: Guilherme Gazzo <5263975+ggazzo@users.noreply.github.com>
Co-authored-by: Douglas Fabris <27704687+dougfabris@users.noreply.github.com>
@murtaza98 murtaza98 mentioned this pull request Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants