Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

GC: Add Beta Servicer feature #2174

Closed
dadams39 opened this issue Jan 19, 2023 · 2 comments · Fixed by #2205
Closed

GC: Add Beta Servicer feature #2174

dadams39 opened this issue Jan 19, 2023 · 2 comments · Fixed by #2205
Assignees

Comments

@dadams39
Copy link
Contributor

  • To complete Backup / Restore pipelines, a beta connector is required for Pages calls.
Related to:
@dadams39
Copy link
Contributor Author

To lower the amount of code imported, use a similar approach:

@dadams39
Copy link
Contributor Author

This is the best approach as of yet.
The approach involves creating a minimal connector from Kiota for the Sites package and generating the API with the backup Models. In testing, it was noted that all 10k models were generated. The generated model files can be pruned as we already have the majority of the necessary models within v1.0.

aviator-app bot pushed a commit that referenced this issue Jan 31, 2023
## Description
To add needed features, Kiota-Generated Beta Connector created
<!-- Insert PR description-->

## Does this PR need a docs update or release note?

- [x] ✅ Yes, it's included


## Type of change

<!--- Please check the type of change your PR introduces: --->
- [x] 🌻 Feature

## Issue(s)

<!-- Can reference multiple issues. Use one of the following "magic words" - "closes, fixes" to auto-close the Github issue. -->
* closes  #2086<issue>
* closes #2174 
* closes #2223

## Test Plan
Compilation and current CI tests will fail if the package does not align with current operational standards. 
- [x] ⚡ Unit test
- [x] 💚 E2E
aviator-app bot pushed a commit that referenced this issue Jan 31, 2023
## Description
Creates Beta Service for betasdk package. Abstraction loosely complies with the methods found in `graph.Servicer()`. 
Will aid when Sharepoint functionality is supported in v1.0 and `betasdk` package is removed. 
<!-- Insert PR description-->

## Does this PR need a docs update or release note?
- [x] ⛔ No 

## Type of change

<!--- Please check the type of change your PR introduces: --->
- [x] 🌻 Feature

## Issue(s)

<!-- Can reference multiple issues. Use one of the following "magic words" - "closes, fixes" to auto-close the Github issue. -->
- related to * #2174<issue>

## Test Plan

<!-- How will this be tested prior to merging.-->
- [x] 💪 Manual
aviator-app bot pushed a commit that referenced this issue Feb 3, 2023
## Description
- Adds logic to retrieve  `SharePoint.Pages` from M365
- Anchor PR for `SharePoint.Pages` feature support. Restore Pipeline 
PR to remain in Draft to ensure PR Train is stable until the solution to #2174 is implemented. 
<!-- Insert PR description-->

## Does this PR need a docs update or release note?

- [x] ⛔ No 

## Type of change

<!--- Please check the type of change your PR introduces: --->
- [x] 🌻 Feature

## Issue(s)
* related to #2173
* related to #2071
## Test Plan
- [x] ⚡ Unit test

NOTE: Tests will fail in CI due to complications with #2086.
aviator-app bot pushed a commit that referenced this issue Feb 6, 2023
## Description
Updates SharePoint Restore Collection Logic.
Test Suite included.
Restore Pipeline is not connected in this PR for ease of parsing. 

It is noted that there is a large amount of code duplication between Lists and Pages. Code Clean-Up will address these issues once issue #2174 has been handled.  As this will require the use of an HTTP client that is not necessary for other services. 
<!-- Insert PR description-->

## Does this PR need a docs update or release note?

- [x] ⛔ No 

## Type of change

<!--- Please check the type of change your PR introduces: --->
- [x] 🌻 Feature


## Issue(s)

<!-- Can reference multiple issues. Use one of the following "magic words" - "closes, fixes" to auto-close the Github issue. -->
* related to  #2169<issue>

## Test Plan

- [x] ⚡ Unit test
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant