Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Comments

nextwavedev_22_4_changing_logo#5

Merged
bcko merged 2 commits intomainfrom
logo-change
May 1, 2025
Merged

nextwavedev_22_4_changing_logo#5
bcko merged 2 commits intomainfrom
logo-change

Conversation

@LiteWilliamNguyen
Copy link
Contributor

@LiteWilliamNguyen LiteWilliamNguyen commented Apr 29, 2025

Sunmmary & Changes 📃

Changin old logo to new logo
Changed description of image

Screenshots / Visual Aids 🔎

image

Checklist ✅

  • I have tested this PR locally and it works as expected.
  • This PR resolves an issue (Resolves #issue-number).
  • Reviewers, assignees(self), tags, and labels are correctly assigned.
  • Squash commits and enable auto-merge if approved.

Copy link
Contributor

@JesseCaddell JesseCaddell left a comment

Choose a reason for hiding this comment

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

image
I see the logo change, nice work!

Copy link

@bcko bcko left a comment

Choose a reason for hiding this comment

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

Can you delete the previous logo image on our repo?

@bcko
Copy link

bcko commented Apr 30, 2025

Let's wait one more day and merge this on Wednesday. @LiteWilliamNguyen

Copy link
Contributor

@jmathew12 jmathew12 left a comment

Choose a reason for hiding this comment

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

Logo has been updated and it works. Good job!

@LiteWilliamNguyen
Copy link
Contributor Author

Look like I have the change approved, but the merging is blocked for me as I am not authorized to pus this branch.

@bcko bcko merged commit 67da5c3 into main May 1, 2025
@bcko bcko deleted the logo-change branch May 1, 2025 01:06
@bcko bcko mentioned this pull request May 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants