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.

Sign up for GitHub

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

Jump to bottom

[Fix]Resolved sharing link #29

Merged
merged 2 commits into from
Feb 7, 2018
Merged

[Fix]Resolved sharing link #29

merged 2 commits into from
Feb 7, 2018

Conversation

lukyamuziB
Copy link
Contributor

@lukyamuziB lukyamuziB commented Feb 7, 2018 •
edited

PR Status

READY

Story ID

153727855

Added Migrations

NO

Description

Bug fix for sharing buttons

New Packages Added

None

Todos

  • Tested and working locally
  • Tested and working on development environment
  • Added more Unit tests (if appropriate)
  • Reviewed and approved by the scrum master
  • Reviewed and approved by a TTL
  • Deployment to team Heroku account

Impacted Areas in the application

Blog sharing

Known Issues (optional)

None

@coveralls
Copy link

coveralls commented Feb 7, 2018 •
edited

Coverage Status

Coverage increased (+0.4%) to 81.296% when pulling c1c0dbe on lukyamuziB:bg-153727855-blog_features into 894da23 on andela:develop.

@johnseremba johnseremba merged commit d8bd742 into andela:develop Feb 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@johnseremba johnseremba johnseremba approved these changes

Assignees

@LehruAngela LehruAngela

@lukyamuziB lukyamuziB

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@lukyamuziB @coveralls @johnseremba @LehruAngela