Skip to content

Add Lemmy 1.0.0 support - #87

Merged
MV-GH merged 104 commits into
masterfrom
lemmy-1.0.0
Oct 17, 2026
Merged

MV-GH merged 104 commits into
masterfrom
lemmy-1.0.0

Conversation

@MV-GH

@MV-GH MV-GH commented May 24, 2026 •

Copy link
Copy Markdown
Owner

Seems to work in initial basic tests.

its deployed https://central.sonatype.com/artifact/it.vercruysse.lemmyapi/lemmy-api-jvm

Also fixes: #67

@dessalines

dessalines commented May 26, 2026 •

Copy link
Copy Markdown
Contributor

Nice, thx. We'll try to get the version stuff figured out quickly. Unfortunately I gotta go back to work on lemmy-ui again as a lot of issues just popped up there.

@dessalines

Copy link
Copy Markdown
Contributor

Another thing I'm noticing working through jerboa, the CommentActions aren't matching.

lemmy 1.0 uses CommentActions.vote_is_upvote: Boolean? , but in this PR its val like_score: Int = 0

@MV-GH

MV-GH commented Jul 7, 2026

Copy link
Copy Markdown
Owner Author

Eh didnt really like the true|null|false, so kept the previous behaviour to reduce the refactoring burden. But I dont rlly mind it. But I would prob make into an enum (Upvote/Downvote/NoVote)

@dessalines

Copy link
Copy Markdown
Contributor

An enum would be fine too, although IMO its best to use as minimal transforms from the original API as possible. Up to you.

@dessalines

Copy link
Copy Markdown
Contributor

Another minor issue that means this might need to be regenerated: community title is now optional.

@dessalines

Copy link
Copy Markdown
Contributor

Another related to votes: VoteView has pub is_upvote: bool , but in the API its still a numeric score

@dessalines

Copy link
Copy Markdown
Contributor

Sweet, thx.

@dessalines

dessalines commented Jul 11, 2026 •

Copy link
Copy Markdown
Contributor

Nice thx, I'll start adding the vote changes in my 1.0 dev branch soon.

@MV-GH

MV-GH commented Jul 18, 2026

Copy link
Copy Markdown
Owner Author

I released 0.6.0-beta.3, it doesn't contain any v1 changes but I have refactored the public API, see the README how it looks now. It also includes the Result<> changes. No more unexpected errors.

You can leave it for me, if you cant figure it out. You can stay on that version for now.

@MV-GH
MV-GH merged commit f5335b8 into master Oct 17, 2026
5 checks passed
@MV-GH
MV-GH deleted the lemmy-1.0.0 branch October 17, 2026 16:30
@dessalines

Copy link
Copy Markdown
Contributor

Sweet, I'm still working on jerboa obvi, so I'll let you know in other places if I have any other API related issues.

@MV-GH

MV-GH commented Oct 19, 2026

Copy link
Copy Markdown
Owner Author

Sure, I have also begun with some initial End to End tests with real Lemmy local instance (docker). So ill have confidence that it works before actually having to try it out in Jerboa

MV-GH added a commit that referenced this pull request Oct 19, 2026
Add Lemmy 1.0.0 support.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
MV-GH added a commit that referenced this pull request Oct 19, 2026
MV-GH added a commit that referenced this pull request Oct 19, 2026
MV-GH added a commit that referenced this pull request Oct 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use Result do not throw exceptions

2 participants