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 24
24
25
25
removeUnwanted =
26
26
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 [
28
40
"_module"
29
41
"_freeformOptions"
30
42
"warnings"
You can’t perform that action at this time.
0 commit comments