Skip to content

Newberts_CodeReview_1#7

Open
Newbtree45 wants to merge 1 commit intomasterfrom
Newberts_CodeReview_1
Open

Newberts_CodeReview_1#7
Newbtree45 wants to merge 1 commit intomasterfrom
Newberts_CodeReview_1

Conversation

@Newbtree45
Copy link
Copy Markdown
Collaborator

I saw that you guys were adding some retrofit to the gradle. So I added a retrofit object that you could use and the beginnings of a dao interface for movie to put any movies into a list.

I saw that you guys were adding some retrofit to the gradle. So I added a retrofit object that you could use and the beginnings of a dao interface for movie to put any movies into a list.
@Newbtree45
Copy link
Copy Markdown
Collaborator Author

It seems that as I was making these changes you guys did it yourselves. I'll make some other changes in another branch.

@Newbtree45
Copy link
Copy Markdown
Collaborator Author

The original master ran before the retrofit changes but seems to have trouble running now, but that might be on my end. I like how you have your packages structured as it make it easier to find things. The code was easy to understand and had a good amount of comments.

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