New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Solved issue #13617 #13619

Merged
merged 7 commits into from Sep 4, 2017

Conversation

Projects
None yet
2 participants
@akritianand
Contributor

akritianand commented Aug 27, 2017

Unnecessary data should not come in clipboard copied table - SOLVED

Signed-off-by: Akriti Anand anand.akriti@gmail.com

akritianand added some commits Aug 27, 2017

Solved issue number #13617
Unnecessary data should not come in clipboard copied table - SOLVED
@nijel

This comment has been minimized.

Show comment
Hide comment
@nijel

nijel Aug 28, 2017

Member

But this will remove the "Showing rows..." part, which probably makes sense to include...

Member

nijel commented Aug 28, 2017

But this will remove the "Showing rows..." part, which probably makes sense to include...

@akritianand

This comment has been minimized.

Show comment
Hide comment
@akritianand

akritianand Aug 31, 2017

Contributor

Okay, I can rectify that.

Contributor

akritianand commented Aug 31, 2017

Okay, I can rectify that.

@codecov

This comment has been minimized.

Show comment
Hide comment
@codecov

codecov bot Aug 31, 2017

Codecov Report

❗️ No coverage uploaded for pull request base (master@adfe2ac). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #13619   +/-   ##
=========================================
  Coverage          ?   53.19%           
=========================================
  Files             ?      468           
  Lines             ?    81658           
  Branches          ?        0           
=========================================
  Hits              ?    43439           
  Misses            ?    38219           
  Partials          ?        0

codecov bot commented Aug 31, 2017

Codecov Report

❗️ No coverage uploaded for pull request base (master@adfe2ac). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #13619   +/-   ##
=========================================
  Coverage          ?   53.19%           
=========================================
  Files             ?      468           
  Lines             ?    81658           
  Branches          ?        0           
=========================================
  Hits              ?    43439           
  Misses            ?    38219           
  Partials          ?        0

akritianand added some commits Aug 31, 2017

@nijel nijel self-assigned this Sep 4, 2017

@nijel nijel added this to the 4.8.0 milestone Sep 4, 2017

@nijel nijel merged commit 4f6335b into phpmyadmin:master Sep 4, 2017

2 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
Scrutinizer Analysis: 1 updated code elements – Tests: passed
Details
codacy/pr Good work! A positive pull request.
Details
@nijel

This comment has been minimized.

Show comment
Hide comment
@nijel

nijel Sep 4, 2017

Member

Merged, thanks for your contribution!

Member

nijel commented Sep 4, 2017

Merged, thanks for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment