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

Fedora url #742

Merged
merged 6 commits into from
Dec 11, 2019
Merged

Fedora url #742

merged 6 commits into from
Dec 11, 2019

Conversation

Natkeeran
Copy link

** Please hold on from testing. Will provide more info. **

GitHub Issue: (link)
Part of the work towards: Islandora/documentation#926

@bradspry
Copy link

bradspry commented Nov 8, 2019

This is a fantastic integration on multiple levels! Thank You for your total grasp of Drupal multi-site and settings.php! 💯

@dannylamb
Copy link

Travis is going to continue to fail until #747 gets merged, but I've tested this with all the other PRs for Islandora/Alpaca#65 and it's all good. I'm happy moving forward, as this has nothing to do with why tests are failing.

@dannylamb dannylamb merged commit 63a035f into Islandora:8.x-1.x Dec 11, 2019
@elizoller
Copy link
Member

does this mean you can just modify the flysystem block in the settings.php to accomodate a new root?
like

  'fedora' => [
    'driver' => 'fedora',
    'config' => [
      'root' => 'http://127.0.0.1:8080/fcrepo/rest/',
    ],
  ],
];

could become

$settings['flysystem'] = [
  'fedora' => [
    'driver' => 'fedora',
    'config' => [
      'root' => 'http://127.0.0.1:8080/fcrepo/rest/mynewroot/',
    ],
  ],
];

@dannylamb
Copy link

Yes, although we may still have to smooth out islandora-playbook to make sure everything deploys properly if you build a new box.

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.

4 participants