Problem/Motivation

There is a bug in the log importer.

Proposed resolution

Fix bug.
Add tests.

See parent issue for beta evalution.

Remaining tasks

Review
Commit

User interface changes

None.

API changes

None.

CommentFileSizeAuthor
#2 fix_config_importer-2565139-2.patch1.83 KBjoelpittet

Comments

joelpittet created an issue. See original summary.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
StatusFileSize
new fix_config_importer-2565139-2.patch1.83 KB

@alexpott found this bug while working on #2564353: Remove !placeholder usage from SafeMarkup::format()

I'm filing this on his behalf in hopes to help add tests and separate from an unrelated bug fix.

Status: Needs review » Needs work

The last submitted patch, 2: fix_config_importer-2565139-2.patch, failed testing.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Needs work » Closed (duplicate)

Closing - this is fixed and tested in #2564353: Remove !placeholder usage from SafeMarkup::format()