Skip to content

add docs on missing or empty variables in submit.yml.erb - #1426

Open
johrstrom wants to merge 3 commits into
latestfrom
debug-missing-submit
Open

johrstrom wants to merge 3 commits into
latestfrom
debug-missing-submit

Conversation

@johrstrom

@johrstrom johrstrom commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

https://osc.github.io/ood-documentation-test/debug-missing-submit/how-tos/debug/debug-interactive-apps.html#empty-variables-in-submit-yml-erb

Fixes #241 though that issue has changed over time.

From the original issue you'd get an error message about undefined or missing method X, this no longer occurs and whatever is missing is just nil without any error at all. So users would end up submitting empty strings or zeros where this missing variable should be.

This also just adds a ToC to this page.

@moffatsadeghi moffatsadeghi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a few questions / feedback from a non-technical perspecitive:

Could we add a sentence about where administrators should look for the error output after adding this?

Could we also like soften the last paragraph to say that a missing variable may be due to a misspelling or an option missing from the form section just so we aren't implying those are the only two possible causes unless those are the only two possible causes.

@johrstrom

Copy link
Copy Markdown
Contributor Author

Could we add a sentence about where administrators should look for the error output after adding this?

It's tricky but I can try with an example. It's tricky because there may be no error.

Could we also like soften the last paragraph to say that a missing variable may be due to a misspelling or an option missing from the form section just so we aren't implying those are the only two possible causes unless those are the only two possible causes.

Yea I'll use the phrase 'most likely' and re-arrange it a little bit.

@johrstrom

Copy link
Copy Markdown
Contributor Author

Updates made and I've updated the original comment's link to point directly to the page and section that's being updated.

This branch has not been deployed

No deployments
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.

Document common pitfall 'undefined method' in a bc app

2 participants