Skip to content

Require API access to be enabled#490

Merged
Katsute merged 5 commits into
mainfrom
api@b3252a7
Jul 22, 2025
Merged

Require API access to be enabled#490
Katsute merged 5 commits into
mainfrom
api@b3252a7

Conversation

@Katsute

@Katsute Katsute commented Jul 18, 2025

Copy link
Copy Markdown
Member

Relevant Issues

List any closed and/or relevant issues

Changes Made

List any changes made.

  • Add API access option
  • Disable API access by default
  • Fixes to tests

Release Notes

It is now required to enable API access in the options menu in order to use the extension API.

@Katsute Katsute self-assigned this Jul 18, 2025
@Katsute Katsute changed the title move api behind flag Require API access to be enabled Jul 18, 2025
@Katsute
Katsute marked this pull request as ready for review July 20, 2025 19:18
@Katsute

Katsute commented Jul 20, 2025

Copy link
Copy Markdown
Member Author

vscode/test-electron does not support reloading anymore, tests need to be run manually for each 'reload'

@Katsute

This comment has been minimized.

kdevbot[bot]
kdevbot Bot previously approved these changes Jul 20, 2025

@kdevbot kdevbot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved by @Katsute

@Katsute

This comment has been minimized.

@kdevbot kdevbot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved by @Katsute

@Katsute
Katsute added this pull request to the merge queue Jul 22, 2025
Merged via the queue into main with commit 4b4c714 Jul 22, 2025
7 checks passed
@Katsute
Katsute deleted the api@b3252a7 branch July 22, 2025 01:28
@kdevbot kdevbot Bot locked and limited conversation to collaborators Jul 22, 2025
@Katsute Katsute added the v4 label Jul 24, 2025
@Katsute Katsute added this to the M4 milestone Aug 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant