Skip to content

Declare id field as abstract#203

Merged
thjaeckle merged 1 commit intoeclipse-ditto:masterfrom
vhdirk:TS2611
Nov 15, 2022
Merged

Declare id field as abstract#203
thjaeckle merged 1 commit intoeclipse-ditto:masterfrom
vhdirk:TS2611

Conversation

@vhdirk
Copy link
Contributor

@vhdirk vhdirk commented Nov 15, 2022

Fixes #202

Signed-off-by: Dirk Van Haerenborgh [email protected]

Signed-off-by: Dirk Van Haerenborgh <[email protected]>
@thjaeckle
Copy link
Member

Thanks for providing the fix, @vhdirk
Could you verify that the JS client works for you with that fix?

I would like to provide not too many bugfix releases and my TS knowledge is quite limited

@thjaeckle thjaeckle added bug Something isn't working javascript labels Nov 15, 2022
@vhdirk
Copy link
Contributor Author

vhdirk commented Nov 15, 2022

@thjaeckle I've just built and tested this against our application. I am happy to report this fully works as expected :)

Copy link
Member

@thjaeckle thjaeckle left a comment

Choose a reason for hiding this comment

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

Awesome, a thousand thanks 👍

@thjaeckle thjaeckle merged commit c91933d into eclipse-ditto:master Nov 15, 2022
@vhdirk vhdirk deleted the TS2611 branch November 16, 2022 12:40
@thjaeckle thjaeckle added this to the 3.0.1 milestone Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working javascript

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[JS]: incompatible with typscript 4.7.4

2 participants