It would be useful to import only content that has changed instead of deleting and re-importing all content for a static site. We should introduce an "import-partial" command, or augment how "import" works, to load and re-save or delete existing entities when ran.

CommentFileSizeAuthor
#12 interdiff-3040312-11-12.txt3.67 KBsamuel.mortenson
#12 3040312-12.patch33.36 KBsamuel.mortenson
#11 interdiff-3040312-10-11.txt7.46 KBsamuel.mortenson
#11 3040312-11.patch33.01 KBsamuel.mortenson
#10 interdiff-3040312-9-10.txt7.63 KBsamuel.mortenson
#10 3040312-10.patch32.01 KBsamuel.mortenson
#9 interdiff-3040312-9.patch-7-9.txt2.04 KBsamuel.mortenson
#9 3040312-9.patch25.78 KBsamuel.mortenson
#7 interdiff-3040312-6-7.txt9.01 KBsamuel.mortenson
#7 3040312-7.patch25.11 KBsamuel.mortenson
#6 interdiff-3040312-5-6.txt2.53 KBsamuel.mortenson
#6 3040312-6.patch23.92 KBsamuel.mortenson
#5 interdiff-3040312-3-5.txt1.99 KBsamuel.mortenson
#5 3040312-5.patch23.29 KBsamuel.mortenson
#3 3040312-3.patch21.57 KBsamuel.mortenson

Comments

samuel.mortenson created an issue. See original summary.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented

I'm working on this now.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new 3040312-3.patch21.57 KB

Here's my current progress on this. I was thinking about making this the default architecture for the "tome:import" command but it feels like there could still be edge cases and I don't want to break existing sites. Maybe that's a 8.x-2.x thing. :-)

Need to think about what I've done more and write tests.

Status: Needs review » Needs work

The last submitted patch, 3: 3040312-3.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented
Status: Needs work » Needs review
StatusFileSize
new 3040312-5.patch23.29 KB
new interdiff-3040312-3-5.txt1.99 KB
samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented
StatusFileSize
new 3040312-6.patch23.92 KB
new interdiff-3040312-5-6.txt2.53 KB

Some weirdness with array references.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented
StatusFileSize
new 3040312-7.patch25.11 KB
new interdiff-3040312-6-7.txt9.01 KB

More refactors, tweaks, etc.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented

Note to self - hashes.json should be completely rebuilt whenever a full export is performed.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented
StatusFileSize
new 3040312-9.patch25.78 KB
new interdiff-3040312-9.patch-7-9.txt2.04 KB

Just needs tests now, and manual testing of multilingual.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented
StatusFileSize
new 3040312-10.patch32.01 KB
new interdiff-3040312-9-10.txt7.63 KB

Added test coverage, fixed many bugs.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented
StatusFileSize
new 3040312-11.patch33.01 KB
new interdiff-3040312-10-11.txt7.46 KB

Fixed more bugs! Things are looking really good now.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented
StatusFileSize
new 3040312-12.patch33.36 KB
new interdiff-3040312-11-12.txt3.67 KB

  • samuel.mortenson authored d047be5 on 8.x-1.x 
    Issue #3040312 by samuel.mortenson: Create a "tome:import-partial"...
samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented
Status: Needs review » Fixed

Woot!

Status: Fixed » Closed (fixed)

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