search

mghebre20 / InstaParse

0 stars 0 forks source link

Project Feedback! #1

Open codepathreview opened 4 years ago

codepathreview commented 4 years ago

It looks like your video walkthrough doesn't seem to show some of the required stories as follows:

  1. User sees app icon in home screen.
  2. User can sign up to create a new account using Parse authentication
  3. The current signed in user is persisted across app restarts
  4. User can pull to refresh the last 20 posts submitted to "Instagram"
  5. User can tap a post to view post details, including timestamp and caption.

Can you update this and resubmit?

codepathreview commented 4 years ago

You can resubmit your video walkthrough with the stretch features until Monday 11:59 pm, Pacific Time. /cc @codepathreview

mghebre20 commented 4 years ago

Hey, I've updated my readme with the walkthrough video but it seems the video is not showing on my end? It seems my file might be too big. Is there a url converter website I can use for my gif for resizing?

mghebre20 commented 4 years ago

Hi, I've managed to resolve the issue! Turns out I needed to create two gifs since the file was too large.

On Mon, Jul 13, 2020 at 6:42 AM CodePath Reviewers notifications@github.com wrote:

You can resubmit your video walkthrough with the stretch features until Monday 11:59 pm, Pacific Time. /cc @codepathreview https://github.com/codepathreview

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/mghebre20/InstaParse/issues/1#issuecomment-657483391, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQBIMODHPDS62U4G36NWUADR3LQK5ANCNFSM4OXF5P4Q .

codepathreview commented 4 years ago

Thank you for letting us know!

We have revisited your submission and it seems like the GIF links in the repository encountered an error "non-image content-type returned". We have managed to review a working GIF and the following required stories are still missing:

  1. The current signed in user is persisted across app restarts (to show this, the current login user's feeds should appear once you close the app and open it again)
  2. User can pull to refresh the last 20 posts submitted to "Instagram" (doesn't need to have 20 posts but just shows a refresh in the feeds)
  3. User can tap a post to view post details, including timestamp and caption (in the detail view the caption appeared however there is no timestamp)

Can you update this and resubmit? You can add another GIF showing these lacking required stories. /cc @codepathreview

mghebre20 commented 4 years ago

Hi, I included these in the second video all towards the end. You will see these user stories after I demonstrated scrolling down to 20 posts part. It starts with a pull to refresh and a lion post pops up!

On Wed, Jul 15, 2020 at 9:43 AM CodePath Reviewers notifications@github.com wrote:

Thank you for letting us know!

We have revisited your submission and it seems like the GIF links in the repository encountered an error "non-image content-type returned". We have managed to review a working GIF and the following required stories are still missing:

  1. The current signed in user is persisted across app restarts (to show this, the current login user's feeds should appear once you close the app and open it again)
  2. User can pull to refresh the last 20 posts submitted to "Instagram" (doesn't need to have 20 posts but just shows a refresh in the feeds)
  3. User can tap a post to view post details, including timestamp and caption (in the detail view the caption appeared however there is no timestamp)

Can you update this and resubmit? You can add another GIF showing these lacking required stories. /cc @codepathreview https://github.com/codepathreview

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/mghebre20/InstaParse/issues/1#issuecomment-658775912, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQBIMOGIWVWAQTUIKQ4YME3R3WXATANCNFSM4OXF5P4Q .

mghebre20 commented 4 years ago

Hey there, I’m just following to see if you were able to see these user stories. The video stops for a short period at the end and resumes demonstrating the rest of the user stories?

On Wed, Jul 15, 2020 at 11:26 AM Million Ghebre millionghebre1@gmail.com wrote:

Hi, I included these in the second video all towards the end. You will see these user stories after I demonstrated scrolling down to 20 posts part. It starts with a pull to refresh and a lion post pops up!

On Wed, Jul 15, 2020 at 9:43 AM CodePath Reviewers < notifications@github.com> wrote:

Thank you for letting us know!

We have revisited your submission and it seems like the GIF links in the repository encountered an error "non-image content-type returned". We have managed to review a working GIF and the following required stories are still missing:

  1. The current signed in user is persisted across app restarts (to show this, the current login user's feeds should appear once you close the app and open it again)
  2. User can pull to refresh the last 20 posts submitted to "Instagram" (doesn't need to have 20 posts but just shows a refresh in the feeds)
  3. User can tap a post to view post details, including timestamp and caption (in the detail view the caption appeared however there is no timestamp)

Can you update this and resubmit? You can add another GIF showing these lacking required stories. /cc @codepathreview https://github.com/codepathreview

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/mghebre20/InstaParse/issues/1#issuecomment-658775912, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQBIMOGIWVWAQTUIKQ4YME3R3WXATANCNFSM4OXF5P4Q .