indifferentghost

indifferentghost

Member Since 6 years ago

Phoenix, AZ

Experience Points
72
follower
Lessons Completed
97
follow
Lessons Completed
40
stars
Best Reply Awards
150
repos

850 contributions in the last year

Pinned
⚡ A curated portfolio of projects.
⚡ A simple webserver written in C
⚡ Conway's Game of Life in React/canvas
⚡ A dockerized, webpack'd insta-clone
⚡ A test to closely resemble a mockup
Activity
Dec
1
2 days ago
open pull request

indifferentghost wants to merge 100devs/course-tracker

indifferentghost
indifferentghost

add logo

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

Adds logo per issue #113

Questions and concerns

Once navbar #116 is merged we can get this into the design.

indifferentghost
indifferentghost
open pull request

indifferentghost wants to merge 100devs/course-tracker

indifferentghost
indifferentghost

add logo

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

Adds logo per issue #113

Questions and concerns

Once navbar #116 is merged we can get this into the design.

indifferentghost
indifferentghost

This should probably be moved to assets. And maybe cleaned up a bit:

https://fvsch.com/svg-icons

pull request

indifferentghost merge to 100devs/course-tracker

indifferentghost
indifferentghost

add logo

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

Adds logo per issue #113

Questions and concerns

Once navbar #116 is merged we can get this into the design.

pull request

indifferentghost merge to 100devs/course-tracker

indifferentghost
indifferentghost

add logo

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

Adds logo per issue #113

Questions and concerns

Once navbar #116 is merged we can get this into the design.

Nov
25
1 week ago
pull request

indifferentghost merge to 100devs/course-tracker

indifferentghost
indifferentghost

Styled theme

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

Adds ThemeProvider context from styled-components to handle color palette

Questions and concerns

Nov
24
1 week ago
pull request

indifferentghost merge to 100devs/course-tracker

indifferentghost
indifferentghost

Fixing-merge-mistake: deleted extra footers

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

Deleted the extra footers in Feed and CreatePost. Having not realized they were abstracted to the Layout component I merged my versions of the Feed and CreatePost into main.

I also then abstracted the is admin prop in the Navbar and added the functionality that closes the navbar once the Home, Create a post or Log out links are clicked.

I also added a dependency on the useEffect of the Feed so that it re-renders on log-out.

Questions and concerns

pull request

indifferentghost merge to 100devs/course-tracker

indifferentghost
indifferentghost

Task-99 Remove window reload from logout

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

Removed the need for window reload on logout

Questions and concerns

Nov
23
1 week ago
pull request

indifferentghost merge to 100devs/course-tracker

indifferentghost
indifferentghost

add logo

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

Adds logo per issue #113

Questions and concerns

Once navbar #116 is merged we can get this into the design.

pull request

indifferentghost merge to 100devs/course-tracker

indifferentghost
indifferentghost

Task 121 - Lift Up Common Components

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

related to issue #121

Questions and concerns

pull request

indifferentghost merge to 100devs/course-tracker

indifferentghost
indifferentghost

Style update

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • UI/UX

Description

Updated font sizes, padding and media queries for a cleaner look

Questions and concerns

open pull request

indifferentghost wants to merge 100devs/course-tracker

indifferentghost
indifferentghost

Task-99 Remove window reload from logout

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

Removed the need for window reload on logout

Questions and concerns

indifferentghost
indifferentghost
  const { user, isAdmin, getAdminStatus } = useContext(AuthContext);
pull request

indifferentghost merge to 100devs/course-tracker

indifferentghost
indifferentghost

Task-99 Remove window reload from logout

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

Removed the need for window reload on logout

Questions and concerns

Nov
21
1 week ago
push

indifferentghost push 100devs/course-tracker

indifferentghost
indifferentghost
indifferentghost
indifferentghost

Update .github/workflows/staging.yml

Remove incorrect comments

indifferentghost
indifferentghost

Update .github/workflows/staging.yml

indifferentghost
indifferentghost

Merge pull request #124 from 100devs/staging

Create staging.yml

commit sha: 5e2e06cc82ebac926205825c3773d91801fd61af

push time in 1 week ago
Activity icon
delete
deleted time in 1 week ago
push

indifferentghost push 100devs/course-tracker

indifferentghost
indifferentghost

Update .github/workflows/staging.yml

commit sha: e265b5bb94e0dd3bab2d46f20424cbbddd0db9a3

push time in 1 week ago
push

indifferentghost push 100devs/course-tracker

indifferentghost
indifferentghost

Update .github/workflows/staging.yml

Remove incorrect comments

commit sha: 8f16b87893bd7421d9040a34d36e990055754d10

push time in 1 week ago
Activity icon
created branch
createdAt 1 week ago
Nov
17
2 weeks ago
open pull request

indifferentghost wants to merge 100devs/course-tracker

indifferentghost
indifferentghost

Task-99 Remove window reload from logout

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

Removed the need for window reload on logout

Questions and concerns

indifferentghost
indifferentghost

This can just be

          <TextLink text="Log out" link="/" onClick={logout}/>
pull request

indifferentghost merge to 100devs/course-tracker

indifferentghost
indifferentghost

Task-99 Remove window reload from logout

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

Removed the need for window reload on logout

Questions and concerns

Nov
16
2 weeks ago
pull request

indifferentghost merge to 100devs/course-tracker

indifferentghost
indifferentghost

GitHub actions

Will the staging environment need to be set up once we have heroku? Everything up to that point is ready for review.

Issue #10 setting up Github Actions

open pull request

indifferentghost wants to merge 100devs/course-tracker

indifferentghost
indifferentghost

GitHub actions

Will the staging environment need to be set up once we have heroku? Everything up to that point is ready for review.

Issue #10 setting up Github Actions

indifferentghost
indifferentghost

I still think we should remove the yarn lock here again, but I'm not gonna fight it.

pull request

indifferentghost merge to 100devs/course-tracker

indifferentghost
indifferentghost

GitHub actions

Will the staging environment need to be set up once we have heroku? Everything up to that point is ready for review.

Issue #10 setting up Github Actions

Previous