Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

修复:邮件服务支持列表链接失效 #2163

Closed
wants to merge 0 commits into from
Closed

修复:邮件服务支持列表链接失效 #2163

wants to merge 0 commits into from

Conversation

Tunglies
Copy link
Contributor

@Tunglies Tunglies commented Nov 1, 2023

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Other... Please describe:

邮件服务支持列表链接失效

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Copy link

vercel bot commented Nov 1, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
qinglong ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 1, 2023 3:38pm
qinglong-dvs8 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 1, 2023 3:38pm

@whyour
Copy link
Owner

whyour commented Nov 1, 2023

@whyour whyour force-pushed the develop branch 3 times, most recently from 1140d4a to 20f615e Compare November 1, 2023 08:44
@Tunglies
Copy link
Contributor Author

Tunglies commented Nov 1, 2023

是不是可以改成 https://raw.githubusercontent.com/nodemailer/nodemailer/master/lib/well-known/services.json

看上去不太直观。
你的提议提示我可以把 Supported services 列到仓库的文件中并外链到 raw github 中。
例如 https://raw.githubusercontent.com/whyour/qinglong/develop/mail_support.md 之类的?
在填写邮箱通知类型中只需要填入 emailService 如 QQ 或 Gmail。
在查询支持服务时只关心服务名称,不必关注额外的参数细节。

@whyour
Copy link
Owner

whyour commented Nov 1, 2023

是不是可以改成 https://raw.githubusercontent.com/nodemailer/nodemailer/master/lib/well-known/services.json

看上去不太直观。 你的提议提示我可以把 Supported services 列到仓库的文件中并外链到 raw github 中。 例如 https://raw.githubusercontent.com/whyour/qinglong/develop/mail_support.md 之类的? 在填写邮箱通知类型中只需要填入 emailService 如 QQ 或 Gmail。 在查询支持服务时只关心服务名称,不必关注额外的参数细节。

可以的,是不是可以直接用 nodemailer 库里的文件生成一个 md 文档

@Tunglies
Copy link
Contributor Author

Tunglies commented Nov 1, 2023

是不是可以改成 https://raw.githubusercontent.com/nodemailer/nodemailer/master/lib/well-known/services.json

看上去不太直观。 你的提议提示我可以把 Supported services 列到仓库的文件中并外链到 raw github 中。 例如 https://raw.githubusercontent.com/whyour/qinglong/develop/mail_support.md 之类的? 在填写邮箱通知类型中只需要填入 emailService 如 QQ 或 Gmail。 在查询支持服务时只关心服务名称,不必关注额外的参数细节。

可以的,是不是可以直接用 nodemailer 库里的文件生成一个 md 文档

我生成了一个 md 并链接到源仓库。 试试看?

@whyour
Copy link
Owner

whyour commented Nov 1, 2023

是不是可以改成 https://raw.githubusercontent.com/nodemailer/nodemailer/master/lib/well-known/services.json

看上去不太直观。 你的提议提示我可以把 Supported services 列到仓库的文件中并外链到 raw github 中。 例如 https://raw.githubusercontent.com/whyour/qinglong/develop/mail_support.md 之类的? 在填写邮箱通知类型中只需要填入 emailService 如 QQ 或 Gmail。 在查询支持服务时只关心服务名称,不必关注额外的参数细节。

可以的,是不是可以直接用 nodemailer 库里的文件生成一个 md 文档

我生成了一个 md 并链接到源仓库。 试试看?

我觉得可能还是搞复杂了,这里只是做个能看到所有服务的地方,不然就需要改成选择服务,而不是直接输入,如果使用邮箱推送消息,还是要知道里面的参数的 https://github.com/nodemailer/nodemailer/blob/master/lib/well-known/services.json 是不是直接用这个链接?

@Tunglies
Copy link
Contributor Author

Tunglies commented Nov 1, 2023

是不是可以改成 https://raw.githubusercontent.com/nodemailer/nodemailer/master/lib/well-known/services.json

看上去不太直观。 你的提议提示我可以把 Supported services 列到仓库的文件中并外链到 raw github 中。 例如 https://raw.githubusercontent.com/whyour/qinglong/develop/mail_support.md 之类的? 在填写邮箱通知类型中只需要填入 emailService 如 QQ 或 Gmail。 在查询支持服务时只关心服务名称,不必关注额外的参数细节。

可以的,是不是可以直接用 nodemailer 库里的文件生成一个 md 文档

我生成了一个 md 并链接到源仓库。 试试看?

我觉得可能还是搞复杂了,这里只是做个能看到所有服务的地方,不然就需要改成选择服务,而不是直接输入,如果使用邮箱推送消息,还是要知道里面的参数的 https://github.com/nodemailer/nodemailer/blob/master/lib/well-known/services.json 是不是直接用这个链接?

就用你提到的 nodemailer 的链接吧。

@whyour
Copy link
Owner

whyour commented Nov 1, 2023

解决下冲突吧

@Tunglies Tunglies closed this Nov 1, 2023
@Tunglies
Copy link
Contributor Author

Tunglies commented Nov 1, 2023

解决下冲突吧

似乎我操作错了,把解决冲突弄成本 branch 强制 sync。白天弄弄吧。

Tunglies added a commit to Tunglies/qinglong that referenced this pull request Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants