Skip to content

Narjes development#1

Open
narjes-abs wants to merge 4 commits intofmousalou:masterfrom
narjes-abs:narjes_development
Open

Narjes development#1
narjes-abs wants to merge 4 commits intofmousalou:masterfrom
narjes-abs:narjes_development

Conversation

@narjes-abs
Copy link
Copy Markdown

@narjes-abs narjes-abs commented Feb 8, 2020

completed two items because had problem in documentation and time limit.

✅ Github Account Authorization Page
✅ Repositories Search Screen - consisting of a search bar and a list of results.

  • Commits Page - By selecting a repository, the list of commits of it must be shown
  • Profile Page - User can view their profile (Update profile has bonus point)

notice: know usage of ui test and unit test but never had chance to really work with it in real project, that's why I couldn’t implement much about it

.
added necessary files
part of RepositoryViewController
added test libraries
part of login added(client secter is missing)
@narjes-abs narjes-abs requested a review from fmousalou February 8, 2020 05:58
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.

1 participant