Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add mode prop for opening file editors #5256

Merged
merged 1 commit into from
Jun 15, 2021
Merged

Conversation

kulmann
Copy link
Contributor

@kulmann kulmann commented Jun 11, 2021

Description

We've added a parameter called mode to the different ways of opening a file editor. The mode can be edit or create and reflects whether the file editor was opened in an editing mode or in a creation mode.

Resisted the urge to refactor anything, as the extension loading and handling will get an urgently needed overhaul soon anyway.

Related Issue

Motivation and Context

Enable extension developers to react differently to edit and create actions.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@kulmann kulmann self-assigned this Jun 11, 2021
@kulmann kulmann requested a review from LukasHirt June 14, 2021 14:05
@kulmann kulmann force-pushed the file-editor-handler-tweaks branch from c96b3f9 to f02ff7c Compare June 14, 2021 19:45
@ownclouders
Copy link
Contributor

💥 Acceptance tests OCIS-Trashbin-Upload-Move-Journey failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests Trashbin failed. The build is cancelled...

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

22.2% 22.2% Coverage
0.0% 0.0% Duplication

@kulmann kulmann merged commit 1139e2c into master Jun 15, 2021
@delete-merged-branch delete-merged-branch bot deleted the file-editor-handler-tweaks branch June 15, 2021 07:28
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.

Add fileEditor distinction for create and edit in callbacks
3 participants