Problem/Motivation

While working on #2876085: Before upgrading, audit for potential ID conflicts, it was discovered that that the migration labels for public and private files are the same name. Let's fix that.

Proposed resolution

Take the chunks out of the parent issue and move them into a new patch here.

Remaining tasks

  • Code it.
  • Review it.

User interface changes

n/a

API changes

None
.

Data model changes

None.

CommentFileSizeAuthor
#3 2916642-3.patch1.35 KBquietone
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heddn created an issue. See original summary.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Acro Commerce commented
Issue summary: View changes

Let's update the label:

d6_file.yml => Public Files
d7_file.yml => Public Files
d7_file_private.yml => Private Files

quietone’s picture

quietone CreditAttribution: quietone at Acro Commerce commented
FileSize
2916642-3.patch1.35 KB
heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Acro Commerce commented
Status: Active » Reviewed & tested by the community

Label is now updated per issue summary. I'm assuming this will come back green. Straight to rtbc.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented

Crediting @heddn because for the reviews and outline the direction of the patch.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Priority: Major » Normal
Status: Reviewed & tested by the community » Fixed

Committed and pushed 969cafbf57 to 8.5.x and 0674bc6d6a to 8.4.x. Thanks!

  • alexpott committed 969cafb on 8.5.x 
    Issue #2916642 by quietone, heddn: Private and Public file migrations...

  • alexpott committed 0674bc6 on 8.4.x 
    Issue #2916642 by quietone, heddn: Private and Public file migrations...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.