Skip to content

Remove Kennisnet verification#151

Closed
ATimmeh33 wants to merge 1 commit intomasterfrom
SIG-2558-Remove-kennisnet
Closed

Remove Kennisnet verification#151
ATimmeh33 wants to merge 1 commit intomasterfrom
SIG-2558-Remove-kennisnet

Conversation

@ATimmeh33
Copy link
Copy Markdown
Collaborator

No description provided.

@ATimmeh33 ATimmeh33 added this to the SemVer 5.0 milestone Sep 26, 2025
@ATimmeh33 ATimmeh33 requested a review from Copilot September 26, 2025 15:38
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes Kennisnet verification functionality from the SignhostAPIClient, following the deprecation notice that indicated it would be removed in SemVer 4.

  • Removes the obsolete KennisnetVerification class
  • Removes RequireKennisnetVerification property from test request bodies and API responses
  • Cleans up test data to match the updated API structure

Reviewed Changes

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

File Description
KennisnetVerification.cs Completely removes the obsolete KennisnetVerification class
RequestBodies.resx Removes RequireKennisnetVerification from test request JSON data
APIResponses.resx Removes RequireKennisnetVerification from test response JSON data

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ATimmeh33
Copy link
Copy Markdown
Collaborator Author

Now part of #153

@ATimmeh33 ATimmeh33 closed this Nov 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants