feat: add support for origin server and user info #2663

dwong2708 · 2025-11-19T00:14:31Z

Description

Resolves: #2596

This PR introduces the following improvements:

Display of the number of sections, subsections, and units.
Display of the instance name where the archive was created.
Display of the email of the user who generated the archive.

Before:

Now:

Where can I find this section?

This UI appears when creating a new V2 library using the “Create from archive” option.

Testing instructions

Create a new V2 library.
Select the Create from archive option.
Upload a backup archive .zip file.
Review the summary section and confirm the displayed information.

Other information

Related backend PR: openedx/edx-platform#37626

Best Practices Checklist

We're trying to move away from some deprecated patterns in this codebase. Please
check if your PR meets these recommendations before asking for a review:

Any new files are using TypeScript (.ts, .tsx).
Avoid propTypes and defaultProps in any new or modified code.
Tests should use the helpers in src/testUtils.tsx (specifically initializeMocks)
Do not add new fields to the Redux state/store. Use React Context to share state among multiple components.
Use React Query to load data from REST APIs. See any apiHooks.ts in this repo for examples.
All new i18n messages in messages.ts files have a description for translators to use.
Avoid using ../ in import paths. To import from parent folders, use @src, e.g. import { initializeMocks } from '@src/testUtils'; instead of from '../../../../testUtils'

openedx-webhooks · 2025-11-19T00:14:36Z

Thanks for the pull request, @dwong2708!

This repository is currently maintained by @bradenmacdonald.

Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review.

🔘 Get product approval

If you haven't already, check this list to see if your contribution needs to go through the product review process.

