Skip to content

SQ2-1065 - redesign API around Kira::V2::Client (v3.0 breaking)#14

Merged
jmnsf merged 1 commit into
masterfrom
sq2-1065-v3-redesign
May 14, 2026
Merged

SQ2-1065 - redesign API around Kira::V2::Client (v3.0 breaking)#14
jmnsf merged 1 commit into
masterfrom
sq2-1065-v3-redesign

Conversation

@jmnsf

@jmnsf jmnsf commented May 12, 2026

Copy link
Copy Markdown
Contributor

Stacked on #13 (SQ2-1047), #12 (SQ2-1052), #11 (SQ2-1050), #10 (SQ2-1048). Top of the stack.

Major: changes the API surface to a more RESTful format. Instead of instancing a resource module with the API token etc, every resource is now reachable via a Kira::V2::Client class.

API

client = Kira::V2::Client.new(token: "...", base_url: nil)
interview = client.interview(id)

interview.applicants.create(first_name:, last_name:, email:, external_id: nil)
interview.webhooks.list
interview.webhooks.create(endpoint:, event_subscriptions:, secret:, active: true)
interview.webhooks.delete(uid)

@linear

linear Bot commented May 12, 2026

Copy link
Copy Markdown

SQ2-1065

@jmnsf jmnsf self-assigned this May 12, 2026
@jmnsf jmnsf marked this pull request as ready for review May 12, 2026 15:31
@jmnsf jmnsf changed the title SQ2-1065 - feat!: redesign API around Kira::V2::Client (v3.0 breaking) SQ2-1065 - redesign API around Kira::V2::Client (v3.0 breaking) May 12, 2026
@jmnsf jmnsf force-pushed the sq2-1047-faraday-bump branch from 1565bd5 to 72c28a8 Compare May 13, 2026 10:56
@jmnsf jmnsf force-pushed the sq2-1065-v3-redesign branch from 628d152 to 1507683 Compare May 13, 2026 10:57
@jmnsf jmnsf changed the base branch from sq2-1047-faraday-bump to master May 13, 2026 10:58

@migalvalm migalvalm left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM ✅ just a minor question

Comment thread lib/kira/v2/client.rb
@jmnsf jmnsf merged commit 9ff6bc0 into master May 14, 2026
1 check passed
@jmnsf jmnsf deleted the sq2-1065-v3-redesign branch May 14, 2026 11:43
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