Skip to content

Cryptex#9

Open
JFerrer95 wants to merge 7 commits intobloominstituteoftechnology:masterfrom
JFerrer95:master
Open

Cryptex#9
JFerrer95 wants to merge 7 commits intobloominstituteoftechnology:masterfrom
JFerrer95:master

Conversation

@JFerrer95
Copy link
Copy Markdown

No description provided.

@JFerrer95
Copy link
Copy Markdown
Author

@nelglez

@nelglez
Copy link
Copy Markdown

nelglez commented May 2, 2019

Hey Jonathan, your code looks fine, I did make a few changes to make it work. In line 13 you have @objc var countdownTimer: Timer? you don't really need @objc so I removed it var countdownTimer: Timer?

In viewDidLoad() you need to call newCryptexAndReset() and in your func reset() { you need to call updateViews()

After I made those changes the app works as expected.

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