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

Attempt to rebase content-card onto develop [DO NOT MERGE] #265

Conversation

BenHenning
Copy link
Member

@BenHenning BenHenning commented Oct 26, 2019

This is a manually rolled version of #245 that does not depend on merge-fix. Steps to create:

  1. Update introduce-image-parser-replica to latest develop (I created a separate branch to avoid changing the current branch)
  2. Use git diff merge-fix..content-card to create a patch file snapshot of Fix #161: Exploration player contentcard supports rich-text part [Blocked: #205] #245
  3. Create a new branch off develop and apply the patch using git apply --reject to apply only the valid parts of the patch
  4. Merge in the up-to-date image parser replica branch; resolve conflicts
  5. Manually resolve the rejections to make this PR look like Fix #161: Exploration player contentcard supports rich-text part [Blocked: #205] #245

@BenHenning BenHenning changed the base branch from develop to introduce-image-parser-replica October 26, 2019 00:08
…-card-from-develop

Conflicts:
	app/src/main/java/org/oppia/app/activity/ActivityComponent.kt
@BenHenning BenHenning changed the base branch from introduce-image-parser-replica to introduce-image-parser-replica-up-to-date October 26, 2019 00:10
@BenHenning
Copy link
Member Author

@veena14cs consider using this as a base branch to replace #245 with a develop-based set of changes since it seems like some additional work is needed to decouple the content card work from the merge-fix branch.

@BenHenning
Copy link
Member Author

Old base branch that was never meant to be merged.

@BenHenning BenHenning closed this Jun 23, 2020
@BenHenning BenHenning deleted the content-card-from-develop branch June 23, 2020 20:00
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.

1 participant