Skip to content
This repository was archived by the owner on Oct 21, 2024. It is now read-only.

Update challenger on new challenger link #183

Open
mnarayan1 wants to merge 5 commits into
base: development
Choose a base branch
from
Open

Update challenger on new challenger link #183

mnarayan1 wants to merge 5 commits into from

Conversation

@mnarayan1
Copy link
Contributor

@mnarayan1 mnarayan1 commented Sep 5, 2022

Overview

  • When a logged-in player clicks on a new challenger's link, their localhost and firebase should be updated with the new challenger.

Follow ups

While working on this PR, I found this bug. This might be due to errors on my local build.

  • Some areas of the app seem to require a log-in, causing errors if the user is not logged in already. I think this error originated in the sidebar.
    image

@mnarayan1 mnarayan1 requested a review from Devanshusp September 5, 2022 17:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@Devanshusp Devanshusp Awaiting requested review from Devanshusp

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mnarayan1