Skip to content

Conversation

@flacial
Copy link
Member

@flacial flacial commented Jan 23, 2023

Motivation

Adding the 3rd section, linked accounts to the settings page.

Changes

  • Create linked accounts section
  • Test the code to ensure it's working as expected
  • Add styles to make the list look nicer
  • Replace GetApp query with UserInfo as UserInfo has extra fields that we need

Testing

Preview link

  1. Login
  2. Go to /settings/account
  3. Scroll down to Linked accounts section
  4. Try to connect to Discord and ensure it's working
  5. Try to unlink from Discord and ensure it's working

Related issues

@vercel
Copy link

vercel bot commented Jan 23, 2023

@flacial is attempting to deploy a commit to the c0d3-prod Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Jan 23, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
c0d3-app ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 24, 2023 at 5:41PM (UTC)

@codecov
Copy link

codecov bot commented Jan 23, 2023

Codecov Report

Merging #2706 (a680f72) into master (d344e3a) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #2706   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          189       189           
  Lines         3537      3555   +18     
  Branches       978       984    +6     
=========================================
+ Hits          3537      3555   +18     
Impacted Files Coverage Δ
pages/settings/account/index.tsx 100.00% <100.00%> (ø)

@flacial flacial merged commit 2cad8d7 into garageScript:master Jan 24, 2023
@flacial flacial self-assigned this Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🦄 Done

Development

Successfully merging this pull request may close these issues.

2 participants