Closed (outdated)
Project:
Graph API
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Nov 2016 at 14:57 UTC
Updated:
11 Feb 2022 at 09:43 UTC
Jump to comment: Most recent
Comments
Comment #2
clemens.tolboomYeah ... I'll look into that soon as I worked on that code a little too.
Comment #3
clemens.tolboomComment #4
clemens.tolboomReading https://www.drupal.org/docs/8/creating-custom-modules/add-a-composerjson... and https://www.drupal.org/node/2405811 it seems like project users have to install Drupal AND GraphAPI through composer all the way.
Checking our composer.json I have no clue why some requirements are added
For us developer we have to manually add "clue/graph": "dev-master" and others but I know lots of changes and code moves are done in that project so this needs more work :-/
Comment #5
clemens.tolboomComment #6
clemens.tolboomgit blame composer.jsongivesComment #8
clemens.tolboomSo we can fix this for now by let graph-uml decide on version.
This is now kinda fixed. When rendering starts working we need to get back to this I guess.
Comment #9
clemens.tolboomI had to
Comment #10
clemens.tolboomThis is not working as expected.
Changing Drupal composer.json makes the composer.lock file out of date.
Running
composer installon graphapi directory makes it's class loader work.Comment #11
joachim commentedI just did a git pull, and composer install worked:
Comment #12
joachim commentedWhoops, didn't mean to change status!
Comment #13
clemens.tolboomIt works but not as documented by d.o ... we'll live for now so not critical.
Comment #14
clemens.tolboomI cannot get demo pages to work http://drupal.d8/admin/config/system/graphapi/formats
$text .= 'See ' . l("$project_id", 'http://drupal.org/project/' . $project_id) . ' on drupal.org';Temp fixing both and install the remaining dependencies through root composer as the provided dependencies do not work.
does not help as
which makes me wonder how many code changes we need to make it work again.
Comment #15
joachim commented