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

DSEGOG-319 Expose Header #121

Merged
merged 10 commits into from
May 28, 2024

Conversation

MRichards99
Copy link
Collaborator

Very simple PR to expose the Content-Disposition header so the frontend can see the filename of the file for the export endpoint. Creating as a draft PR while its being tested by Kacper

@codecov-commenter
Copy link

codecov-commenter commented May 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.99%. Comparing base (315bae2) to head (b18ceef).

Additional details and impacted files
@@                       Coverage Diff                        @@
##           DSEGOG-316-parallel-mode-switch     #121   +/-   ##
================================================================
  Coverage                            94.99%   94.99%           
================================================================
  Files                                   50       50           
  Lines                                 2838     2838           
  Branches                               297      297           
================================================================
  Hits                                  2696     2696           
  Misses                                 105      105           
  Partials                                37       37           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

- This will fix the Safety CI job that reports this dependency contains a vulnerability
@MRichards99 MRichards99 changed the base branch from main to DSEGOG-316-parallel-mode-switch May 15, 2024 06:50
MRichards99 and others added 4 commits May 24, 2024 13:21
DSEGOG-307 Kill API Instances Cleanly in Echo Ingest Script
Bumps [requests](https://github.com/psf/requests) from 2.31.0 to 2.32.2.
- [Release notes](https://github.com/psf/requests/releases)
- [Changelog](https://github.com/psf/requests/blob/main/HISTORY.md)
- [Commits](psf/requests@v2.31.0...v2.32.2)

---
updated-dependencies:
- dependency-name: requests
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
….32.2

Bump requests from 2.31.0 to 2.32.2
@MRichards99 MRichards99 marked this pull request as ready for review May 28, 2024 07:58
@MRichards99
Copy link
Collaborator Author

Just made this PR ready for review as I've got confirmation that the change worked for the frontend team. I'll merge main into this branch to get the CI fully passing

Copy link
Contributor

@patrick-austin patrick-austin left a comment

Choose a reason for hiding this comment

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

It's showing changes from some of the other merges that were made against main but haven't yet been merged into the 316 branch. Not necessarily a problem, just pointing it out in case it affects the merge strategy.

@MRichards99 MRichards99 merged commit 28f58a0 into DSEGOG-316-parallel-mode-switch May 28, 2024
9 checks passed
@MRichards99 MRichards99 deleted the DSEGOG-319-expose-headers branch May 28, 2024 08:54
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.

None yet

3 participants