Skip to content

Conversation

@manuelseeger
Copy link
Owner

@manuelseeger manuelseeger commented Apr 25, 2025

[0.3.1] - 2025-04-25

Added

  • Click to open query in mouse hover
  • Add odata.openResultInNewPane setting to control if query results open in a new pane (new default) or the active pane

Fixed

  • Profile UI not updating after new profile selection
  • Credentials synced between VSCode profiles; Now stored in local secret store

@manuelseeger manuelseeger requested a review from Copilot April 25, 2025 10:21
Copy link

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 bumps the version to 0.3.1 and adds new functionality to open query results in a new pane while addressing profile UI update issues and credentials synchronization between VSCode profiles. Key changes include refactoring configuration merging in defaultConfig, updating module import paths to use stubs, and adjusting formatting and naming conventions across the codebase.

Reviewed Changes

Copilot reviewed 33 out of 34 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/odata2ts/defaultConfig.ts Refactored getMinimalConfig to use spread operator over deepmerge
src/odata2ts/data-model/* Updated import paths and reformatted error messages
src/odata2ts/NamingModel.ts Changed OverridableNamingOptions to extend NameSettings
src/contracts/types.ts, src/configuration.ts Added openResultInNewPane support in configuration
src/commands.ts, src/hovers.ts, src/chat.ts Adjusted profile retrieval and formatting improvements
assets/main.js Removed deprecated webview script
README.md, CHANGELOG.md, .github/copilot-instructions.md Updated documentation and development guidelines
Files not reviewed (1)
  • package.json: Language not supported
Comments suppressed due to low confidence (1)

src/odata2ts/NamingModel.ts:17

  • [nitpick] Changing OverridableNamingOptions to extend NameSettings instead of PartialDeep now requires all properties to be provided. Confirm that this stricter typing is intentional to maintain backward compatibility.
export interface OverridableNamingOptions extends NameSettings {}

@manuelseeger manuelseeger merged commit c9f7c52 into main Apr 25, 2025
1 check 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