Skip to content

Setup feed tab screens#183

Merged
OnyekachiSamuel merged 14 commits intodevelopfrom
setup-feed-tab-screens
Jul 20, 2020
Merged

Setup feed tab screens#183
OnyekachiSamuel merged 14 commits intodevelopfrom
setup-feed-tab-screens

Conversation

@OnyekachiSamuel
Copy link
Copy Markdown
Collaborator

What this PR does:
Structures the Feed screen tabs

Checklist

  • Compiler warnings resolved
  • User-facing strings in Localizable.strings file

Which issue(s) this PR fixes:
Issue Link

Special notes for reviewers:
They are a couple of Todos and comments I added in the code, this is to show some direction on what needs to be done next in those places, they will all be removed when the tasks have been completed.

@todo
Copy link
Copy Markdown

todo bot commented Jul 17, 2020

The if statement will be removed later, it was added to visualize what we have currently

// TODO: The if statement will be removed later, it was added to visualize what we have currently
if indexPath.item == 0 {
cell.backgroundColor = .orange
} else if indexPath.item == 1 {
cell.backgroundColor = .blue
} else if indexPath.item == 2 {


This comment was generated by todo based on a TODO comment in d362bdb in #183. cc @FightPandemics.

@todo
Copy link
Copy Markdown

todo bot commented Jul 17, 2020

The network call will happen here and the fetched post data will be appended to the snapshot

// TODO: The network call will happen here and the fetched post data will be appended to the snapshot
let posts: [Post] = [.init(id: 1), .init(id: 2)]
var snapShot = NSDiffableDataSourceSnapshot<Section, Post>()
snapShot.appendSections([.main])
snapShot.appendItems(posts)
dataSource.apply(snapShot)


This comment was generated by todo based on a TODO comment in d362bdb in #183. cc @FightPandemics.

@todo
Copy link
Copy Markdown

todo bot commented Jul 17, 2020

This will be handled with view model

// TODO: This will be handled with view model
var post: Post?
override func setupViews() {}
}


This comment was generated by todo based on a TODO comment in d362bdb in #183. cc @FightPandemics.

@todo
Copy link
Copy Markdown

todo bot commented Jul 17, 2020

The network call will happen here and the fetched post data will be appended to the snapshot

// TODO: The network call will happen here and the fetched post data will be appended to the snapshot
let posts: [Post] = [.init(id: 1), .init(id: 2), .init(id: 3)]
var snapShot = NSDiffableDataSourceSnapshot<Section, Post>()
snapShot.appendSections([.main])
snapShot.appendItems(posts)
dataSource.apply(snapShot)


This comment was generated by todo based on a TODO comment in d362bdb in #183. cc @FightPandemics.

@todo
Copy link
Copy Markdown

todo bot commented Jul 17, 2020

The network call will happen here and the fetched post data will be appended to the snapshot

// TODO: The network call will happen here and the fetched post data will be appended to the snapshot
let posts: [Post] = [.init(id: 1)]
var snapShot = NSDiffableDataSourceSnapshot<Section, Post>()
snapShot.appendSections([.main])
snapShot.appendItems(posts)
dataSource.apply(snapShot)


This comment was generated by todo based on a TODO comment in d362bdb in #183. cc @FightPandemics.

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.

2 participants