[Enhancement] - Include versioning to frontend UI#282
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
|
Claude finished @leekahung's task in 1m 18s —— View job Code Review - PR #282: Frontend Versioning (Updated)Tasks
SummaryBoth pieces of feedback from the previous review have been addressed. This is ready to merge.
import { version } from "./package.json";
export default defineConfig({
define: {
__APP_VERSION__: JSON.stringify(version),
},
// ...
});This is the simplest correct fix. The version is read from
The Everything else looks good:
|
yangm2
left a comment
There was a problem hiding this comment.
LGTM - the backend version is in pyproject.toml. I guess we'd need a new backend endpoint so that the frontend could query the version from the backend. We'd also want to expose some of the deployment variables, not secrets, (like GOOGLE_DATASTORE_ID and MODEL_NAME).
That sounds good. |
What type of PR is this? (check all applicable)
Description
Related to PR #180, this PR implements UI to show versioning for the frontend UI. When the backend version is merged in, a follow-up PR could be made to include the version for our backend to help with versioning for Tenant First Aid.
Related Tickets & Documents
QA Instructions, Screenshots, Recordings
Please replace this line with instructions on how to test your changes, a note on the devices and browsers this has been tested on, as well as any relevant images for UI changes.
Added/updated tests?
Documentation
Architecture.mdhas been updated[optional] Are there any post deployment tasks we need to perform?