Delivery note not generating, showing same as invoice

Same issue as Delivery note the same as invoice - Self-Hosted | v5 - Discourse

Just recently migrated to V5.3.84-C78 from V4

Running on Shared Hosting, installed by Softaculous

I was trying everything out to make sure everything works but I found that the delivery note is not generating, it is just displaying the invoice

Showing Invoice

Showing delivery note (note delivery note is ticked)

Hi,

@david any thoughts on this? I haven’t been able to reproduce it.

@waaiez

Can you please check the browser console for any errors or messages when you click on The Delivery Note option. Something doesn’t sounds right here.

No errors

Some more information:
Running PHP 7.4

When clicking view pdf to view the invoice console shows a request being made to:
https://sub.domain.co.za/public/client/invoice/u79ir3bMAxQ8aK76eW2yj9Wta4i0e1KZ/download?t=1651446363160

Clicking the delivery note option console show a request being made to:
https://sub.domain.co.za/public/index.php/api/v1/invoices/JAPdRkYeGy/delivery_note?&per_page=999999

Thanks, that’s helpful.

Is it possible mod_rewrite isn’t enabled? I wouldn’t expect to see ‘index.php’ in the URL.

I had some custom rules because of some unrelated issues which I managed to fix
Commented out those rules but it is still showing the invoice instead of a delivery note

URL when viewing invoice:
https://sub.domain.co.za/public/client/invoice/u79ir3bMAxQ8aK76eW2yj9Wta4i0e1KZ/download?t=1651491002549

URL when viewing delivery note:
https://sub.domain.co.za/api/v1/invoices/JAPdRkYeGy/delivery_note?&per_page=999999

Current .htaccess file:

<IfModule mod_rewrite.c>
  RewriteEngine On
  
  # added by me
  #RewriteBase /
  
  RewriteRule "^.env" - [F,L]
  #RewriteRule "^storage" - [F,L]
  RewriteRule ^(.well-known)($|/) - [L]
  
  #RewriteCond %{REQUEST_URI} !^/public/? [NC]
  RewriteRule ^(.*)$ public/$1 [L]
</IfModule>

Does it make a difference if you use the web or desktop app?

No difference, still showing the invoice

@hillel

it looks like we aren’t passing a cache buster ?t parameter for the delivery notes, i think it may be a cache issue?

Thanks, I’ll add it in the next release