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

GC: SharePoint: Backup: Abstract / Serialize #2187

Merged
merged 180 commits into from
Feb 8, 2023
Merged

Conversation

dadams39
Copy link
Contributor

@dadams39 dadams39 commented Jan 19, 2023

Description

Changes address updates to sharePoint.Collection.Populate().

  • SharePoint Collections support Lists and Pages. Drives are supported in OneDrive at this time.
  • List's serialize function is abstracted to support Pages.

Collection needs to support List and Pages support. Additionally, needs to use a similar interface as in exchange to ensure retry and async collection population

Does this PR need a docs update or release note?

  • ✅ Yes, it's included

Type of change

  • 🌻 Feature

Issue(s)

Test Plan

Test can be completed locally. Per Issue #2086, the Beta library interferes with overall caching.
TestSharePointPageCollection_Populate() inspects changes

  • ⚡ Unit test

dadams39 and others added 23 commits January 18, 2023 22:55
Uses Beta library for pages.
Removes minimal definition of SitePages from page.
Adds logic for fetching Page M365 IDs.
Adds function collectPages() to logic of dataCollections.
Adds field to hold Beta service within struct for populate method.
Increases swap space for build.
Troubleshooting
Troubleshooting passed 8 GB.
Adds abstraction to the backup process.
Adds new identifier to NewCollection() call
Adds new identifier to NewCollection() call
@dadams39 dadams39 temporarily deployed to Testing January 19, 2023 18:19 — with GitHub Actions Inactive
@dadams39 dadams39 temporarily deployed to Testing January 19, 2023 18:19 — with GitHub Actions Inactive
@dadams39 dadams39 temporarily deployed to Testing February 6, 2023 15:34 — with GitHub Actions Inactive
@dadams39 dadams39 temporarily deployed to Testing February 6, 2023 15:34 — with GitHub Actions Inactive
Collection test for Pages added.
@dadams39 dadams39 temporarily deployed to Testing February 8, 2023 16:47 — with GitHub Actions Inactive
@dadams39 dadams39 temporarily deployed to Testing February 8, 2023 16:47 — with GitHub Actions Inactive
@dadams39 dadams39 temporarily deployed to Testing February 8, 2023 16:47 — with GitHub Actions Inactive
@dadams39 dadams39 temporarily deployed to Testing February 8, 2023 16:48 — with GitHub Actions Inactive
@dadams39 dadams39 temporarily deployed to Testing February 8, 2023 16:48 — with GitHub Actions Inactive
@dadams39 dadams39 temporarily deployed to Testing February 8, 2023 16:48 — with GitHub Actions Inactive
Updates to consolidate TestCollection_Items into table format.
@dadams39 dadams39 temporarily deployed to Testing February 8, 2023 17:40 — with GitHub Actions Inactive
@dadams39 dadams39 temporarily deployed to Testing February 8, 2023 17:40 — with GitHub Actions Inactive
@dadams39 dadams39 temporarily deployed to Testing February 8, 2023 17:41 — with GitHub Actions Inactive
@aviator-app
Copy link
Contributor

aviator-app bot commented Feb 8, 2023

Aviator status

Aviator will automatically update this comment as the status of the PR changes.

This PR was merged using Aviator.

@aviator-app aviator-app bot temporarily deployed to Testing February 8, 2023 17:43 Inactive
@aviator-app aviator-app bot temporarily deployed to Testing February 8, 2023 17:43 Inactive
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 8, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

No Coverage information No Coverage information
0.0% 0.0% Duplication

@aviator-app aviator-app bot temporarily deployed to Testing February 8, 2023 17:44 Inactive
@aviator-app aviator-app bot temporarily deployed to Testing February 8, 2023 17:44 Inactive
@aviator-app aviator-app bot temporarily deployed to Testing February 8, 2023 17:44 Inactive
@aviator-app aviator-app bot temporarily deployed to Testing February 8, 2023 17:44 Inactive
@aviator-app aviator-app bot merged commit 3d244c9 into main Feb 8, 2023
@aviator-app aviator-app bot deleted the backup-page-flow-2 branch February 8, 2023 18:09
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.

2 participants