Skip to content

fix: keep content if no main found in html#85

Merged
auniverseaway merged 2 commits intomainfrom
nomain
Sep 3, 2025
Merged

fix: keep content if no main found in html#85
auniverseaway merged 2 commits intomainfrom
nomain

Conversation

@kptdobe
Copy link
Contributor

@kptdobe kptdobe commented Sep 1, 2025

This reverts the "no main leads to empty doc" behavior. But doc2aem is still moving reshaping the document and moves everything inside main...

@github-actions
Copy link

github-actions bot commented Sep 1, 2025

LCOV of commit 5fbafa5 during Install, lint, and test #189

Summary coverage rate:
  lines......: 96.4% (1600 of 1659 lines)
  functions..: 96.6% (57 of 59 functions)
  branches...: no data found

Files changed coverage rate:
                  |Lines       |Functions  |Branches    
  Filename        |Rate     Num|Rate    Num|Rate     Num
  ======================================================
  src/collab.js   | 3.7%    430| 0.0%    16|    -      0

@kptdobe kptdobe changed the title fix: keep no main fix: keep content if no main found in html Sep 1, 2025
}
});
const main = tree.children.find((child) => child.tagName === 'main');
console.log('tree', tree);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Guessing these logs are intentional?

@auniverseaway auniverseaway merged commit 9fd25ac into main Sep 3, 2025
3 checks passed
@auniverseaway auniverseaway deleted the nomain branch September 3, 2025 17:55
adobe-bot pushed a commit that referenced this pull request Dec 8, 2025
# 1.0.0 (2025-12-08)

### Bug Fixes

* add check for shared secret ([#108](#108)) ([a79731f](a79731f))
* add span to known HTML tags ([#101](#101)) ([fbf016d](fbf016d))
* do not empty document if error ([#90](#90)) ([a406add](a406add))
* encode HTML brackets ([#100](#100)) ([6c5b5fd](6c5b5fd))
* encode HTML brackets ([#74](#74)) ([015163e](015163e))
* keep content if no main found in html ([#85](#85)) ([9fd25ac](9fd25ac))
* null doc crashes aem2doc ([#84](#84)) ([6197114](6197114))
* ommit stacktraces in error map for non-dev ([#110](#110)) ([1f896ac](1f896ac))
* remove html comments ([#68](#68)) ([cf05aae](cf05aae))

### Features

* disable stack traces on production ([#109](#109)) ([79c40bd](79c40bd))
* Prevent implicit document creation via If-Match header ([#105](#105)) ([b33ae65](b33ae65))
adobe-bot pushed a commit that referenced this pull request Dec 8, 2025
# 1.0.0 (2025-12-08)

### Bug Fixes

* add check for shared secret ([#108](#108)) ([a79731f](a79731f))
* add span to known HTML tags ([#101](#101)) ([fbf016d](fbf016d))
* do not empty document if error ([#90](#90)) ([a406add](a406add))
* encode HTML brackets ([#100](#100)) ([6c5b5fd](6c5b5fd))
* encode HTML brackets ([#74](#74)) ([015163e](015163e))
* keep content if no main found in html ([#85](#85)) ([9fd25ac](9fd25ac))
* null doc crashes aem2doc ([#84](#84)) ([6197114](6197114))
* ommit stacktraces in error map for non-dev ([#110](#110)) ([1f896ac](1f896ac))
* remove html comments ([#68](#68)) ([cf05aae](cf05aae))
* trigger release ([ca061d4](ca061d4))

### Features

* disable stack traces on production ([#109](#109)) ([79c40bd](79c40bd))
* Prevent implicit document creation via If-Match header ([#105](#105)) ([b33ae65](b33ae65))
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.

2 participants