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.

Sign up for GitHub

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

Jump to bottom

Use email api #252

Closed
wants to merge 4 commits into from
Closed

Use email api #252

wants to merge 4 commits into from

Commits on Feb 25, 2020

  1. Extract email settings in dedicated file

    @gbonnefille
    gbonnefille committed Feb 25, 2020
    Configuration menu
    Copy the full SHA
    f09f5a8 View commit details
    Browse the repository at this point in the history

  2. Use official API for setting email 

    Using the official API will reduce code in this role.
Furthermore, using an official API remove the need to deploy
dedicated scripts. And such deploy can be annoying depending
on the overall deployment (native, docker, docker-compose, kubernetes).
So, using official API will offer the ability to reuse easier this role
in much more context.
    @gbonnefille
    gbonnefille committed Feb 25, 2020
    Configuration menu
    Copy the full SHA
    add7930 View commit details
    Browse the repository at this point in the history

  3. QA: add spaces before and after variable

    @gbonnefille
    gbonnefille authored Feb 25, 2020
    Configuration menu
    Copy the full SHA
    a934507 View commit details
    Browse the repository at this point in the history

Commits on Aug 10, 2020

  1. Merge branch 'master' into use-email-api

    @gbonnefille
    gbonnefille authored Aug 10, 2020
    Configuration menu
    Copy the full SHA
    8b739ee View commit details
    Browse the repository at this point in the history