Skip to content

Added more details on SMTP authentication#70

Open
Som23Git wants to merge 1 commit into
postalserver:mainfrom
Som23Git:patch-1
Open

Added more details on SMTP authentication#70
Som23Git wants to merge 1 commit into
postalserver:mainfrom
Som23Git:patch-1

Conversation

@Som23Git

Copy link
Copy Markdown
Contributor

Team, I was trying to improvise the documentation while I try to work on the postal server. Added an example snippet to support the SMTP authentication and correlated it with demo screenshot from the server itself.

@willpower232

Copy link
Copy Markdown
Contributor

Isn't this technically incorrect because the username is usually prefixed with the organisation slug? If you see above the list of credentials there is a link to the details to use for SMTP which shows the username (and first SMTP credential I think)

@Som23Git

Som23Git commented Aug 30, 2024

Copy link
Copy Markdown
Contributor Author

Hi @willpower232,

Thank you for your review—I really appreciate it.

My bad, I misunderstood earlier. I see your point now.

Isn't this technically incorrect because the username is usually prefixed with the organization slug?

So, you're saying this is correct? (See attached screenshot: correct_username_displayed_help.png)

image

It seems that the previous screenshot, smtp_credentials.png, was pointing to the friendly name (a.k.a. name) of the generated credential.

smtp_credentials

Replacing smtp_credentials.png with correct_username_displayed_help.png should fix the issue(within this PR), right?

What do you think? Thanks!

@willpower232

Copy link
Copy Markdown
Contributor

Yeah something like that 👌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants