Skip to content

2026 Updates - Part 2 - #2

Merged
zachcmielreverb merged 1 commit into
mainfrom
zc/listing-detail
Jul 23, 2026
Merged

zachcmielreverb merged 1 commit into
mainfrom
zc/listing-detail

Conversation

@zachcmielreverb

Copy link
Copy Markdown
Contributor

Adds a ListingsDetailRequest and ListingsDetailResponse

@zachcmielreverb
zachcmielreverb requested a review from a team July 23, 2026 17:13
@zachcmielreverb zachcmielreverb self-assigned this Jul 23, 2026
@zachcmielreverb
zachcmielreverb requested review from Copilot and reverb-ios-robot and removed request for a team July 23, 2026 17:14
@zachcmielreverb
zachcmielreverb requested review from JohnBueno and nathanhreverb and removed request for reverb-ios-robot July 23, 2026 17:14

Copilot AI 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.

Pull request overview

Adds GraphQL request/response models for a listing detail endpoint, enabling the app’s networking layer to represent (encode/decode) listing detail queries similarly to the existing listings search flow.

Changes:

  • Added ListingsDetailRequest (GraphQL query + variables) for fetching listing detail by ID.
  • Added ListingsDetailResponse (decodable model hierarchy) for parsing the listing detail GraphQL response.
  • Registered the new Swift files in the Xcode project so they build with the target.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
Interview/Networking/ListingsDetailRequest.swift Adds an encodable GraphQL request for listing detail by ID.
Interview/Networking/ListingsDetailResponse.swift Adds decodable models for the listing detail GraphQL response payload.
Interview.xcodeproj/project.pbxproj Includes the new request/response files in the project and build sources.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Interview/Networking/ListingsDetailRequest.swift
Comment thread Interview/Networking/ListingsDetailResponse.swift
@zachcmielreverb
zachcmielreverb merged commit c600d51 into main Jul 23, 2026
2 checks passed
@zachcmielreverb
zachcmielreverb deleted the zc/listing-detail branch July 23, 2026 17:20
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.

3 participants