Skip to content

ITP-MAY-2026 | Data Flows | Alasdair MacDonald #142

Open

Description

@MacDonald91

On the issue, add:
Codewars = (https://codeyourfuture.github.io/Codewars-Progress-Checker/#MacDonald91

[sprint 2] [Book Library Project] = CodeYourFuture/Module-Data-Flows#486

[programmer humour task] = CodeYourFuture/Module-Data-Flows#499

A link to your GitHub repo and deployed Netlify site for the = https://github.com/MacDonald91/Project-TV-Show and Netlify = https://cyf-macdonald91-tv.netlify.app/
A link to your level 400 and level 500 pull requests for the “[TV Show Project]
level 400 = Jacknguyen4438/TV-Project-ITP-May-2026#2
level 500 = MacDonald91/Project-TV-Show#2

A link to your issue for “[LinkedIn Social Selling Index] = Your issue must show your current index and actions you will take to improve it. https://docs.google.com/document/d/1OxT8pYzVmRjXAKRo9p2kyBPHOBHB62OC_MU4dfjwDTE/edit?usp=sharing

Activity

  1. cjyuan commented on Apr 20, 2026

    @cjyuan

    - Codewars: "ITP: Data Flow - Sprint 2: katas not yet completed

    • Codewars: "ITP: Data Flow - Sprint 3: katas not yet completed

    • "Book Library Project" (PR not yet completed)

    • Missing a link to your level 500 PR (or a link to your main branch if it is your level-500 implementation)

    • Missing a link to your issue for “LinkedIn Social Selling Index”. The issue must show your current index and actions you will take to improve it.

    • TV Show App: the following level 500 requirement has not yet been met:

      • Add a navigation link to enable the user to return to the "shows listing"
        • When this is clicked, the episodes listing should be hidden
  2. changed the title [-]ITP-JAN-2026 | Data Flows | Alasdair MacDonald[/-] [+]ITP-MAY-2026 | Data Flows | Alasdair MacDonald[/+] on May 4, 2026
  3. cjyuan commented on May 16, 2026

    @cjyuan
    • TV Show Project

      • Missing a link to your level 400 PR
      • Missing a link to your level 500 PR (or a link to your main branch if it is your level-500 implementation)
      • The deployed app on Netlify is not a level-500 implementation.
    • Missing a link to your issue for “LinkedIn Social Selling Index”. The issue must show your current index and actions you will take to improve it.

  4. MacDonald91 commented on May 21, 2026

    @MacDonald91
    OwnerAuthor

    Hi, I've now completed all work above. Completed the TV show project by myself with the solution to level 300 from the repo as advised to finish the deadline today.

  5. illicitonion commented on May 21, 2026

    @illicitonion

    On your TV show project - the behaviour around filters when you change views isn't super clear.

    e.g. if I search shows for "american dad", then click on that show, then search episodes for "pilot", then press "Home", I still see the "american dad" TV show filter, but it shows all of the shows, but says it's only showing 2/398. Then if I delete the "t" at the end of "pilot" it starts showing me episodes rather than TV shows.

    Can you work out what's going on and clean that up?

    Otherwise in general your code looks good, but one small issue (doesn't block marking this as complete, but something to research and try to fix after) - you currently have an XSS vulnerability due to how you set innerHTML - have a read up on this and see if you can understand and fix the issue

    Everything else looks good to go :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions