Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: editing comment when GITLAB_COMMENT_TYPE is note #100

Merged
merged 1 commit into from
Mar 20, 2023

Conversation

QuiiBz
Copy link
Contributor

@QuiiBz QuiiBz commented Mar 20, 2023

Fix a bug introduced by #98

When GITLAB_COMMENT_TYPE is note and we have already created the note, it fails to update it with the following error: "note_id is invalid".

This was caused by MergeRequestDiscussions.editNote & MergeRequestNotes.note not having the same signature. The former takes both the discussion id and note id, but the latter only takes the note id.

@changeset-bot
Copy link

changeset-bot bot commented Mar 20, 2023

🦋 Changeset detected

Latest commit: 089be37

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
changesets-gitlab Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@stackblitz
Copy link

stackblitz bot commented Mar 20, 2023

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@JounQin JounQin merged commit 4498c2f into un-ts:main Mar 20, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants