-
-
Notifications
You must be signed in to change notification settings - Fork 204
Incorrect URLs in emails sent from API #597
Comments
@wellingguzman we all waiting for you, where are you? :) |
It might be the path, or the logo might be missing too... either way let's also make sure that we're using the newest logo. I can provide it if needed... |
I’m guessing the path to the logo is using the same wrong |
As I said XD |
The password reset url are based on the Fixing this #502 (comment) issue was making this value hard default to This should be improved to try to guess the url based on the request. |
@wellingguzman @rijkvanzanten — let's get this one solved. Any ideas on the best way to get this path? |
Also, #609 is related to the mail having wrong links ^^. |
The problem with the logo is that the link to the I am going to fallback to the url based on the request instead of |
The API has not field for But in
Otherwise it will fallback to the url based on the HTTP Request (guessing your url), the same way it guesses the uploaded files full URLs. The Logo url has been updated to The URLs should be working as expected now. Otherwise let us know and we will reopen this ticket to address any issues with the new implementation. |
@wellingguzman @rijkvanzanten — would it make sense to include this as a global setting in the App so it can be adjusted more easily? |
@benhaynes I believe so, the option is there so it would make sense to add that field to the settings page. I open a ticket for this: #670 |
I added the field to to Directus Settings Fields. I added that missing field the bottom of the form. I wonder if that looks ok, or should it be moved to the top, along with the So it looks like this:
If we want to have this conversation we could talk about otherwise, the field has been added by 4f82d08. |
Gotcha. @rijkvanzanten — can we add this field into the App? |
@benhaynes I don't know if @rijkvanzanten needs to do something on the app side, but this needs to be done in the API side. Add these fields records in the |
Oh, I thought you said it was already added. 😕 I'll re-open... but let's get this added (if it isn't already) and call it done! |
oh no, I added the I was referring to add the missing |
👍 |
When requesting a password reset, the link directs to localhost and not to the api url.
I guess the image is also not showing because of that.
Directus 2.0.8
The text was updated successfully, but these errors were encountered: