Skip to content

Add support for share contact message#1013

Merged
nathantannar4 merged 7 commits into
MessageKit:3.0.0-betafrom
TAGonSoft:share_contact
Apr 5, 2019
Merged

Add support for share contact message#1013
nathantannar4 merged 7 commits into
MessageKit:3.0.0-betafrom
TAGonSoft:share_contact

Conversation

@moldovaniosif

Copy link
Copy Markdown
Contributor

What does this implement/fix? Explain your changes.

It add support for share contact message. It implements UI for the share contact cell.

Does this close any currently open issues?

NO

Where has this been tested?

Devices/Simulators: … iPhone XS Simulator, iPhone XS device

iOS Version: …. 12.1.4

Swift Version: …. 4.2

MessageKit Version: …. 3.0.0 beta

Share contact cell UI:
Simulator Screen Shot - 1_iPhone Xs 12 1 - 2019-03-12 at 15 11 10

…are_contact

# Conflicts:
#	Example/Sources/Data Generation/SampleData.swift
#	Sources/Views/Cells/AudioMessageCell.swift
@moldovaniosif

Copy link
Copy Markdown
Contributor Author

I don't know why CircleCi it complains about not founding MessageInputBar.

@nathantannar4

Copy link
Copy Markdown
Member

Ya ignore circle for now I haven't set it up right for the dependency changes. This looks promising! Will for sure include in 3.0. Busy right now but I'll take a look soon

Sent with GitHawk

@nathantannar4 nathantannar4 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just looked over the code. Haven't tested yet.

Comment thread Sources/Extensions/UIImage+Extension.swift Outdated
Comment thread Sources/Protocols/ContactItem.swift Outdated
Comment thread Sources/Protocols/ContactItem.swift Outdated
Comment thread Sources/Protocols/ContactItem.swift Outdated
Comment thread Sources/Protocols/ContactItem.swift Outdated
Comment thread Sources/Views/Cells/ContactMessageCell.swift Outdated
Comment thread Sources/Views/Cells/ContactMessageCell.swift Outdated
Comment thread Sources/Views/Cells/ContactMessageCell.swift Outdated
Comment thread Sources/Views/Cells/ContactMessageCell.swift Outdated
Comment thread Sources/Views/Cells/ContactMessageCell.swift Outdated
Comment thread Sources/Views/Cells/ContactMessageCell.swift Outdated
@nathantannar4 nathantannar4 mentioned this pull request Mar 25, 2019
4 tasks
…into share_contact

# Conflicts:
#	Example/Sources/Data Generation/SampleData.swift
@nathantannar4

Copy link
Copy Markdown
Member

@moldovaniosif You have some build errors with the functions you are using to setup constraints

@nathantannar4 nathantannar4 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please be careful merging in other branches to yours. It looks like you merged in the master branch (or started from) and this thus reverts changes made in the 3.0.0-beta branch

Comment thread Cartfile.resolved Outdated
Comment thread Sources/Extensions/UIView+Extensions.swift
Comment thread Sources/Models/Sender.swift
Comment thread Sources/Protocols/ContactItem.swift Outdated
Comment thread Sources/Views/Cells/AudioMessageCell.swift Outdated
Comment thread Sources/Views/Cells/AudioMessageCell.swift Outdated
@moldovaniosif

moldovaniosif commented Apr 4, 2019

Copy link
Copy Markdown
Contributor Author

The merge with mater was a mistake, I reverted those commits.

/// contact last name
var lastName: String? { get }
/// initials from contact first and last name
var initials: String { get }

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice! I like this better

@nathantannar4 nathantannar4 merged commit 231e4ca into MessageKit:3.0.0-beta Apr 5, 2019
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.

2 participants