Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix logo vectorization and update course view #147

Merged
merged 41 commits into from
Apr 23, 2024

Conversation

ballyalley-o
Copy link
Owner

This pull request updates the logo to a vector format, making it the official logo for the project. The logo file has been replaced with a vector version, and the necessary changes have been made in the code to use the new logo. This improves the scalability and quality of the logo.

…#142: endpoint configurations in bootcampSlice and courseSlice
…#142: Update global types to include Course and Bootcamp interfaces
src/config/asset-directory.ts Outdated Show resolved Hide resolved
@ballyalley-o ballyalley-o requested a review from in-a-iii April 23, 2024 08:41
@@ -20,8 +22,7 @@ const _svg = (svg: string) => {

const ASSET = {
// bg
_BG_TEMP: 'https://images.pexels.com/photos/9672971/pexels-photo-9672971.jpeg?auto=compress&cs=tinysrgb&w=1260&h=750&dpr=2',
default_badge: 'https://thecodecoachprojct.com/upload/badge/no-badge.png',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Good to see you removed the unused bg temp

@@ -4,6 +4,7 @@ const GLOBAL = {
APP_SERVER: import.meta.env.VITE_APP_SERVER,
APP_ORIGIN: import.meta.env.VITE_APP_ORIGIN,
APP_ENV: import.meta.env.VITE_APP_ENV,
DEFAULT_BADGE: import.meta.env.VITE_DEFAULT_BADGE,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Great, good to go for me,

@ballyalley-o ballyalley-o merged commit 6c24700 into development Apr 23, 2024
@ballyalley-o ballyalley-o linked an issue Apr 23, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request priority 1
Projects
Status: Done
2 participants