Skip to content

Old Code#70

Open
ricealonzo wants to merge 5 commits intomainfrom
55-add-alonzo-rice-favorite-song
Open

Old Code#70
ricealonzo wants to merge 5 commits intomainfrom
55-add-alonzo-rice-favorite-song

Conversation

@ricealonzo
Copy link
Contributor

No description provided.

@ricealonzo ricealonzo linked an issue Dec 11, 2024 that may be closed by this pull request
@ricealonzo ricealonzo changed the title error Old Code Dec 11, 2024
Copy link
Contributor

@m-nt-s-no m-nt-s-no left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

artist's first name is spelled Nipsey. Otherwise good.

Copy link
Contributor

@borvux borvux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also make sure to do git pull origin main to pull all the changes that have done, seems like your branch is super behind. You will have to go through a merge conflict.

@borvux borvux requested review from amandaag39 and borvux and removed request for aldoportillo, dsimp, m-nt-s-no and teresa-c-pham June 25, 2025 16:44
Copy link
Contributor

@borvux borvux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure to pass the Prettier Format Check action

@ricealonzo ricealonzo requested a review from borvux June 25, 2025 17:35
Copy link
Contributor

@borvux borvux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor fix, make sure to pass the format checker again

<p class="mb-1">Nipsey Hussle</p>
<small class="text-body-secondary">Submitted by Alonzo Rice</small>

href="https://www.youtube.com/watch?v=VMk6i7Q0k54"
Copy link
Contributor

@borvux borvux Jun 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing <a> tag here, causing this:
Screenshot 2025-06-26 at 12 59 18 PM

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey when i pull up the live site everything seems to be working on my end i do not get the same syntax error . can you rerun the live site?

@ricealonzo ricealonzo requested review from borvux and m-nt-s-no June 29, 2025 14:35
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.

Add Alonzo's favorite song

3 participants