I noticed when there are more pages in the topic, the links from the Notifications section (bell in the header) do not scroll exactly to the post, as is the case with the red notification boxes on the front page.

there is a difference in the path, here is the path from the home page (from red notification box):
http://www.domain.eu/index.php?u=/topic/topic_number/topic_name/post...

and here is the path from the Notifications section (bell in the header)
http://www.domain.eu/index.php?u=/topic/topic_number/post...

topic_name is missing, when I added the missing topic_name there, the link worked nicely

I found the definition of these links in the file "app.js"

if (data.tid) {
//this is <v.3.7 notification so link needs to be built manually
    link = 'topic/' + data.tid + '/post-' + data.pid +
'&page=from_notify&nid=' + notification.id + '#post-' + data.pid;
    unique = parseInt(data.tid);}

but I don't know in which variable the topic name is defined so I can put it in the path

Please advice

Thanks

Hi

We do not actually send the topic name from the server since that's not required for the scrolling to work.
We will have a look at the issue and if the topic name is required we will make the required changes.

Thank you for digging through this issue.

Thanks, I will be waiting...

I also noticed that link "show all" from Notifications section sometimes gives an error and does not work. Please see the error:

Please advice

Yes, you are right. It's a bug. Thanks for letting us know.
For a fix you can do this:

In file sites/default/assets/js/app.js

Replace on line 1368

        if (window.location.hash) {

with

        if (window.location.hash && window.location.href.match("/user/profile/[0-9]+/edit") != null) {

We checked why this is happening. Apparently when we migrated our router the topic name which was supposed to be optional has become a mandatory parameter in the path. This is a bug, we will fix it. Again, thanks!

Thank you very much!