Skip to content
This repository has been archived by the owner on May 5, 2020. It is now read-only.

Finished merging VCs, solved all complier warnings #74

Merged
merged 2 commits into from
Jun 9, 2017
Merged

Finished merging VCs, solved all complier warnings #74

merged 2 commits into from
Jun 9, 2017

Conversation

identity2
Copy link
Contributor

@identity2 identity2 commented Jun 2, 2017 •
edited

Major Changes

  • Finished merging all VCs.
  • Stored the customizable parts of an avatar in UserDefaults, so the configuration could persist between game sessions and VCs. (Should be changed after Firebase is integrated)
  • Merge the multiple shops into a single ShopViewController
  • Fixed all compiler warnings.
  • Added a Customizables struct to store all the image names of customizable parts.

Minor Changes

  • Customized appearance will now correctly apply to avatars in ScenarioViewController and result VC will now.
  • Added a group in the project file structure to categorize files.
  • Added ResultViewController class.

(Issue #40 #71 #73 are fixed now)

@sunjunkie sunjunkie self-requested a review June 8, 2017 23:36
@sunjunkie sunjunkie merged commit 0ec6163 into anitab-org:develop Jun 9, 2017
This was referenced Jun 9, 2017
Closed
Closed
@identity2 identity2 deleted the combine-multiple-VCs branch June 13, 2017 05:07
@identity2 identity2 restored the combine-multiple-VCs branch June 13, 2017 05:07
@identity2 identity2 deleted the combine-multiple-VCs branch June 20, 2017 02:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@sunjunkie sunjunkie Awaiting requested review from sunjunkie

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@identity2 @sunjunkie