Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: 🎸 add new control ContactItem #307

Merged
merged 4 commits into from
Jul 1, 2021
Merged

Conversation

shengxu7
Copy link
Member

No description provided.

@shengxu7 shengxu7 requested a review from a team as a code owner June 30, 2021 00:03
@shengxu7 shengxu7 requested review from dyongxu and removed request for a team June 30, 2021 00:03
case email
case message
case videoCall
case detail
Copy link

Choose a reason for hiding this comment

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

Missing Docs Violation: public declarations should be documented. (missing_docs)

case email
case message
case videoCall
Copy link

Choose a reason for hiding this comment

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

Missing Docs Violation: public declarations should be documented. (missing_docs)

case email
case message
Copy link

Choose a reason for hiding this comment

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

Missing Docs Violation: public declarations should be documented. (missing_docs)

@@ -2,13 +2,25 @@ import SwiftUI

/// :nodoc:
public enum ActivityItemType {
case phone
Copy link

Choose a reason for hiding this comment

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

Missing Docs Violation: public declarations should be documented. (missing_docs)

@MarcoEidinger MarcoEidinger requested review from billzhou0223 and MarcoEidinger and removed request for dyongxu June 30, 2021 17:16
@MarcoEidinger
Copy link
Member

@shengxu7 please fix build errors

Copy link
Contributor

@billzhou0223 billzhou0223 left a comment

Choose a reason for hiding this comment

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

@shengxu7 Can you also update the table in the readme to indicate that ContactItem is completed?

@shengxu7
Copy link
Member Author

shengxu7 commented Jul 1, 2021

@shengxu7 Can you also update the table in the readme to indicate that ContactItem is completed?

Updated.

@shengxu7 shengxu7 closed this Jul 1, 2021
@shengxu7 shengxu7 reopened this Jul 1, 2021
descriptionText

Spacer(minLength: 24)
}.frame(width: mainViewSize.width * (1 - self.splitPercent!))
Copy link

Choose a reason for hiding this comment

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

Force Unwrapping Violation: Force unwrapping should be avoided. (force_unwrapping)

}
.frame(width: mainViewSize.width * self.splitPercent!)
Copy link

Choose a reason for hiding this comment

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

Force Unwrapping Violation: Force unwrapping should be avoided. (force_unwrapping)

@billzhou0223 billzhou0223 merged commit 15eebe3 into SAP:main Jul 1, 2021
@shengxu7 shengxu7 deleted the ContactItem branch July 1, 2021 19:35
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