https://github.com/sensehack/virtualtourist
Udacity 6th Project
https://github.com/sensehack/virtualtourist
Last synced: 2 months ago
JSON representation
Udacity 6th Project
- Host: GitHub
- URL: https://github.com/sensehack/virtualtourist
- Owner: SensehacK
- Created: 2016-12-08T12:30:15.000Z (over 8 years ago)
- Default Branch: master
- Last Pushed: 2018-02-09T05:14:08.000Z (over 7 years ago)
- Last Synced: 2025-01-22T19:38:35.603Z (4 months ago)
- Language: Swift
- Size: 676 KB
- Stars: 0
- Watchers: 2
- Forks: 0
- Open Issues: 0
-
Metadata Files:
- Readme: README.md
Awesome Lists containing this project
README
# Virtual Tourist
Working on "Virtual Tourist"
Started Working on it 8th November
First commit
8th November
Day 6:
1st Review :
https://review.udacity.com/#!/reviews/300465
Silly mistake of "Added "https:" instead of "http" while building URL. Pins displays" https://github.com/SensehacK/VirtualTourist/commit/95baa20a90174279a6056d87a020a46439c2c07b
Did some suggestions as well.
Day 7 :
2nd Review :
https://review.udacity.com/#!/reviews/300500
Suggestions :
Added more UI elements in Main thread queue.
DispatchQueue.main.async{
//UI Related Code
}
3rd Review :
https://review.udacity.com/#!/reviews/301051
Commit sync errors & led to older files reviewed for Project :
The issue why my images where not appearing. " imageLoadFromCoreOrFlickr(indexPath: indexPath, cell: cell)"
4th Review
https://review.udacity.com/#!/reviews/301240
Updated the App code with running simulatenously downloading images to the collection View. Alert button also works fast now. No blocking call method.
Day 8.
5th Review :
https://review.udacity.com/#!/reviews/301790
Small Change for Networking code to be in Networking file.
Practise will surely improve my skills.
SensehacK