Skip to content
This repository has been archived by the owner on Jun 30, 2022. It is now read-only.

[Linked Account] fix bad request after link #3540

Merged
merged 1 commit into from
Jul 10, 2020

Conversation

KayMKM
Copy link
Contributor

@KayMKM KayMKM commented Jul 8, 2020

Purpose

What is the context of this pull request? Why is it being done?
fix: #3444

Changes

Are there any changes that need to be called out as significant or particularly difficult to grasp? (Include illustrative screenshots for context if applicable.)

  1. add user id in activity to get auth link.
  2. remove logic of generate new user id in home controller to avoid user id be changed after refresh the page.

Tests

Is this covered by existing tests or new ones? If no, why not?

Feature Plan

Are there any remaining steps or dependencies before this issue can be fully resolved? If so, describe and link to any relevant pull requests or issues.

Checklist

General

  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated the appropriate tests
  • I have updated related documentation

@sidmore sidmore merged commit c84a95d into next Jul 10, 2020
@delete-merged-branch delete-merged-branch bot deleted the yuesu/fix_linked_account_link_issue branch July 10, 2020 14:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants