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

First version for certificate generation #77

Draft
wants to merge 25 commits into
base: main
Choose a base branch
from
Draft

Conversation

hgingras
Copy link

No description provided.

@ccoulombe ccoulombe assigned ccoulombe and unassigned ccoulombe Feb 27, 2024
@ccoulombe ccoulombe self-requested a review February 27, 2024 13:03
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remplacer certificate et ceritificate par attestation.

create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
@ccoulombe
Copy link
Collaborator

Il faudra également ajouter l'envoi du certificat par courriel, avec l'option de test à soi-même.

@hgingras
Copy link
Author

hgingras commented Mar 1, 2024

Il faudra également ajouter l'envoi du certificat par courriel, avec l'option de test à soi-même.

@hgingras hgingras closed this Mar 1, 2024
@hgingras hgingras reopened this Mar 1, 2024
Copy link
Collaborator

@ccoulombe ccoulombe left a comment

Choose a reason for hiding this comment

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

Il va manquer également l'envoi par courriel des certificats

create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Show resolved Hide resolved
…ngement pour utiliser le même nom de fichier qu'en FR et ajout de la constante en haut du document. Corriger l'emplacement du logo pour le certificat en français. Ligne 36, ajout d’un paramètre pour contrôler où écrire les certificats. Ligner 35, ajout de —event_id en paramètre requis. Ligne 190, 'end’. Ligne 51, ajout de --event_id pour le usage. Ligne 377, ajout du garde pour protéger le main.
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
create_certificate.py Outdated Show resolved Hide resolved
helene and others added 5 commits March 13, 2024 15:01
…age. Ligne 92-94 et 199-201 et 316-318; enlevé validation langue.
…mplate et modification de la variable de test pour participants fictifs.
…urriel fonctionnel. Argparse copié dans la bonne section. Ajout des templates pour les courriels en français et en englais.
@ccoulombe ccoulombe marked this pull request as draft April 2, 2024 18:10
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remplacer certification par attestation de participation.

Copy link
Author

Choose a reason for hiding this comment

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

Okay

Copy link
Collaborator

Choose a reason for hiding this comment

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

Corriger le titre Ceritificate -> Attestation

Copy link
Author

Choose a reason for hiding this comment

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

Okay


# Create email:
if args.send_atnd or args.send_self:
send_email(eb_event, attended_guest_test, args.email_tplt_dir, args.send_self, args.number_to_send, args.language, args.gmail_user, args.gmail_password, args.self_email, attach_certificate=True)
Copy link
Collaborator

Choose a reason for hiding this comment

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

attended_guest_test n'existe pas

Copy link
Author

Choose a reason for hiding this comment

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

J'ai oublié de changer le nom de la variable. J'ai fait la correction.

@mboisson
Copy link
Member

Il faudrait déplacer les templates et les adresses hardcodées de Calcul Québec dans un dépôt privé, plutôt que dans ce dépôt.

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