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

UX features and bugfixes #133

Open
wants to merge 20 commits into
base: main
Choose a base branch
from

Conversation

qwqVictor
Copy link

This fork contains some improvements, which are briefly descibed below:

  • Support for deleting messages and copying Markdown sources
  • Support for rendering LaTeX formula
  • Typing preview
  • Dockerfile optimization
  • UI icon beautify

Let's talk about Docker.

  1. We should not use npm run dev as Docker image entrypoint, since it starts Vite development server, which can hardly handle requests behind reverse proxies. Because Vite development server injects port number it binds, if we are behind reverse proxies, it will never connect to Vite's websocket endpoint, which causes infinite refreshing. In this case, I compiled the nitro server at image build time and used npm run preview.
  2. We should use layered building, to make use of Docker's layer cache. So I splitted node_module fetching and postinstall triggering.

Another important change is the environment variable. In recent versions the api server endpoint setting is broken, since Nuxt does not recognized the environment variable from .env. I replaced API_BASE_URL with NUXT_PUBLIC_API_BASE_URL to fix the problem. (maybe the root cause of this problem is not here, but it can be a fix. I'm not an expert with Vue ecosystem)

Some demo photos are also updated with HiDPI version.

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.

None yet

1 participant