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

Provide the ability to upload patches #78 #104

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

ChiuchiuSorin
Copy link
Collaborator

Created new endpoints for the package upload operation and refactored the healthCheckJob code to be usable by multiple job.

The import package page:
importPage

The import package result:
packageUpload

The modification for file section:
filesSection

Verified

This commit was signed with the committer’s verified signature.
ChiuchiuSorin Sorin Chiuchiu
* created upload job request and status
* renamed HealthCheckResult to CustomJobResult and HealthCheckResultLevel to CustomJobResultLevel
* created a new rest endpoint for package upload with the role of instantiating a job with the given reference and time stamp
* added a new endpoint to ServerInfo interface for retrieving the XWiki installation path
* implemented the logic of package archive upload which includes: original file search and backup creation, batch file save (file replacement), batch backup restore in case of an error
* created the ImportPackage page for user interface
* created the hidden PackagesStore for archive storage
* created liveData table for saving a package and starting the upload process
* created liveData table (with PackagesJSON document) for showing the already uploaded packages
* created a template for executing and showing the progress of a package upload
* code refactoring
* added transalations

Verified

This commit was signed with the committer’s verified signature.
ChiuchiuSorin Sorin Chiuchiu
# Conflicts:
#	application-admintools-ui/pom.xml

Verified

This commit was signed with the committer’s verified signature.
ChiuchiuSorin Sorin Chiuchiu
* remove packagesStore document
@ChiuchiuSorin ChiuchiuSorin self-assigned this Oct 10, 2024
@ChiuchiuSorin ChiuchiuSorin linked an issue Oct 10, 2024 that may be closed by this pull request
@ChiuchiuSorin
Copy link
Collaborator Author

There are no tests added yet, I will add them in a future commit.

Verified

This commit was signed with the committer’s verified signature.
ChiuchiuSorin Sorin Chiuchiu

Verified

This commit was signed with the committer’s verified signature.
ChiuchiuSorin Sorin Chiuchiu
* created upload job request and status
* renamed HealthCheckResult to CustomJobResult and HealthCheckResultLevel to CustomJobResultLevel
* code refactoring
@ChiuchiuSorin
Copy link
Collaborator Author

This PR depends on modifications made in #107.

Verified

This commit was signed with the committer’s verified signature.
ChiuchiuSorin Sorin Chiuchiu
* removed healthCheckJob.xml

Verified

This commit was signed with the committer’s verified signature.
ChiuchiuSorin Sorin Chiuchiu
* refactored CustomJobResult to JobResult
* refactored CustomJobResultLevel to JobResultLevel

Verified

This commit was signed with the committer’s verified signature.
ChiuchiuSorin Sorin Chiuchiu
* refactored CustomJobResult to JobResult
* refactored CustomJobResultLevel to JobResultLevel

Verified

This commit was signed with the committer’s verified signature.
ChiuchiuSorin Sorin Chiuchiu
# Conflicts:
#	application-admintools-ui/src/main/resources/AdminTools/Code/Translations.xml

Verified

This commit was signed with the committer’s verified signature.
ChiuchiuSorin Sorin Chiuchiu
# Conflicts:
#	application-admintools-ui/src/main/resources/AdminTools/Code/Translations.xml

Verified

This commit was signed with the committer’s verified signature.
ChiuchiuSorin Sorin Chiuchiu
* added unit tests for UploadJob
* refactored UploadJob in UploadJob and UploadJobFileProcessor

Verified

This commit was signed with the committer’s verified signature.
ChiuchiuSorin Sorin Chiuchiu
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.

Provide the ability to upload patches
1 participant