Project

Profile

Help

Refactor #4382

closed

Refactor ArtifactFileField

Added by daviddavis almost 4 years ago. Updated almost 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Sprint:
Sprint 49
Quarter:

Description

Some problems with the ArtifactFileField pre_save:

- It's hard to read and understand what it's doing
- It's complex
- It calls super twice which seems strange

I spent some time trying to refactor it while working on S3 support but ultimately gave up. We should refactor this file and perhaps look at other ways to save Artifact files.

Actions #1

Updated by dkliban@redhat.com almost 4 years ago

  • Groomed changed from No to Yes
  • Sprint Candidate changed from No to Yes
  • Sprint set to Sprint 48
Actions #2

Updated by rchan almost 4 years ago

  • Sprint changed from Sprint 48 to Sprint 49
Actions #3

Updated by ttereshc almost 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ttereshc

Added by ttereshc almost 4 years ago

Revision 5bdcb336

Refactor ArtifactFileField

closes #4382 https://pulp.plan.io/issues/4382

Added by ttereshc almost 4 years ago

Revision 5bdcb336

Refactor ArtifactFileField

closes #4382 https://pulp.plan.io/issues/4382

Actions #4

Updated by ttereshc almost 4 years ago

  • Status changed from ASSIGNED to POST
Actions #5

Updated by ttereshc almost 4 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #6

Updated by daviddavis over 3 years ago

  • Sprint/Milestone set to 3.0.0
Actions #7

Updated by bmbouter over 3 years ago

  • Tags deleted (Pulp 3, Pulp 3 RC Blocker)
Actions #8

Updated by bmbouter almost 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF