Rather than taking over node/1/revisions, it seems like Diff's override gets added, resulting in two tabs:

Two tabs

phenaproxima looked into it and said this is because hook_menu_alter() isn't yet converted.

Comments

webchick’s picture

Well. Here's your first problem. There are no docs explaining wtf you're supposed to do. :P

#2177041-68: Remove all implementations of hook_menu

phenaproxima’s picture

Since hook_menu_alter is a purely logical hook that may very well be altering foreign routes, it's not feasible to convert it. However, the plan here is to have the HookMenuAlter plugin generate the boilerplate for altering routes and all different link types.

phenaproxima’s picture

Status: Active » Needs review

As of commit e4a6832, HookMenuAlter now generates all the necessary boilerplate. You can test this against the diff module, which implements it.

webchick’s picture

Status: Needs review » Closed (fixed)

Confirmed boilerplate is there. I can't vouch for its accuracy, but it points off in the right directions for more info so looks good to me. :)