Skip to content

Prevent unwanted concatenation of "null" to feedback URL#111325

Merged
bpasero merged 1 commit intomicrosoft:masterfrom
abandonedaccount6235:patch-1
Nov 26, 2020
Merged

Prevent unwanted concatenation of "null" to feedback URL#111325
bpasero merged 1 commit intomicrosoft:masterfrom
abandonedaccount6235:patch-1

Conversation

@abandonedaccount6235
Copy link
Contributor

This PR fixes #111324 by replacing the null concatenation with an empty string instead so that the resulting query parameter for "sad" feedback is ref_src. To test:

  1. Open the tweet feedback UI using the button in the bottom right of code.
  2. Click the sad face
  3. Enter some tweet content
  4. Hit tweet
  5. The resulting URL should contain ref_src and NOT nullref_src

Copy link

@rheh rheh left a comment

Choose a reason for hiding this comment

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

LGMT

@joaomoreno joaomoreno assigned bpasero and unassigned joaomoreno Nov 26, 2020
@bpasero bpasero added this to the November 2020 milestone Nov 26, 2020
@bpasero bpasero merged commit 2d2a9e7 into microsoft:master Nov 26, 2020
@bpasero
Copy link
Member

bpasero commented Nov 26, 2020

Thanks!

@github-actions github-actions bot locked and limited conversation to collaborators Jan 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Using the feedback tweet UI to tweet "sad" feedback results in an incorrect query parameter

4 participants