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

Any plans to update to upstream 3.4 #1

Open
gihrig opened this issue Jan 20, 2017 · 12 comments
Open

Any plans to update to upstream 3.4 #1

gihrig opened this issue Jan 20, 2017 · 12 comments

Comments

@gihrig
Copy link
Collaborator

gihrig commented Jan 20, 2017

Question:

I thought about giving it a try, but if you have it in your near term plans, I don't need to duplicate that work.

@jeffbski
Copy link
Owner

@gihrig That would be excellent. I have a few other things on my plate right now, so I would appreciate whatever help you can provide.

@gihrig
Copy link
Collaborator Author

gihrig commented Jan 27, 2017

@jeffbski The 3.4 conversion is mostly done 🎉

I'm not sure how to get it to your repo, since it's not based on a fork of react-boilerplate-logic. You can find it in the dev branch.

Current status:

  • Mostly copy/pasted from your 3.3.3 version
  • Example app works and all tests pass (on OS X w/ Chrome - not tested on Windows)
  • I struggled with app/utils/test/asyncInjectors.test.js and app/containers/HomePage/tests/logic.test.js, I got them working, but you should take a look and make corrections as you see fit.
  • I converted the docs, with the exception of the 'sagas' section of docs/general/introduction.md. Please have a look at this, just search for "saga" and edit as appropriate.

@jeffbski
Copy link
Owner

@gihrig That is fantastic news!

I'll get it copied over into a branch and take a look at what you have suggested.

Thanks for all your hard work on this!

@gihrig
Copy link
Collaborator Author

gihrig commented Jan 29, 2017

I see you have opened a 3.4 branch

I just pushed some changes I missed earlier, please refresh

Oops! I was mistaken. That's the upstream 3.4 branch. Too bad it's not possible to fork a fork due to name conflicts.

In any case I'm expecting to keep my rbp-340-logic project up to date with the upstream dev branch changes, and possibly some others. Is that of interest to you here?

@jeffbski
Copy link
Owner

@gihrig yeah, that 3.4 branch was just there from the original fork. That's great that you are planning to keep those branches up to date, it really help to have someone familiar with everything to do that.

I'm certainly open to your ideas on what will be the best way to maintain going forward.

I think at some point we will need to detach react-boilerplate-logic from react-boilerplate in github since forks aren't searchable unless they have more stars than the original. So as it is now I don't believe we are being search indexed either.

I don't think that prevents continuing to pull changes from other repos with git (since they were based on the same thing). Once one sets up the remotes in git it can be fetched just like any other repo.

I'd also welcome having you or any others as contributors to help us keep this moving forward. I'm still new to RBP so it makes sense to involve those with more experience with RBP as well.

If you have any ideas or thoughts on this, I'd love to hear them.

@gihrig
Copy link
Collaborator Author

gihrig commented Jan 30, 2017

forks aren't searchable unless they have more stars than the original.

Good point, didn't know that.

Once one sets up the remotes in git it can be fetched just like any other repo.

Fetched yes, but mering is another story. I have a work project based on react-boilerplate and I have been reasonably successful keeping it up to date with the source project.

This is done through separate git remotes, origin for my GH repo, which is not a fork, and upstream which is the source react-boilerplate.

My work project exists in branches based on the upstream branches. I've had pretty much mixed results being able to merge upstream branches into my main branch. Sometimes that works, sometimes it results in so many conflicts that I just open both projects in Beyond Compare and manually merge changes.

In the end it's a lot of work, but possible.

@jeffbski
Copy link
Owner

Yeah, I don't know what is the best strategy either.

@gihrig
Copy link
Collaborator Author

gihrig commented Jan 30, 2017

It works best if done frequently. The hard part comes when I get busy and neglect it for a few weeks, then the changes become overwhelming.

@jeffbski
Copy link
Owner

I've merged in your dev branch here, reviewed the files you mentioned and updated the intro doc. I think it is looking pretty good.

@jeffbski
Copy link
Owner

I also just added you as a collaborator.

@gihrig
Copy link
Collaborator Author

gihrig commented Jan 31, 2017

Thank you Jeff! 😄

@jeffbski
Copy link
Owner

You are welcome. Many thanks to you @gihrig for doing so much of the 3.4 conversion, it would have taken me so much longer since I'm new to the project. I just had to merge and resolve a few things. I guess that's why teamwork is so great.

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

No branches or pull requests

2 participants