同时搜索公开话题和私信

继续讨论 创建/查看和创建权限(再次)

我认为这是一个很棒的想法。目前我们有 in:private 修饰符来搜索私信,所以也许可以类似地添加 in:allin:everywhere

9 个赞

@david:谢谢创建这个帖子(我本来打算等我有代码了再发的 :))。我昨天和今天会议间隙稍微修改了一下,现在有一个添加了 in:all 搜索标签的概念验证:GitHub - adqm/discourse at search_all · GitHub

有几件事值得思考:

  • 这只是一个快速 hack,除了在一个包含几条帖子的小型测试实例上进行测试外,我还没有真正测试过。
  • in:all 这个名字合适吗?
  • 或许可以在列表中显示哪些结果是私信(PM),哪些是主题。
  • 需要在“高级搜索”窗口中添加一个复选框。

如果有时间,我很乐意继续修改。而且我对 Ruby、Rails 或 Discourse 代码库还不太熟悉,所以任何反馈和建设性批评都非常欢迎!

5 个赞

太棒了!准备好后请提交一个 PR,这样大家就能更方便地评论了。

我觉得没问题,除非有人有更好的建议?cc @codinghorror

确实如此,但刚开始不必为此担心。这可以作为后续的 PR 来实现。

同样,这确实不错,但不一定非要同时完成。毕竟这个功能主要是面向高级用户的。

6 个赞

再稍微测试了一下,发现它还没完全准备好……不过我会继续折腾,等我修复已知的 bug 后就会提交 PR。

7 个赞

刚开了一个 PR。欢迎提出想法、反馈和批评!

7 个赞

in:all 没问题,in:everything 输入起来太麻烦了,依我之见。

8 个赞

感谢 @hartz,此功能现已实现 :confetti_ball:

您可以使用 in:all 在一个位置搜索您的私信和公开主题。例如:

11 个赞

嗯。PM 结果旁边不应该显示 PM 信封吗?否则你怎么知道哪个是哪个?

12 个赞

是的,那样很好。@hartz 其实已经提出过这个建议,但我坚持先实现基本功能。

@hartz 你愿意尝试在第二个 PR 中实现这个功能吗?如果不行的话,我可以来查看一下。

8 个赞

当然,我很乐意尝试一下。我想在代码库中多探索一番(因为我还不太清楚整体情况),这正是一个好机会。

不过,我可能还需要几天时间才能真正着手查看,所以如果你愿意直接介入并处理,那也没问题。

6 个赞

@david:我终于有机会查看这个功能了(在搜索结果中区分私信和主题)。我本地实现了一个不太优雅的临时方案,看起来能行,但可能还需要大量优化。如果我觉得它还没到可以正式推出的阶段,我是应该在开发分类下单独开一个主题讨论,还是直接提一个 PR,并把它作为讨论的地方?

4 个赞

如果您已有可运行的代码,提交一个 PR 会非常有帮助——这样我们更容易针对代码的具体部分进行评论。如果您愿意,GitHub 推出了新的“草稿 PR”功能(在提交前点击“创建拉取请求”旁边的箭头即可)。

7 个赞

好的,听起来不错,谢谢!我刚刚提交了一个 PR

7 个赞

图标现已合并——感谢 @hartz :slight_smile:

8 个赞

此主题已在 5 天后自动关闭。不再允许新的回复。