Skip to content
This repository has been archived by the owner on Dec 10, 2020. It is now read-only.

(add) register button (upd) fix display bugs (upd) mvvm navigation from login page #12

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

mathieumasset
Copy link

No description provided.

var url = "https://www.en-marche.fr/adhesion";

#if DEBUG
url = "https://staging.en-marche.fr/adhesion";
Copy link
Member

Choose a reason for hiding this comment

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

@mathieumasset est-ce que ça serait compliqué de passer par une var d'env pour stocker le host et supprimer le IF DEBUG?

var url = env.SERVER_HOST + "/adhesion";


Settings.AppSettings.AddOrUpdateValue("client_id", "33f63935-0793-41b9-89a8-1d9bb74e5fe5");
Settings.AppSettings.AddOrUpdateValue("client_secret", "_X5lVzYtkoVBotQM1pmty/8el3UGrRgIpfzqXeUL0jbY=");
Settings.AppSettings.AddOrUpdateValue("WS_BASE_URL", "https://en-marche.fr");
#else
Settings.AppSettings.AddOrUpdateValue("client_id", "33f63935-0793-41b9-89a8-1d9bb74e5fe5");
Copy link
Member

Choose a reason for hiding this comment

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

il faut qu'on arrive à supprimer toutes les infos sensibles du code et faire passer le max sur var d'env

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants