Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix section parameter to allow proper hierarchy #12319

Merged
merged 3 commits into from
Dec 21, 2023

Conversation

obergmartin
Copy link
Contributor

Reference issue

Discussion on Discourse about Report.add_html().
HTML chunks were added to the main level, not within the supplied section.

The section value passed to self._add_or_replace was hard coded as None and didn't use the supplied section argument.

Copy link

welcome bot commented Dec 20, 2023

Hello! 👋 Thanks for opening your first pull request here! ❤️ We will try to get back to you soon. 🚴

@larsoner
Copy link
Member

@obergmartin could you add a doc/changes/devel/12319.bugfix.rst file that mentions this fix along with your name using the :newcontrib: reST role (see here) and your name in doc/changes/names.inc? Then we can get you proper credit on merge, thanks!

@obergmartin
Copy link
Contributor Author

@larsoner glad to be a contributor, looking forward to doing more!

Copy link
Member

@drammock drammock left a comment

Choose a reason for hiding this comment

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

tested locally and it works for me. Thanks @obergmartin

@drammock drammock enabled auto-merge (squash) December 21, 2023 20:41
@drammock drammock merged commit ca7fe26 into mne-tools:main Dec 21, 2023
28 checks passed
Copy link

welcome bot commented Dec 21, 2023

🎉 Congrats on merging your first pull request! 🥳 Looking forward to seeing more from you in the future! 💪

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.

3 participants