If it does, you'll need to submit a product proposal for your contribution, and have it reviewed by the Product Working Group.
- This process (including the steps you'll need to take) is documented here.
If it doesn't, simply proceed with the next step.

🔘 Provide context

To help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:

Dependencies

This PR must be merged before / after / at the same time as ...
Blockers

This PR is waiting for OEP-1234 to be accepted.
Timeline information

This PR must be merged by XX date because ...
Partner information

This is for a course on edx.org.
Supporting documentation
Relevant Open edX discussion forum threads

🔘 Get a green build

If one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green.

Where can I find more information?

If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources:

When can I expect my changes to be merged?

Our goal is to get community contributions seen and reviewed as efficiently as possible.

However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:

The size and impact of the changes that it introduces
The need for product review
Maintenance status of the parent repository

💡 As a result it may take up to several weeks or months to complete a review and merge your PR.

codecov · 2025-11-21T16:48:44Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.85%. Comparing base (54cfbeb) to head (644b076).
⚠️ Report is 23 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2663      +/-   ##
==========================================
+ Coverage   94.83%   94.85%   +0.01%     
==========================================
  Files        1234     1232       -2     
  Lines       27764    27873     +109     
  Branches     6094     6133      +39     
==========================================
+ Hits        26331    26439     +108     
- Misses       1375     1376       +1     
  Partials       58       58

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

diana-villalvazo-wgu · 2025-11-21T19:42:01Z

src/library-authoring/create-library/CreateLibrary.tsx

+                      {
+                        (restoreStatus.result.createdBy?.email && restoreStatus.result.createdOnServer) && (
+                          <div className="d-flex align-items-md-center gap-2">
+                            <Icon src={PersonOutline} style={{ width: '20px', height: '20px', marginRight: '8px' }} />


instead of inline styling for margin can we use bootstrap classes? like: 'mr-2'

https://paragon-openedx-v23.netlify.app/foundations/css-utilities/

Done. Thanks

holaontiveros

Other than th comment from Diana LGTM

src/library-authoring/create-library/CreateLibrary.test.tsx

src/library-authoring/create-library/messages.ts

diana-villalvazo-wgu

LGTM 🚀

jacobo-dominguez-wgu

👍🏻

diana-villalvazo-wgu · 2025-11-25T22:32:32Z

Before merge i will wait to see if @bradenmacdonald have some comments to add 😄

bradenmacdonald · 2025-11-26T23:17:48Z

From a very quick look, this seems good, and I'm happy with the other reviews that were done here. Go ahead and merge :)

* feat: add support for origin server and user info * test: add coverage for restore archive summary * test: increase coverage for restore archive summary * fix: address comments

feat: add support for origin server and user info

1660ad8

openedx-webhooks added the open-source-contribution PR author is not from Axim or 2U label Nov 19, 2025

openedx-webhooks added this to Contributions Nov 19, 2025

github-project-automation bot moved this to Needs Triage in Contributions Nov 19, 2025

mphilbrick211 added the mao-onboarding Reviewing this will help onboard devs from an Axim mission-aligned organization (MAO). label Nov 19, 2025

mphilbrick211 moved this from Needs Triage to Waiting on Author in Contributions Nov 19, 2025

test: add coverage for restore archive summary

9e1564c

test: increase coverage for restore archive summary

47be3ef

dwong2708 marked this pull request as ready for review November 21, 2025 19:31

dwong2708 requested a review from holaontiveros November 21, 2025 19:31

diana-villalvazo-wgu reviewed Nov 21, 2025

View reviewed changes

jesusbalderramawgu approved these changes Nov 21, 2025

View reviewed changes

holaontiveros approved these changes Nov 21, 2025

View reviewed changes

jacobo-dominguez-wgu reviewed Nov 21, 2025

View reviewed changes

src/library-authoring/create-library/CreateLibrary.test.tsx Outdated Show resolved Hide resolved

jacobo-dominguez-wgu reviewed Nov 21, 2025

View reviewed changes

src/library-authoring/create-library/CreateLibrary.test.tsx Outdated Show resolved Hide resolved

jacobo-dominguez-wgu reviewed Nov 21, 2025

View reviewed changes

src/library-authoring/create-library/messages.ts Outdated Show resolved Hide resolved

fix: address comments

644b076

dwong2708 requested review from diana-villalvazo-wgu and jacobo-dominguez-wgu November 24, 2025 18:57

diana-villalvazo-wgu approved these changes Nov 24, 2025

View reviewed changes

diana-villalvazo-wgu requested a review from bradenmacdonald November 24, 2025 18:58

jacobo-dominguez-wgu approved these changes Nov 24, 2025

View reviewed changes

diana-villalvazo-wgu merged commit 294fe42 into openedx:master Nov 27, 2025
7 checks passed

github-project-automation bot moved this from Waiting on Author to Done in Contributions Nov 27, 2025

diana-villalvazo-wgu deleted the dwong2708/libV2_restore_adjustments branch November 27, 2025 15:04

dwong2708 mentioned this pull request Nov 27, 2025

feat: add support for origin server and user info (#2663) #2710

Open

7 tasks

feat: add support for origin server and user info #2663

feat: add support for origin server and user info #2663

Uh oh!

Conversation

dwong2708 commented Nov 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Where can I find this section?

Testing instructions

Other information

Best Practices Checklist

Uh oh!

openedx-webhooks commented Nov 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

codecov bot commented Nov 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

diana-villalvazo-wgu Nov 21, 2025

Choose a reason for hiding this comment

Uh oh!

jacobo-dominguez-wgu Nov 21, 2025

Choose a reason for hiding this comment

Uh oh!

dwong2708 Nov 24, 2025

Choose a reason for hiding this comment

Uh oh!

holaontiveros left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

diana-villalvazo-wgu left a comment

Choose a reason for hiding this comment

Uh oh!

jacobo-dominguez-wgu left a comment

Choose a reason for hiding this comment

Uh oh!

diana-villalvazo-wgu commented Nov 25, 2025

Uh oh!

bradenmacdonald commented Nov 26, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

dwong2708 commented Nov 19, 2025 •

edited

Loading

openedx-webhooks commented Nov 19, 2025 •

edited

Loading

codecov bot commented Nov 21, 2025 •

edited

Loading