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

refactor: explicit firstComment argument in createDiscussion() #3292

Merged

Conversation

samuelstroschein
Copy link
Member

closes opral/lix-sdk#164

It was unclear that the content argument in createDiscussion() was meant to be the first comment. This change makes it explicit by renaming the argument to firstComment.

closes opral/lix-sdk#164

It was unclear that the `content` argument in `createDiscussion()` was meant to be the first comment. This change makes it explicit by renaming the argument to `firstComment`.
Copy link

changeset-bot bot commented Dec 19, 2024

🦋 Changeset detected

Latest commit: 2d3ab95

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

This PR includes changesets to release 7 packages
Name Type
@lix-js/sdk Patch
csv-app Patch
lix-file-manager Patch
@lix-js/host Patch
@lix-js/plugin-csv Patch
@lix-js/sdk-example-server-protocol-handler-hono Patch
lix-website-server 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

Copy link

nx-cloud bot commented Dec 19, 2024

View your CI Pipeline Execution ↗ for commit 2d3ab95.

Command Status Duration Result
nx run-many --nx-bail --target=build --parallel ✅ Succeeded 7s View ↗
nx run-many --target=test --parallel ✅ Succeeded 22s View ↗
nx run-many --target=lint --parallel ✅ Succeeded 1s View ↗

☁️ Nx Cloud last updated this comment at 2024-12-19 16:44:54 UTC

@samuelstroschein samuelstroschein merged commit 76a3ecc into main Dec 19, 2024
3 checks passed
@samuelstroschein samuelstroschein deleted the lixdk-257-creatediscussion-explicit-firstcomment-prop branch December 19, 2024 16:45
@github-actions github-actions bot locked and limited conversation to collaborators Dec 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

createDiscussion explicit firstComment prop
1 participant