Skip to content

Conversation

@AishDani
Copy link
Contributor

@AishDani AishDani commented Sep 4, 2025

No description provided.

@AishDani AishDani requested a review from a team as a code owner September 4, 2025 06:34
Copy link
Contributor

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 implements bug fixes related to authentication token management and API service improvements. The changes focus on ensuring proper token handling during login/logout flows and fixing dynamic token retrieval in API calls.

  • Added a new clearAuthToken action to properly clear authentication state
  • Converted static options objects to dynamic functions for fresh token retrieval
  • Fixed locale mapping logic to handle key updates correctly

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
ui/src/store/slice/authSlice.tsx Added clearAuthToken reducer action and exported it
ui/src/services/api/stacks.service.ts Converted options from static object to function for dynamic token retrieval
ui/src/services/api/migration.service.ts Converted options from static object to function and updated all API calls
ui/src/pages/Migration/index.tsx Added numeric label filtering in locale validation
ui/src/pages/Login/index.tsx Added token clearing logic and imported necessary functions
ui/src/components/DestinationStack/Actions/LoadLanguageMapper.tsx Fixed locale key mapping by adding old key deletion logic

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

@umeshmore45
Copy link
Contributor

@sayalijoshi27 please review this

@sayalijoshi27 sayalijoshi27 merged commit caf002a into dev Sep 4, 2025
8 checks passed
@sayalijoshi27 sayalijoshi27 deleted the bugfix/beta-release branch September 4, 2025 13:36
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.

4 participants