File tree Expand file tree Collapse file tree 1 file changed +13
-1
lines changed Expand file tree Collapse file tree 1 file changed +13
-1
lines changed Original file line number Diff line number Diff line change 2424
2525 removeUnwanted =
2626 attrs :
27- builtins . removeAttrs attrs [
27+ # FIXME: We incorrectly remove _freeformOptions here.
28+ #
29+ # However we can't fix the bug because we derive page names from attrnames;
30+ # the correct behaviour would be to ignore attrnames and use option locs.
31+ #
32+ # As a workaround, merge the freeform options at the top of these attrs,
33+ # however this can run into name conflicts 😢
34+ #
35+ # We should move this workaround to where we decide the page name and
36+ # whether to nest into a sub-page, so that we can keep the original
37+ # _freeformOptions attr as intended.
38+ attrs . _freeformOptions or { }
39+ // builtins . removeAttrs attrs [
2840 "_module"
2941 "_freeformOptions"
3042 "warnings"
You can’t perform that action at this time.
0 commit comments