Skip to content

1187 task investigate all site warnings in the console tab#1320

Open
wlh3903 wants to merge 19 commits intodevfrom
1187-task-investigate-all-site-warnings-in-the-console-tab
Open

1187 task investigate all site warnings in the console tab#1320
wlh3903 wants to merge 19 commits intodevfrom
1187-task-investigate-all-site-warnings-in-the-console-tab

Conversation

@wlh3903
Copy link

@wlh3903 wlh3903 commented Feb 24, 2026

  • Cleaned up all console.logs
  • Fixed most console error messages (ones involving redux reworks left i.e., Lab 2 exercise)
  • No discrepancies across browsers (ex: chrome vs safari)
  • Accessibility functions
  • Pages can scale without distorting page
  • No dead links
  • Navbar is consistent across the site
  • Pages are screen-reader accessible
  • Contrast meets standards for accessibility
  • Pages are keyboard accessible
  • Code is cleaned up and bug-free (ex: debug statements removed)

@wlh3903 wlh3903 requested review from JackyFTW and ejt7508 February 24, 2026 16:58
@wlh3903 wlh3903 self-assigned this Feb 24, 2026
@wlh3903 wlh3903 linked an issue Feb 24, 2026 that may be closed by this pull request
3 tasks
Copy link
Contributor

@shmowen shmowen left a comment

Choose a reason for hiding this comment

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

Couple things:

  1. Got some merge conflicts that need fixing
  2. Opening the labs to the About section shows off this warning in console:
Image

Not totally sure if this is within the scope of this story, might be a CRA error.

Copy link
Contributor

@JackyFTW JackyFTW left a comment

Choose a reason for hiding this comment

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

Still seems like there's a good amount of site warnings! Make sure to investigate each lab's exercises. I have provided a handful below to get started (Labs 1-5)

Copy link
Contributor

Choose a reason for hiding this comment

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

Clicking start on the Lab 1 exercise I get the following warning still:

Image

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, when running out of hints in the Lab 1 exercise, I get the following warning:

Image

Copy link
Contributor

Choose a reason for hiding this comment

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

I was able to receive this warning in Lab 2's exercise after updating the colors and trying the simulation over:

Image

Copy link
Contributor

Choose a reason for hiding this comment

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

I also get this warning when opening the Lab 3 repair up:

Image

Copy link
Contributor

Choose a reason for hiding this comment

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

Lab 3: navigating from UserUpdatedExercise to continue throws this warning

Image

Copy link
Contributor

Choose a reason for hiding this comment

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

Lab 4: Loading FormSkipToMainBroken page has quite a few warnings appear

Image

Copy link
Contributor

Choose a reason for hiding this comment

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

Lab 5: warning when loading the repair page

Image

Copy link
Contributor

Choose a reason for hiding this comment

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

Lab 5: form guidance has a warning on the FormGuidance page

Image

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.

Task: Investigate all site warnings in the console tab

3 participants