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!: Use the new API project with breaking changes #200

Merged
merged 2 commits into from
Sep 19, 2023
Merged

Conversation

taorepoara
Copy link
Member

@taorepoara taorepoara commented Sep 15, 2023

About this PR

Adapt to API breaking changes:

type => _type
action => listener
remove deprecated elements

How to test my changes

Build the Docker image and test it in a Lenra app (see the README).

Waiting for lenra-io/server#452

Checklist

  • I didn't over-scope my PR
  • My PR title matches the commit convention
  • I did not include breaking changes
  • I made my own code-review before requesting one

I included unit tests that cover my changes

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

I added/updated the documentation about my changes

  • 📜 README.md
  • 📕 docs/*.md
  • 📓 docs.lenra.io
  • 🙅 no documentation needed

@taorepoara taorepoara requested a review from pichoemr September 19, 2023 12:10
@taorepoara taorepoara marked this pull request as ready for review September 19, 2023 12:10
@taorepoara taorepoara merged commit 2dcd706 into beta Sep 19, 2023
@taorepoara taorepoara deleted the change-api branch September 19, 2023 12:40
@shiipou
Copy link
Contributor

shiipou commented Sep 19, 2023

🎉 This PR is included in version 1.0.0-beta.92 🎉

The release is available on:

Your semantic-release bot 📦🚀

taorepoara added a commit that referenced this pull request Oct 8, 2023
* feat!: Use the new API project with breaking changes

* build(deps): Define server version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request released on @beta
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants