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

Danilo silveira #19

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Danilo silveira #19

wants to merge 10 commits into from

Conversation

danilodns
Copy link

Coding Challenge

Minimum requirement:

  • Fork the repo
  • Parse the json file included in the project
  • Display the tweets in the order the json file defines them. The app should display the author, the tweet and the date it was tweeted at. Tweets are variable length, so the cells must be properly sized to the content
  • When done, send a pull request to this project

Bonuses

  • Highlight the mentions (@username) and/or links in a different font/color
  • Display a tweet's thread when tapping on a giving tweet. Due to the very simplistic data model made available to you, it's probably best to simplify this: if the user taps on the first tweet of a thread, display all the replies in ascending chronological order, if the user taps on a reply to another tweet, only show the tapped tweet and the tweet it's replying to.
    • In order to see the highlight the selected tweet, the first click on the tweet it will highlight it, the second click it will open a thread
  • Display avatar images (feel free to use AFNetworking/AlamoFire or just use NSURLSession for that)
    • I used SDWebImage to download and cache the image, It is very close to NSURLCache but it returns UIImage instead of Data
  • Animate/highlight a tweet when it is selected (e.g. make it "bigger", in an animated fashion)

Simulator Screenshot - iPhone 15 Pro - 2024-07-24 at 20 04 25
Simulator Screenshot - iPhone 15 Pro - 2024-07-24 at 20 04 18
Simulator Screenshot - iPhone 15 Pro - 2024-07-24 at 20 04 10

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