search

UksusoFF / webtrees-tree_view_full_screen

9 stars 0 forks source link

Error with webtrees 2.0 #1

Closed Fa10175 closed 4 years ago

Fa10175 commented 4 years ago

I try your module with webtrees 2.0 and i have this error:

Fatal error: Declaration of UksusoFF\WebtreesModules\TreeViewFullScreen\TreeViewFullScreen::defaultMenuOrder() must be compatible with Fisharebest\Webtrees\Module\ModuleMenuInterface::defaultMenuOrder(): int in ...........\htdocs\webtrees\modules_v4\webtrees-tree_view_full_screen\module.php on line 13

Thanks for any solution

UksusoFF commented 4 years ago

@Fa10175 I still on 1.7 branch. 2.0 version released yesterday. I will update own install and check this issue.

UksusoFF commented 4 years ago

@Fa10175 Please check 0.3 version.

Fa10175 commented 4 years ago

I try 0.3 version. it 's ok for me no error and it do his job thanks

Fa10175 commented 4 years ago

do you try to update "Webtrees - Face" also? let's me know, i can tested also if you need. Thanks and good job

Le lun. 9 déc. 2019 à 14:41, Kirill Uksusov notifications@github.com a écrit :

@Fa10175 https://github.com/Fa10175 Please check 0.3 version: https://github.com/UksusoFF/webtrees-tree_view_full_screen/releases/tag/0.3

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/UksusoFF/webtrees-tree_view_full_screen/issues/1?email_source=notifications&email_token=ACRP2AMEE3ANQTXGI7H6DMDQXZDG7A5CNFSM4JXZL77KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGJGQ7Y#issuecomment-563243135, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACRP2ALCNM4HJ33TE6P6DB3QXZDG7ANCNFSM4JXZL77A .

UksusoFF commented 4 years ago

Yep, but Faces have many logic with database. Now inspecting how it's implemented in wt 2.0. I will update it asap.

UksusoFF commented 4 years ago

@Fa10175 please check new version: https://github.com/UksusoFF/webtrees-faces And create new issue if any troubles with it.