Skip to content

Migration to TypeScript and changed API-calls - #11

Merged
zagemello merged 7 commits into
mainfrom
Migration_to_typescript_and_changed_api_calls
Nov 14, 2025
Merged

Migration to TypeScript and changed API-calls#11
zagemello merged 7 commits into
mainfrom
Migration_to_typescript_and_changed_api_calls

Conversation

@Wunderwaffel

@Wunderwaffel Wunderwaffel commented Nov 11, 2025

Copy link
Copy Markdown
Collaborator
  • Migrated away from JavaScript to TypeScript
  • Changed the way API calls are made (services + environment variables)
  • Changed main.css settings
  • Added pre-commit hooks for automatic code formatting and linting

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +3 to +6
interface User {
id: number;
username: string;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Align auth store user fields with API response

The new User interface assumes a property named id, but the login API returns user_id (see UserResponse in the backend). Because setAuth stores the raw response object, authStore.user.id is always undefined after a successful login, so any view or subsequent request that reads the user ID will fail. Either map user_id to id when storing the user or update the interface to match the backend payload.

Useful? React with 👍 / 👎.

@zagemello zagemello left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@zagemello
zagemello merged commit 53d7afc into main Nov 14, 2025
4 checks passed
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.

2 participants