Skip to content

Add a lot of API types#145

Draft
ShadowJonathan wants to merge 3 commits intomlemgroup:masterfrom
ShadowJonathan:add-api-types
Draft

Add a lot of API types#145
ShadowJonathan wants to merge 3 commits intomlemgroup:masterfrom
ShadowJonathan:add-api-types

Conversation

@ShadowJonathan
Copy link
Collaborator

This adds a lot of types from mlemgroup/mlem#52, the only missing ones are a lot of the PM, User, Admin, and the Misc endpoints

@ShadowJonathan ShadowJonathan requested a review from mormaer June 14, 2023 12:13
// lemmy_db_schema::source::language::Language
struct APILanguage: Decodable {
let id: Int
let code: String
Copy link
Collaborator

Choose a reason for hiding this comment

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

I wonder if these codes will match what Apple give us back from the Natural Language framework, which appears to be BCP 47?

let federationEnabled: Bool
let federationDebug: Bool
let federationWorkerCount: Int
let captchaEnabled: Bool
Copy link
Collaborator

Choose a reason for hiding this comment

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

do you know where captcha is used, is it during registration? does it impact the application in any way for the operations we currently perform with the API?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I haven't implemented registration yet, there's captchas for endpoints, yeah

@ShadowJonathan ShadowJonathan marked this pull request as draft June 14, 2023 17:03
@ShadowJonathan ShadowJonathan marked this pull request as ready for review June 14, 2023 19:45
@ShadowJonathan ShadowJonathan requested a review from mormaer June 14, 2023 20:09
@ShadowJonathan ShadowJonathan marked this pull request as draft June 15, 2023 11:25
@ShadowJonathan
Copy link
Collaborator Author

I'm converting this to draft, so that I can implement the rest of the API types

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.

2 participants