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

US1 #9

Merged
merged 4 commits into from
May 8, 2020
Merged

US1 #9

merged 4 commits into from
May 8, 2020

Conversation

DMadrid
Copy link
Contributor

@DMadrid DMadrid commented May 8, 2020

Description

US1 - Refactor Express server to TS

Related US/Task

US1

Motivation and Context

Enables typing in express server

How Has This Been Tested

Manual Testing, Verified by Matthew

Types of changes

  • Refactoring / dependency upgrade / docs change
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Non-functional requirement

Visual screenshots

N/A

* WIP

* resolve ts errors

* Clean up

* Update types

* Update types

* Fix swagger docs

* Remove paths from tsconfig

Our dev environment cannot currently support ts config paths

* Finish cleanup of ts

* linting

* Add in missing lint files

* Update docker file

* rearrange swaggar docs

* add back in config

* APM Server missing message broke, this fixes it

* signed commits

Co-authored-by: Madrid <[email protected]>
@frisbm frisbm self-requested a review May 8, 2020 21:52
@DMadrid DMadrid linked an issue May 8, 2020 that may be closed by this pull request
@DMadrid DMadrid added the enhancement New feature or request label May 8, 2020
cyfhir/dependencies.txt Outdated Show resolved Hide resolved
@DMadrid DMadrid merged commit 3bc5f3e into master May 8, 2020
@DMadrid DMadrid deleted the us1 branch May 8, 2020 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

US1: Refactor Express Server to Typescript
2 participants