Skip to content

Fix Sequence Tests#151

Merged
n7studios merged 11 commits into
masterfrom
fix-sequence-tests
Jul 17, 2026
Merged

Fix Sequence Tests#151
n7studios merged 11 commits into
masterfrom
fix-sequence-tests

Conversation

@n7studios

Copy link
Copy Markdown
Contributor

Summary

Fixes the create/update/delete sequence email test since the API changes rightly no longer permit setting a position of an existing email in a sequence to one that already exists.

1) ConvertKitAPIKeyTest::testCreateGetUpdateAndDeleteSequenceEmail
GuzzleHttp\Exception\ClientException: Client error: `PUT [https://api.kit.com/v4/sequences/1030824/emails/10087666`](https://api.kit.com/v4/sequences/1030824/emails/10087666%60) resulted in a `422 Unprocessable Content` response:
{"errors":["position 1 is already taken in this sequence"]}

Testing

Existing tests pass.

Checklist

@n7studios n7studios self-assigned this Jul 16, 2026
@n7studios n7studios added the bug label Jul 16, 2026
@n7studios
n7studios requested review from a team, ciccio-kit and noelherrick and removed request for a team July 16, 2026 09:50
@n7studios
n7studios marked this pull request as ready for review July 16, 2026 09:50
Filter Subscribers: Add `include` parameter support
Forms & Tags: Add `include` parameter support
@n7studios
n7studios merged commit 0f31ae2 into master Jul 17, 2026
@n7studios
n7studios deleted the fix-sequence-tests branch July 17, 2026 00:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants