Skip to content
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

Added Linkedin as a sharing option #1508

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

Dishebh
Copy link
Member

@Dishebh Dishebh commented Jan 13, 2020

Fixes #1491

Checklist

  • I have read the Contribution & Best practices Guide and my PR follows them.
  • My branch is up-to-date with the Upstream master branch.
  • I have added necessary documentation (if appropriate)
  • Added Surge preview link

Changes proposed in this pull request:

  • Added linkedin as a sharing option in infobox

Screenshot from 2020-01-13 11-49-53

@codecov
Copy link

codecov bot commented Jan 13, 2020

Codecov Report

Merging #1508 into development will increase coverage by 0.13%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##           development    #1508      +/-   ##
===============================================
+ Coverage        63.12%   63.26%   +0.13%     
===============================================
  Files               51       51              
  Lines             1451     1451              
  Branches           181      181              
===============================================
+ Hits               916      918       +2     
+ Misses             425      424       -1     
+ Partials           110      109       -1
Impacted Files Coverage Δ
src/app/intelligence/intelligence.component.ts 76% <0%> (ø) ⬆️
src/app/speechtotext/speechtotext.component.ts 54.28% <0%> (+2.85%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5f71e17...35ee4fe. Read the comment docs.

@Dishebh
Copy link
Member Author

Dishebh commented Jan 13, 2020

@praveenojha33 @samagragupta can you pls review?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add LinkedIn as sharing option for infobox component
1 participant