Skip to content

Cryptex#3

Open
mitchellgbudge wants to merge 8 commits intobloominstituteoftechnology:masterfrom
mitchellgbudge:master
Open

Cryptex#3
mitchellgbudge wants to merge 8 commits intobloominstituteoftechnology:masterfrom
mitchellgbudge:master

Conversation

@mitchellgbudge
Copy link
Copy Markdown

@Cdunn2013

…a Cryptex.swift file to configure the model object.
Added a number of functions per the instructions
Fixed a bug to take care of errors within CryptexViewController.swift
Cryptex functions for initial first two puzzles. Need to try and configure more to appear. UIAlert needs some cleaning up
Cleaned up some code in CryptexVC to be more readable.
@cdunn95
Copy link
Copy Markdown

cdunn95 commented May 8, 2019

Mitchell,
App works as expected and is in great condition as far as MVC goes. Great job!

Need to go back and add additional puzzles.
@mitchellgbudge
Copy link
Copy Markdown
Author

@nelglez

@nelglez
Copy link
Copy Markdown

nelglez commented May 9, 2019

Good work, app seems to be working correctly and no warnings in the console.

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