Dark Mode

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

refactor: fix language issues#28

Open
LoraTod wants to merge 1 commit intomainfrom
fix-lang
Open

refactor: fix language issues#28
LoraTod wants to merge 1 commit intomainfrom
fix-lang

Conversation

Copy link

LoraTod commented Feb 5, 2026

No description provided.

LoraTod force-pushed the fix-lang branch from 9e73f43 to ba3e3bd Compare February 9, 2026 12:54
Foriger force-pushed the fix-lang branch 3 times, most recently from 30eb66f to 0a2030c Compare February 11, 2026 09:15
Foriger requested changes Feb 16, 2026
LoraTod force-pushed the fix-lang branch from 0a2030c to 6d69856 Compare February 16, 2026 15:20
Foriger previously approved these changes Feb 16, 2026
tatiNo5 reviewed Feb 16, 2026
/// Custom text specified in the MIRACL Trust Portal for the custom verification.
public let verificationCustomText: String

/// Label of the identity which will be used for identity verification.
Copy link

tatiNo5 Feb 16, 2026

Choose a reason for hiding this comment

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

I'm not sure should I follow the portal or the documentation vocabulary for the terminology but in both places we say User Verification, not Identity Verification. IMO we need to use the same terms everywhere and this is easy to be mixed. Having the fact we have nowhere (at least I don't know) described our terminology, it could be confusing for s.o. what is what.

Copy link

tatiNo5 Feb 16, 2026

Choose a reason for hiding this comment

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

Having in mind the above, here it should be User ID Type, not Identity Type.

/// Custom text specified in the MIRACL Trust Portal for the custom verification.
var verificationCustomText: String { get }

/// Label of the identity which will be used for identity verification.
Copy link

tatiNo5 Feb 16, 2026

Choose a reason for hiding this comment

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

It's the same here and I won't comment more about the topic as I suppose we need to decide what language to use and how to proceed with the topic further. @Foriger

// MARK: SDK Configuration

/// Getting singleton instance of the MIRACLTrust class.
/// Gets singleton instance of the MIRACLTrust class.
Copy link

tatiNo5 Feb 16, 2026

Choose a reason for hiding this comment

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

Gets a singleton instance.. ?

/// - Parameter configuration:object storing configurations of the SDK.
/// Configures the SDK with values issued by MIRACL Trust and stored in the ``Configuration`` object.
/// It is recommended to call this method after the application is launched.
/// - Parameter configuration: object storing configurations of the SDK.
Copy link

tatiNo5 Feb 16, 2026

Choose a reason for hiding this comment

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

an object storing...?

///
/// Use this method to authenticate another device or application with the usage of
/// Universal Link created by a MIRACL platform.
/// Use this method to authenticate another device or application using an
Copy link

tatiNo5 Feb 16, 2026

Choose a reason for hiding this comment

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

**a** universal link is corrected bellow

Foriger dismissed their stale review via 7bc8a50 March 2, 2026 09:33
Foriger force-pushed the fix-lang branch from 6d69856 to 7bc8a50 Compare March 2, 2026 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

tatiNo5 tatiNo5 left review comments

Foriger Foriger left review comments

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants