Skip to content

Conversation

@seancrim23
Copy link

Description

Added functionality to be able to dump data dynamically from the database based on the model types that we pass. The results are then modified into a CSV format and are returned as the endpoint response.

Related PRs

N/A

Todos

  • [x ] Tests
  • Documentation

Deploy Notes

N/A

Steps to Test or Reproduce

  • When testing, please note that the model types that are passed as the argument for the endpoint ("/csv/:model_type") should have a capitalize first letter ex. Entity, User, etc.
commands to test PR

Impacted Areas in Application

List general components of the application that this PR will affect:

  • None

@codecov
Copy link

codecov bot commented Apr 16, 2020

Codecov Report

Merging #111 into master will decrease coverage by 1.67%.
The diff coverage is 41.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #111      +/-   ##
==========================================
- Coverage   67.34%   65.66%   -1.68%     
==========================================
  Files          12       13       +1     
  Lines         343      367      +24     
==========================================
+ Hits          231      241      +10     
- Misses        112      126      +14     
Impacted Files Coverage Δ
src/utils/index.js 79.31% <16.66%> (-16.35%) ⬇️
src/routes/csv.js 47.05% <47.05%> (ø)
src/routes/index.js 100.00% <100.00%> (ø)

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 4c0994c...13d40cd. Read the comment docs.

@seancrim23 seancrim23 requested a review from stoopidJSON April 17, 2020 13:32
@stoopidJSON
Copy link
Member

@all-contributors please add @seancrim23 for code

@allcontributors
Copy link
Contributor

@revjtanton

I've put up a pull request to add @seancrim23! 🎉

@stoopidJSON stoopidJSON merged commit dc8ffab into master Apr 17, 2020
@stoopidJSON stoopidJSON deleted the seancrim23/issue-99 branch April 17, 2020 19:54
@carorm carorm linked an issue Apr 17, 2020 that may be closed by this pull request
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Export to CSV (w/o roles, w/o questions)

3 participants