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

Update to latest from Panopoly #122

Open
RobLoach opened this issue Nov 12, 2015 · 5 comments
Open

Update to latest from Panopoly #122

RobLoach opened this issue Nov 12, 2015 · 5 comments
Labels

Comments

@RobLoach
Copy link
Contributor

Two possible methods:

  1. Open new branch
  2. Update through git cli remotes
  3. Send PR for the update

or

  1. Create a backup on pantheon
  2. Verify update is legit in multidev
  3. Pull code from pantheon and push into branch on github for PR
@RobLoach
Copy link
Contributor Author

Opened at https://dashboard.pantheon.io/sites/f0072597-f475-4513-af94-13a33b630923#plb-122 , will require a git conflict resolution, option 1 above.

@RobLoach RobLoach self-assigned this Dec 10, 2015
@RobLoach RobLoach removed their assignment Jan 13, 2016
@chhavi
Copy link

chhavi commented Jan 20, 2016

I have updated panopoly to the latest version. I referred to the below documentation.

I've tested all the pages on my local setup and everything looks fine.

https://www.drupal.org/node/2272177.
Pull request can be seen here - #124

@andrewmallis
Copy link
Member

@chhavi – I am not clear from your comment which of the steps you followed from the linked doc. Either way, this site is hosted on Pantheon, so we should instead look to:
https://pantheon.io/docs/articles/sites/code/applying-upstream-updates/

Panopoly tracks against https://github.com/populist/panopoly-drops-7 so in this context, it would look something like:

  git pull -Xtheirs git://github.com/populist/panopoly-drops-7.git master
  # resolve conflicts
  git push origin master

@RobLoach – at the end of the day, I would like the dashboard to reflect the updated changes, so please ensure our multidev reflects this. (the current plb-122 one doesn't).

sidenote: We have an upgrade multi-dev branch for this here: https://dashboard.pantheon.io/sites/f0072597-f475-4513-af94-13a33b630923#upgrade
and I tried quickly applying the latest dashboard updates, but it fails. :(

@chhavi
Copy link

chhavi commented Jan 21, 2016

@andrewmallis I first opened the link of Pantheon, but it was asking for credentials. So i thought let's quickly explore the other options and see how it goes. I followed the manual steps to update panopoly. I also took the backup of overriden features using 'Features Override' module. I ran update.php twice and everything was working fine.

But i can also try updating it on Pantheon. I'll connect with @RobLoach to understand the current plb-122 status, and will take it forward.

@chhavi
Copy link

chhavi commented Jan 22, 2016

@andrewmallis I have updated panopoly as per your suggestion -

git pull -Xtheirs git://github.com/populist/panopoly-drops-7.git master
# resolved conflicts
git push origin new-branch

NOTE : Run update.php after pulling the code.
Pull request is available here - #128

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

No branches or pull requests

3 participants