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

Release from Stage to Main #104

Merged
merged 8 commits into from
Aug 16, 2024
Merged

Release from Stage to Main #104

merged 8 commits into from
Aug 16, 2024

Conversation

sukamat
Copy link
Contributor

@sukamat sukamat commented Jul 26, 2024

Releases:

MWPW-152581

Raghu A and others added 8 commits June 27, 2024 12:27
…history

During promopte use the upload content api instead of copy as the version history is lost.

Resolves: MWPW-152581
During promote use the upload content api instead of copy as the version
history is lost.

The else block for lock file can also be removed as another PR. We still
get lock file errors.
(Remove else of await sharepoint.saveFile(content, filePath) )

Test: 
<img width="348" alt="image"
src="https://github.com/adobecom/milo-fg/assets/125877471/eb6acb14-a9a2-45fb-8fb0-fc5c64ded030">
…n excel and ui

1. Report as locked when the file is locked / could not be written due to lock (in Copy or Promote)
2. In Parallel waits for all request to be processed before picking next. Using the inparallel (from miloc) which will pick next tasks as and when a previous task compltes
3. Using request wrapper (from miloc) for handling the and option to skip status
4. Timeout or Fetch errors are also retry (as internally reported as 503 instead)
5. Locked flag is passed over for reporting
Copy to floodgate tree also upload files instead of copy to retain version.
Opened files will reported as locked
…n excel and ui (#103)

Updates

1. Report as locked when the file is locked / could not be written due
to file being open (in Copy or Promote)
2. In parallel waits for all request to be processed before picking
next. Using the inparallel (from miloc) which will pick next tasks as
and when a previous task completes.
3. Using parts of request wrapper (from miloc) for handling the status
code and has option to skip status when required.
4. Timeout (Status code 600) or Fetch errors (Status code 503) are also
retry
5. Locked flag is passed over for reporting
6. Test cases updated

Tests - 
Scenario 1 - Copy from Main Tree to Floodgate Tree retain version
<img width="494" alt="image"
src="https://github.com/user-attachments/assets/37ede04b-4bf7-445a-88dd-e1184b6b6120">

Scenario 2 - File is opened and copy is triggered

![image](https://github.com/user-attachments/assets/07b0e3d2-8862-476b-a75a-38ec0d63c9c6)

Scenario 3 - File is locked and copy is triggered
| Reported in GUI | Reported in Excel |
| -------- | -------- |
| <img width="274" alt="image"
src="https://github.com/user-attachments/assets/9d4b4a6c-0494-4e8f-a8b9-809ec36b0073">
|
![image](https://github.com/user-attachments/assets/dda6090f-b799-47ff-bcd1-789cfcfe0c4a)
|

Scenario 4 - File is opened (or Locked) and promote is triggered
| Reported in GUI | Reported in Excel |
| -------- | -------- |
|
![image](https://github.com/user-attachments/assets/2741c7f6-d407-43cd-863e-ffeea26210d4)
|
![image](https://github.com/user-attachments/assets/4aafab71-2fe7-4716-98ef-85999f0a8d9f)
|
@sukamat sukamat requested a review from raga-adbe-gh July 26, 2024 19:02
@sukamat sukamat merged commit 002ab35 into main Aug 16, 2024
4 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.

3 participants