Skip to content

Revert accidental OneNote rollout from Development - #1538

Merged
Paul Lizer (paullizer) merged 1 commit into
microsoft:Developmentfrom
paullizer:paullizer-revert-accidental-onenote-merge
Sep 26, 2026
Merged

Paul Lizer (paullizer) merged 1 commit into
microsoft:Developmentfrom
paullizer:paullizer-revert-accidental-onenote-merge

Conversation

@paullizer

Copy link
Copy Markdown
Contributor

Summary

  • Normally revert ONLY the accidental Add native OneNote workspace uploads #1525 merge, f3da1b61de7322a9b6a092a258b92d895fb26b6a, using first parent e1e017129c58f01390e3ff533543e004c3f83a5c. No shared history is rewritten.
  • Remove native .one/.onepkg upload support, dispatch, extractor sources/licenses/fixtures/tests, and container packaging from Development pending Reactv2 testing. Existing file formats, frontend code, routes, and authorization are restored exactly to the pre-merge state.
  • Advance the application from 0.261.045 to 0.261.046, rather than downgrading to 0.261.044. Remove the accidental OneNote feature entry while retaining every earlier release entry and adding a corrective Bug Fix note and regression.
  • Preserve the already adapted Reactv2 port on paullizer-react-v2-ui at c9faeae08cfbeccad1ae5212c1d9304f419fdf58. That branch was verified read-only and was not changed or pushed. No second port, deployment, live Azure settings change, data deletion, auto-merge, or PR merge is performed by this correction.

Linked issue

Refs #1525 (corrective follow-up to the merged PR; no new issue).

Release Notes & Latest Features

  • New Feature
  • Bug Fix
  • UI Enhancement
  • Breaking Change
  • Internal only

Is this visible to end users?

  • Yes
  • No

Is this admin-facing (Admin Settings, governance, deployment, config)?

  • Yes
  • No

Should this become a Latest Feature card?

  • Yes
  • No
  • Already added

Screenshot needed for the card?

  • Yes
  • No
  • Attached

Version bump

  • application/single_app/config.py VERSION third segment bumped, or not needed because this is docs-only
  • deployers/version.txt bumped, or not needed because deployers/ was not changed

Application version: 0.261.046. No deployer files changed.

Testing / validation

  • Exact inverse verified: immediately after git revert --no-commit -m 1 f3da1b61de7322a9b6a092a258b92d895fb26b6a, the entire working tree matched the first parent. All 192 original merge paths are reversed, including removal of all 179 newly introduced files.
  • Final scope verified: only four paths differ from the first parent: the config version, corrective release note, fix document, and focused regression. All other application code, packaging, frontend and authorization files match the first parent. Earlier release notes are identical after newline normalization.
  • Corrective regression: 6 tests passed normally and under optimized Python. A read-only red check against the original merge failed all 12 allowlist/category flag combinations as expected, with no harness errors.
  • Integration contracts: 31 tests passed across mixed-source manifests, supported-file-type workspace contracts, and all three route-policy suites. Five existing SWIG deprecation warnings were emitted.
  • Documentation: inventory is current, coverage passed 7/7, and site quality passed 6/6. No inventory regeneration diff was needed.
  • Syntax/whitespace: all 6 surviving changed Python files compiled, and the staged whitespace check passed. No dependency installation, cloud image rebuild, or deployment was needed.
python .\functional_tests\test_onenote_development_revert.py
python -O .\functional_tests\test_onenote_development_revert.py
python .\scripts\build_docs_inventory.py --check
python .\functional_tests\test_docs_app_surface_coverage.py
python .\functional_tests\test_docs_site_quality.py
python -m pytest -q .\functional_tests\test_mixed_source_manifest_contracts.py .\ui_tests\test_workspace_supported_file_types_modal.py .\functional_tests\route_tests\test_route_blueprint_policy_inventory.py .\functional_tests\route_tests\test_route_unauthenticated_policy_contract.py .\functional_tests\route_tests\test_route_policy_test_coverage.py
git -c core.whitespace=blank-at-eol,blank-at-eof,space-before-tab,cr-at-eol diff --cached --check

Documentation

  • Release notes updated, or not needed
  • Feature documentation updated, or not needed
  • Fix documentation updated, or not needed

Fix record: docs/explanation/fixes/ONENOTE_DEVELOPMENT_ROLLOUT_REVERT_FIX.md. The inverse removes the premature OneNote feature documentation and its additions to the user/admin guides.

Security checklist

  • New Flask routes include @swagger_route(security=get_auth_security()) — no routes added or modified
  • Settings sent to non-admin frontends use sanitize_settings_for_user() — existing boundaries unchanged
  • Browser JavaScript is served from local SimpleChat static assets only; no CDN-hosted JS — frontend assets unchanged
  • No secrets, keys, connection strings, or local-only artifacts are included

This PR is intentionally left open for maintainer review and landing.

Revert merge f3da1b6 using first parent e1e0171. Restore Development upload behavior and packaging, preserve the separately adapted Reactv2 port, and advance the application version to 0.261.046. Include corrective release/fix documentation and focused regression coverage. No deployment or shared-history rewrite.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@paullizer
Paul Lizer (paullizer) merged commit 84c9709 into microsoft:Development Sep 26, 2026
12 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.

1 participant