Skip to content

Comments

add direct link to FAQ in web UI where rayhunter log analysis returns warnings#874

Merged
untitaker merged 2 commits intoEFForg:mainfrom
TERR-inss:guide-users-towards-docs
Feb 12, 2026
Merged

add direct link to FAQ in web UI where rayhunter log analysis returns warnings#874
untitaker merged 2 commits intoEFForg:mainfrom
TERR-inss:guide-users-towards-docs

Conversation

@TERR-inss
Copy link

@TERR-inss TERR-inss commented Feb 10, 2026

Pull Request Checklist

  • The Rayhunter team has recently expressed interest in reviewing a PR for this.
    • If not, this PR may be closed due our limited resources and need to prioritize how we spend them.
  • Added or updated any documentation as needed to support the changes in this PR.
  • Code has been linted and run through cargo fmt.
  • If any new functionality has been added, unit tests were also added.
  • CONTRIBUTING.md has been read.

fix #786

@BeigeBox
Copy link
Contributor

Unless I'm reading this wrong the warning alert is inside the {#if !current} block, so it only shows up when the analysis is outdated. It should be moved outside that block so users see the FAQ link whenever there are warnings, not just when they need to reanalyze.

@TERR-inss
Copy link
Author

Unless I'm reading this wrong the warning alert is inside the {#if !current} block, so it only shows up when the analysis is outdated. It should be moved outside that block so users see the FAQ link whenever there are warnings, not just when they need to reanalyze.

Good catch, I'll make that change!

Copy link
Collaborator

@untitaker untitaker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. two changes though (+ what @BeigeBox said)

screenshot for others:

Image

@TERR-inss
Copy link
Author

Thanks for the good feedback @untitaker and @BeigeBox , I just updated the PR based on your comments

@untitaker untitaker merged commit 5efa12f into EFForg:main Feb 12, 2026
31 checks passed
@untitaker
Copy link
Collaborator

thanks!

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.

Guide users towards docs when recording turns red.

3 participants