search

kevinlu1248 / fireship.io

Build and ship your app faster https://fireship.io
0 stars 0 forks source link

sweep: Can add an link or related courses about an particular technology when going through the Tech Stack of an particular course #13

Open kevinlu1248 opened 1 year ago

kevinlu1248 commented 1 year ago

Add the links to the footer and link to their homepages

sweep-ai[bot] commented 1 year ago

Here's the PR! https://github.com/kevinlu1248/fireship.io/pull/14.

💎 Sweep Pro: I used GPT-4 to create this ticket. You have 70 GPT-4 tickets left.

Step 1: 🔍 Code Search

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description. https://github.com/kevinlu1248/fireship.io/blob/f7206bd9d71ed96fa68aeff6d96d8c9dce1516ff/content/courses/supabase/_index.md#L15-L69 https://github.com/kevinlu1248/fireship.io/blob/f7206bd9d71ed96fa68aeff6d96d8c9dce1516ff/content/courses/sveltekit/basics-navigation.md#L1-L14 https://github.com/kevinlu1248/fireship.io/blob/f7206bd9d71ed96fa68aeff6d96d8c9dce1516ff/content/courses/sveltekit/project-link-list.md#L1-L70 https://github.com/kevinlu1248/fireship.io/blob/f7206bd9d71ed96fa68aeff6d96d8c9dce1516ff/content/courses/react/_index.md#L1-L69 https://github.com/kevinlu1248/fireship.io/blob/f7206bd9d71ed96fa68aeff6d96d8c9dce1516ff/app/util/helpers.ts#L1-L4

Step 2: 🧐 Snippet Analysis

From looking through the relevant snippets, I decided to make the following modifications:

File Path Proposed Changes
content/courses/supabase/_index.md In the Tech Stack section, add hyperlinks to the related courses for each technology listed. For example, for the 'supabase' technology, add a hyperlink to the Supabase course.
content/courses/sveltekit/_index.md Similarly, add hyperlinks to the related courses for each technology listed in the Tech Stack section.
content/courses/react/_index.md Again, add hyperlinks to the related courses for each technology listed in the Tech Stack section.
app/util/helpers.ts Depending on how the site is rendering the markdown content, we may need to update the logic in this file to correctly display the hyperlinks. This could involve changes to the getCourseIdFromURL function or other functions in this file.

Step 3: 📝 Planning

I have created a plan for writing the pull request. I am now working my plan and coding the required changes to address this issue. Here is the planned pull request:

Add hyperlinks to related courses in Tech Stack section sweep/add-hyperlinks-to-tech-stack

Description

This PR adds hyperlinks to related courses in the Tech Stack section of each course description. This allows users to easily navigate to other courses that use the same technologies.

Changes Made

  • Modified the markdown files for each course to include hyperlinks to related courses in the Tech Stack section.
  • Updated the rendering logic in the app/util/helpers.ts file to correctly display the hyperlinks.

Related Issue

Fixes #[issue_number]

Screenshots (if applicable)

N/A

Checklist

  • [ ] Tested the changes locally
  • [ ] Updated the documentation (if necessary)
  • [ ] Added unit tests (if applicable)
  • [ ] Verified the code follows the style guidelines
  • [ ] Verified all CI checks pass
  • [ ] Assigned reviewers

Step 4: ⌨️ Coding

I have finished coding the issue. I am now reviewing it for completeness.

Step 5: 🔁 Code Review

Success! 🚀

I'm a bot that handles simple bugs and feature requests but I might make mistakes. Please be kind! Join Our Discord