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

Feature/store style #82

Merged
merged 3 commits into from
Mar 27, 2019
Merged

Feature/store style #82

merged 3 commits into from
Mar 27, 2019

Conversation

942v
Copy link
Contributor

@942v 942v commented Mar 17, 2019

Hi, awesome work @pmusolino !

I've added the alert content stack view and alert style as properties so those can be modified when customizing the UI.

Also, I've added validations to hide the title and description labels if the passed strings are nil.

Thanks,

@pmusolino
Copy link
Owner

Thanks for this PR @942v! Everything is ok, but I will remove the alertStyle variable, there is no reason to exist 👍

@pmusolino pmusolino merged commit b5fd299 into pmusolino:master Mar 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants