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

bug fix for highlight text and comment #72

Merged
merged 1 commit into from
Aug 29, 2019

Conversation

fantastic-genius
Copy link
Contributor

Description

It will be important to ensure applications continuously conform with set style guide standards. Hence, the need for automated review of our app whenever there is a new change to check if changes conform with style guide configurations as the application scales.

Type of change

Please select the relevant option

  • Bug fix (non-breaking changes which fixes an issue)
  • New feature (non-breaking changes which adds functionality)
  • Breaking change (fix or feature that could cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Chore

How Has This Been Tested?

please describe the test that you ran to verify your changes

  • End to end
  • Integration

Checklist:

  • My code follows the style guidelines of this project
  • I have linted my code prior to submission
  • I have made corresponding changes to the documentation
  • My changes generates no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Pivotal Tracker

#168162905

@fantastic-genius fantastic-genius force-pushed the bug/168162905-highlight-tex-and-comment branch from 055658d to 7e51e12 Compare August 28, 2019 09:37
@vincentayorinde vincentayorinde temporarily deployed to demeter-ah-backend-stagi-pr-72 August 28, 2019 09:37 Inactive
- Refactor how highlighted text is saved to database

[Delivers #168162905]
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.

None yet

2 participants