Skip to content

Conversation

wuhan005
Copy link
Owner

Signed-off-by: E99p1ant i@github.red

Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
…r-pwa

Signed-off-by: E99p1ant <i@github.red>

# Conflicts:
#	go.sum
#	internal/cmd/web.go
#	internal/db/db.go
#	internal/db/upload_images.go
#	internal/dbutil/testing.go
#	route/question/page.go
#	route/question/question.go
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
Signed-off-by: E99p1ant <i@github.red>
@wuhan005 wuhan005 requested a review from Copilot September 9, 2025 18:07
Copilot

This comment was marked as outdated.

wuhan005 and others added 2 commits September 10, 2025 02:11
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: E99p1ant <i@github.red>
@wuhan005 wuhan005 requested a review from Copilot September 9, 2025 18:30
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the web application from a server-side rendered template-based architecture to a Progressive Web App (PWA) using Vue.js with a JSON API backend. The refactor maintains the same core functionality while modernizing the frontend architecture.

  • Complete migration from server-rendered HTML templates to a Vue.js frontend with TypeScript
  • Replacement of form-based endpoints with JSON API endpoints for authentication, user management, and question handling
  • Modernization of the build and deployment pipeline to support separate backend and frontend containers

Reviewed Changes

Copilot reviewed 110 out of 115 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
web/* New Vue.js PWA frontend with TypeScript, including components, pages, stores, and API clients
internal/route/* Refactored backend routes to provide JSON APIs instead of template rendering
internal/form/* Updated form validation to support JSON request bodies and multipart data
internal/context/* Simplified context handling for API responses and removed template dependencies
templates/embed.go Removed embedded HTML templates as they're no longer needed
go.mod Updated Go version and dependencies to support the new architecture
Files not reviewed (1)
  • web/pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)

internal/form/form.go:1

  • The logic for handling slice types appears incomplete. When the field is a slice of FileHeader pointers, it should continue to the file handling section below, but the current code structure may not reach that section properly.
// Copyright 2025 E99p1ant. All rights reserved.

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

wuhan005 and others added 3 commits September 10, 2025 02:34
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…r-pwa

Signed-off-by: E99p1ant <i@github.red>

# Conflicts:
#	internal/conf/static.go
#	internal/context/context.go
#	templates/embed.go
Signed-off-by: E99p1ant <i@github.red>
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.

1 participant