Skip to content

fix: correct app version display #452

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Oct 26, 2023
Merged

fix: correct app version display #452

merged 4 commits into from
Oct 26, 2023

Conversation

@louis-jan
Copy link
Contributor

@louis-jan louis-jan commented Oct 25, 2023 •
edited
Loading

  • Correct version display
  • Send a message without any model running yield a next error (bottom left banner)
  • Don't kill nitro process when switching between models
  • App crashes when core plugin is disabled

@louis-jan louis-jan requested review from hiento09 and urmauur October 25, 2023 09:29
@louis-jan louis-jan temporarily deployed to production October 25, 2023 10:07 — with GitHub Actions Inactive
@louis-jan louis-jan requested review from namchuai and urmauur October 25, 2023 10:11
@louis-jan louis-jan temporarily deployed to production October 25, 2023 10:44 — with GitHub Actions Inactive
@louis-jan louis-jan temporarily deployed to production October 25, 2023 14:46 — with GitHub Actions Inactive
@louis-jan louis-jan merged commit 9903b03 into main Oct 26, 2023
@louis-jan louis-jan deleted the fix/fix-app-version branch October 26, 2023 01:59
hiro-v pushed a commit that referenced this pull request Oct 26, 2023
@louis-jan @jan-service-account
fix: correct app version display (#452)
6f81b7f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hiento09 hiento09 hiento09 approved these changes

@urmauur urmauur Awaiting requested review from urmauur

+1 more reviewer

@namchuai namchuai namchuai approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

No open projects
Menlo
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@louis-jan @urmauur @namchuai @hiento09