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

Extend model objects with CustomStringConvertible #8

Merged
merged 4 commits into from Oct 14, 2020

Conversation

larryonoff
Copy link
Contributor

No description provided.

@Skifcha
Copy link
Contributor

Skifcha commented Oct 8, 2020

@larryonoff thanks, we'll get back to it asap. ETA – beginning of the next week.

@Skifcha
Copy link
Contributor

Skifcha commented Oct 13, 2020

@larryonoff it might be worth to remove app libs updates and keep only StringConvertible extension?

@larryonoff
Copy link
Contributor Author

@Skifcha sure. will do.

@larryonoff
Copy link
Contributor Author

@Skifcha done

Copy link
Contributor

@AKyashkin AKyashkin left a comment

Choose a reason for hiding this comment

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

This file is also should be added to the pod, right? Right now he's not even in complied sources.

Adapty/Classes/CustomStringConvertible.swift Outdated Show resolved Hide resolved
@AKyashkin
Copy link
Contributor

AKyashkin commented Oct 14, 2020

@larryonoff can you add this file to SDK? The rest looks fine.

@larryonoff
Copy link
Contributor Author

larryonoff commented Oct 14, 2020

@AKyashkin

@larryonoff can you add this file to SDK? The rest looks fine.

Sorry. I'm not sure I understand what you mean.

Adapty.podspec has pattern for source files in Classes folder, so that Package.swift. I don't see where should I add new file.

@AKyashkin
Copy link
Contributor

The Project itself doesn't see this file in case of the dev pod. No worries, I'll add this small thing later.

@AKyashkin AKyashkin merged commit 55ef5e1 into adaptyteam:develop Oct 14, 2020
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.

None yet

3 participants