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

Added a PlaintextPrettyPrinter and manual selection #151

Merged
merged 5 commits into from
Mar 3, 2019

Conversation

brototyp
Copy link
Member

@brototyp brototyp commented Feb 19, 2019

This PR implements the first steps towards #91. For now I skipped automatic selection of the "correct" pretty printer. It will just display a UIAlertController and lets the user select the pretty printer.

Screenshots

bildschirmfoto 2019-02-19 um 08 29 11
bildschirmfoto 2019-02-19 um 08 31 08
bildschirmfoto 2019-02-19 um 08 30 16

Copy link
Member

@pstued pstued left a comment

Choose a reason for hiding this comment

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

Looks good. Feel free to ping me whenever it's not WIP anymore

Cauli/Response.swift Outdated Show resolved Hide resolved
@brototyp brototyp force-pushed the feature/91-pretty-print-in-inspector branch from 3b2f131 to 9d54b98 Compare February 27, 2019 19:51
@brototyp brototyp force-pushed the feature/91-pretty-print-in-inspector branch from 9d54b98 to 2d7ebb0 Compare March 3, 2019 14:20
@brototyp brototyp changed the title WIP: Added a PlaintextPrettyPrinter and manual selection Added a PlaintextPrettyPrinter and manual selection Mar 3, 2019
@brototyp-bot
Copy link

brototyp-bot commented Mar 3, 2019

1 Warning
⚠️ Are there any changes that should be explained in the README.md?

Generated by 🚫 Danger

@brototyp
Copy link
Member Author

brototyp commented Mar 3, 2019

@pstued This PR isn't WIP anymore and can finally be reviewed 😊

Copy link
Member

@pstued pstued left a comment

Choose a reason for hiding this comment

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

I fixed a crash and improved the usability a little 🙃. Please check my changes and feel free to merge.

@pstued pstued force-pushed the feature/91-pretty-print-in-inspector branch from 619ee06 to 78ffcd3 Compare March 3, 2019 15:10
# Conflicts:
#	CHANGELOG.md
#	Cauliframework.xcodeproj/project.pbxproj
#	Example/cauli-ios-example/Pods/Pods.xcodeproj/project.pbxproj
@brototyp brototyp force-pushed the feature/91-pretty-print-in-inspector branch from 88472ca to c6f29ad Compare March 3, 2019 15:19
@brototyp brototyp merged commit 327e18f into develop Mar 3, 2019
@brototyp brototyp deleted the feature/91-pretty-print-in-inspector branch March 3, 2019 15:34
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