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

feat: 3.0.0 #178

Merged
merged 49 commits into from
Nov 16, 2024
Merged

feat: 3.0.0 #178

merged 49 commits into from
Nov 16, 2024

Conversation

Bugs5382
Copy link
Owner

@Bugs5382 Bugs5382 commented Nov 3, 2024

Search terms

Questioner

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Other things:

  • Write a unit test or update unit tests that cover your change in the code?
  • Set the PR to merge into the develop branch?
  • Clear documentation per the guidelines in the README.md as we are support medical applications?

Bugs5382 and others added 30 commits June 29, 2024 00:56
- Included example and Unit test

Resolves: #48
- added 2 checks missing in the msh header from 2.3
- also refactors code in 2.3 and 2.3.1 for building the MSH
Bumps [typescript](https://github.com/Microsoft/TypeScript) from 5.5.2 to 5.5.3.
- [Release notes](https://github.com/Microsoft/TypeScript/releases)
- [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml)
- [Commits](microsoft/TypeScript@v5.5.2...v5.5.3)

---
updated-dependencies:
- dependency-name: typescript
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [typescript](https://github.com/Microsoft/TypeScript) from 5.5.3 to 5.5.4.
- [Release notes](https://github.com/Microsoft/TypeScript/releases)
- [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml)
- [Commits](microsoft/TypeScript@v5.5.3...v5.5.4)

---
updated-dependencies:
- dependency-name: typescript
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
* default 0 second idle time keeps the connection open
* disables connection timeout logic if set to 0 (stay connected)
* updated unit test to reflect that

BREAKING CHANGE: Default now set to be always connected. Must set connectionTimeout, if you want your client to disconnect after a certain time. The max is still 60 seconds.

Closes #169
# [3.0.0-beta.1](v2.3.1...v3.0.0-beta.1) (2024-10-31)

### Features

* client connection ([e6ee6a3](e6ee6a3)), closes [#169](#169)
* client connection ([#175](#175)) ([aeb4108](aeb4108))

### BREAKING CHANGES

* Default now set to be always connected. Must set connectionTimeout, if you want your client to disconnect after a certain time. The max is still 60 seconds.
- created a codec for MLLP data
- faster than the old method and can make sure very long messages are parsed
- removed any doubt of order of operations when getting a response back

[skip ci]
[skip ci]
- have to stop these units tests so we can get a new release out of this package. we will turn this on once everything is synced up.
- since node-hl7-server is  dev dependency it's ok to do this
- all the tests pass during local development version
- ci fix I hope
@Bugs5382 Bugs5382 self-assigned this Nov 3, 2024
@Bugs5382 Bugs5382 changed the title Develop feat: 3.0.0 Nov 3, 2024
[skip ci]
@Bugs5382 Bugs5382 marked this pull request as draft November 3, 2024 01:15
@Bugs5382 Bugs5382 marked this pull request as ready for review November 3, 2024 01:15
@Bugs5382 Bugs5382 marked this pull request as draft November 8, 2024 20:28
@Bugs5382 Bugs5382 marked this pull request as ready for review November 8, 2024 20:28
@Bugs5382 Bugs5382 closed this Nov 8, 2024
@Bugs5382 Bugs5382 reopened this Nov 8, 2024
@Bugs5382 Bugs5382 added enhancement New feature or request major labels Nov 16, 2024
@Bugs5382 Bugs5382 merged commit 4715907 into main Nov 16, 2024
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request major
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants