Skip to content
This repository was archived by the owner on May 9, 2026. It is now read-only.

Add simpleClient into the WBK function#141

Merged
maxlath merged 7 commits into
maxlath:mainfrom
ogroppo:simple-client
May 9, 2026
Merged

Add simpleClient into the WBK function#141
maxlath merged 7 commits into
maxlath:mainfrom
ogroppo:simple-client

Conversation

@ogroppo
Copy link
Copy Markdown
Contributor

@ogroppo ogroppo commented Mar 25, 2026

Closes #137

Comment thread src/types/options.ts
Comment thread src/client.ts
Comment thread src/index.ts
Comment thread src/simple_client.ts Outdated
Comment thread src/wikibase-sdk.ts
Comment thread src/simple_client.ts Outdated
Comment thread src/simple_client.ts Outdated
Comment thread src/wikibase-sdk.ts
Comment thread src/client.ts Outdated
readonly getEntitiesFromSitelinks: (options: GetEntitiesFromSitelinksOptions) => Promise<WbGetEntitiesResponse>
readonly sparqlQuery: (sparql: string) => Promise<SparqlResults>
readonly getReverseClaims: (options: GetReverseClaimsOptions) => Promise<SparqlResults>
}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

inferred type at the end of file

@ogroppo ogroppo requested a review from maxlath March 27, 2026 22:21
@ogroppo
Copy link
Copy Markdown
Contributor Author

ogroppo commented Apr 12, 2026

Hi @maxlath do you have more comments on this PR?

@maxlath
Copy link
Copy Markdown
Owner

maxlath commented Apr 15, 2026

It seems acbed81 is redundant with what I already push on the main branch in eb7481f: can you rebase this branch on main to make the changes in this PR clearer?

Comment thread tests/simple_client.ts Outdated
Comment thread tests/simple_client.ts Outdated
Comment thread tests/simple_client.ts Outdated
Comment thread tests/simple_client.ts Outdated
Comment thread tests/simple_client.ts Outdated
Comment thread tests/simple_client.ts Outdated
Copy link
Copy Markdown
Contributor Author

@ogroppo ogroppo left a comment

Choose a reason for hiding this comment

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

All comments addressed
Image

@ogroppo
Copy link
Copy Markdown
Contributor Author

ogroppo commented May 2, 2026

Hi @maxlath anything else you need me to add? when can we aim to get this published?

@maxlath maxlath merged commit 6353b26 into maxlath:main May 9, 2026
@maxlath
Copy link
Copy Markdown
Owner

maxlath commented May 9, 2026

Merged and published in v11.4.0.

I was waiting for this PR to be closed to start the repository transfer to Codeberg (where I already moved my other repositories): please open any further PR there https://codeberg.org/maxlath/wikibase-sdk

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.

Are you open to add fetch functions for all methods?

2 participants