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

done #6

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

done #6

wants to merge 5 commits into from

Conversation

TemnosHD
Copy link

wollte die app eigentlich noch um die möglichkeit der auswahl eines kanye west songs erweitern (deshalb kanyeREST), so bleibt er leider erstmal bei einem einzigen song.

@nilsvu
Copy link
Member

nilsvu commented Jun 5, 2016

Witzige API, Abfrage funktioniert 👍 2/2 Pt. Für mehr Funktionalität gibt's Extrapunkte.

@TemnosHD
Copy link
Author

tests hinzugefügt, leider funktioniert der test für die textview nicht

@nilsvu
Copy link
Member

nilsvu commented Jun 19, 2016

API-Abfrage, Datenweitergabe zwischen View Controllern und Unit Tests 👏

5/2 Pt. und Kurs bestanden 💯🎉

Die Tests laufen nicht alle durch, offenbar passt das Format der sampleData nicht ganz, und das Lyrics Label wird nicht gefunden. Probleme mit nicht gefunden Accessibility Labeln hatten auch schon andere, es half die Views des View Controllers im Storyboard einmal zu löschen und neu einzufügen.

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