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

Dismiss LLM onboarding view if token is already present #83

Closed
wants to merge 1 commit into from

Conversation

philippzagar
Copy link
Contributor

@philippzagar philippzagar commented Mar 14, 2024

Dismiss LLM onboarding view if token is already present

♻️ Current situation & Problem

Currently, the LLM onboarding view is shown for every LLM interaction, even when the token is present.

⚙️ Release Notes

  • Dismiss LLM onboarding view if token is already present

📚 Documentation

--

✅ Testing

No testing occurred, responsibility of @nriedman

📝 Code of Conduct & Contributing Guidelines

By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

@philippzagar philippzagar added the enhancement New feature or request label Mar 14, 2024
@philippzagar philippzagar requested a review from nriedman March 14, 2024 17:10
@philippzagar philippzagar self-assigned this Mar 14, 2024
@philippzagar
Copy link
Contributor Author

@nriedman This is NOT a complete PR, please ensure to expand this functionality for all LLM interactions and do proper testing

Copy link

codecov bot commented Mar 14, 2024

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 23.31%. Comparing base (9881f4f) to head (4cd8ede).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
- Coverage   23.32%   23.31%   -0.01%     
==========================================
  Files          62       62              
  Lines        3667     3669       +2     
==========================================
  Hits          855      855              
- Misses       2812     2814       +2     
Files Coverage Δ
Intake/ChiefComplaint/LLMAssistantView.swift 0.00% <0.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9881f4f...4cd8ede. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant