Skip to content

feat: make Android tutor.yaml customizable via Tutor config (#48)#52

Merged
Abdul-Muqadim-Arbisoft merged 2 commits into
releasefrom
muqadim/customizable-tutor-yaml
May 5, 2026
Merged

feat: make Android tutor.yaml customizable via Tutor config (#48)#52
Abdul-Muqadim-Arbisoft merged 2 commits into
releasefrom
muqadim/customizable-tutor-yaml

Conversation

@Abdul-Muqadim-Arbisoft
Copy link
Copy Markdown
Collaborator

Expose every customizable field of the Android app's tutor.yaml as ANDROID_* Tutor settings (Firebase, Segment, Braze, Google, Microsoft, Facebook, Branch, agreement URLs, discovery/program groups, feature flags, etc.) so users can tweak them with tutor config save --set instead of forking the template. Defaults match the previously hardcoded values, so behavior is unchanged for existing users.

Add a {{ patch("android-tutor-yaml") }} hook at the end of the rendered file so other Tutor plugins can append YAML or wholesale replace a group; document the SnakeYAML last-value-wins behavior in the README.

Closes #48

Expose every customizable field of the Android app's tutor.yaml as
ANDROID_* Tutor settings (Firebase, Segment, Braze, Google, Microsoft,
Facebook, Branch, agreement URLs, discovery/program groups, feature
flags, etc.) so users can tweak them with `tutor config save --set`
instead of forking the template. Defaults match the previously
hardcoded values, so behavior is unchanged for existing users.

Add a `{{ patch("android-tutor-yaml") }}` hook at the end of the
rendered file so other Tutor plugins can append YAML or wholesale
replace a group; document the SnakeYAML last-value-wins behavior in
the README.

Closes #48
`twine check` was failing because the underline below the new
"Customising the app configuration (``tutor.yaml``)" section title was
49 characters while the title itself is 50, which docutils rejects as
"Title underline too short" and would break PyPI long_description
rendering.
Copy link
Copy Markdown
Member

@CodeWithEmad CodeWithEmad left a comment

Choose a reason for hiding this comment

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

LGTM. Makes the app more configurable. Does Edly have any plans to add a simple theming feature for this plugin?

@Abdul-Muqadim-Arbisoft
Copy link
Copy Markdown
Collaborator Author

LGTM. Makes the app more configurable. Does Edly have any plans to add a simple theming feature for this plugin?

Thanks for the review, @CodeWithEmad! 🙌

Yes, a simple theming feature is definitely something Edly is interested in, we agree it would be a great addition to the plugin. Unfortunately, due to current capacity constraints on our side, we're not able to pick it up ourselves in the near term and are looking for external contributors who could help drive it forward.

If you have the bandwidth and interest, we'd really appreciate you taking a stab at it, happy to collaborate on it with you. Let us know!

@CodeWithEmad
Copy link
Copy Markdown
Member

Cool! We can close this one for now.
We'll create a new issue for that and either me or someone from the community can take care of the simple themeing feature.

@Abdul-Muqadim-Arbisoft Abdul-Muqadim-Arbisoft merged commit d8f4f88 into release May 5, 2026
2 checks passed
@Abdul-Muqadim-Arbisoft Abdul-Muqadim-Arbisoft deleted the muqadim/customizable-tutor-yaml branch May 5, 2026 15:22
@github-project-automation github-project-automation Bot moved this from In review to Done in Tutor project management May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

There should be a way to set different values inside tutor.yml or override the whole file

4 participants