Skip to content

lasuite-docs: fix mail notifications #432753

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

Merged
merged 1 commit into from
Aug 22, 2025
Merged

Conversation

Ma27
Copy link
Member

@Ma27 Ma27 commented Aug 11, 2025

The build-process doesn't install the mjml-based mail templates which results in the following log-lines when trying to invite somebody by email to collaborate on a document:

Aug 11 08:25:08 lasuite gunicorn[345]:   File "/nix/store/4h2z6psmq0nizy39psyf2lil0cqiwl42-python3.13-django-5.2.4/lib/python3.13/site-packages/django/template/loader.py", line 19, in get_template
Aug 11 08:25:08 lasuite gunicorn[345]:     raise TemplateDoesNotExist(template_name, chain=chain)
Aug 11 08:25:08 lasuite gunicorn[345]: django.template.exceptions.TemplateDoesNotExist: mail/html/template.html

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@Ma27 Ma27 requested a review from soyouzpanda August 11, 2025 09:10
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Aug 11, 2025
@Ma27 Ma27 force-pushed the lasuite-docs-mail branch from cfbf323 to 86f3529 Compare August 13, 2025 05:46
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. and removed 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Aug 13, 2025
@Ma27 Ma27 requested a review from soyouzpanda August 13, 2025 05:55
@soyouzpanda
Copy link
Contributor

Also, the commit adding mjml-cli must be separated from the commit fixing lasuite-docs.

The build-process doesn't install the mjml-based mail templates which
results in the following log-lines when trying to invite somebody by
email to collaborate on a document:

    Aug 11 08:25:08 lasuite gunicorn[345]:   File "/nix/store/4h2z6psmq0nizy39psyf2lil0cqiwl42-python3.13-django-5.2.4/lib/python3.13/site-packages/django/template/loader.py", line 19, in get_template
    Aug 11 08:25:08 lasuite gunicorn[345]:     raise TemplateDoesNotExist(template_name, chain=chain)
    Aug 11 08:25:08 lasuite gunicorn[345]: django.template.exceptions.TemplateDoesNotExist: mail/html/template.html
@Ma27 Ma27 force-pushed the lasuite-docs-mail branch from 86f3529 to 5597a4f Compare August 14, 2025 09:36
@Ma27
Copy link
Member Author

Ma27 commented Aug 14, 2025

Welp, I forgot to remove this, this was a previous attempt to package mjml-cli independently, but gave up.

@Ma27 Ma27 requested a review from soyouzpanda August 14, 2025 09:37
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. and removed 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. labels Aug 14, 2025
@soyouzpanda
Copy link
Contributor

Welp, I forgot to remove this, this was a previous attempt to package mjml-cli independently, but gave up.

ahah, that's what I thought :)
This looks good to me.

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels Aug 15, 2025
@Ma27 Ma27 merged commit 1233891 into NixOS:master Aug 22, 2025
31 of 32 checks passed
@Ma27 Ma27 deleted the lasuite-docs-mail branch August 22, 2025 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants