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

feat: Add authentification dialog #2469

Merged
merged 5 commits into from
Jul 5, 2023
Merged

Conversation

cballevre
Copy link
Contributor

@cballevre cballevre commented Jun 30, 2023

@cballevre cballevre force-pushed the feat/add-authentification-dialog branch from c0e453b to b408137 Compare June 30, 2023 09:45
@cballevre cballevre force-pushed the feat/add-authentification-dialog branch 2 times, most recently from b6f2021 to 5c8fe41 Compare June 30, 2023 13:42
@bundlemon
Copy link

bundlemon bot commented Jun 30, 2023

BundleMon

Unchanged files (3)
Status Path Size Limits
dist/cozy-ui.min.css
19.72KB +10%
transpiled/react/stylesheet.css
19.13KB +10%
dist/cozy-ui.utils.min.css
10.5KB +10%

No change in files bundle size

Groups updated (1)
Status Path Size Limits
transpiled/react/**
589.6KB (+4.14KB +0.71%) -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@cballevre
Copy link
Contributor Author

@JF-Cozy Would it make sense to create a folder to group all field types such as TextField and PasswordField? Or you'd see it in a different place

@JF-Cozy
Copy link
Collaborator

JF-Cozy commented Jul 3, 2023

On ne groupe aucun composant, à part quelques exceptions, je dirai donc non

react/CozyDialogs/PermissionDialog.jsx Show resolved Hide resolved
react/CozyDialogs/styles.styl Outdated Show resolved Hide resolved
react/CozyDialogs/styles.styl Outdated Show resolved Hide resolved
react/CozyDialogs/Readme.md Outdated Show resolved Hide resolved
react/CozyDialogs/Readme.md Outdated Show resolved Hide resolved
react/CozyDialogs/specific/AuthentificationDialog.jsx Outdated Show resolved Hide resolved
react/Icons/EyeClosed.jsx Show resolved Hide resolved
react/CozyDialogs/specific/PasswordField.jsx Outdated Show resolved Hide resolved
react/CozyDialogs/specific/EyeAdornment.jsx Outdated Show resolved Hide resolved
react/CozyDialogs/specific/AuthentificationDialog.jsx Outdated Show resolved Hide resolved
@cballevre cballevre force-pushed the feat/add-authentification-dialog branch from 5c8fe41 to 49f85a7 Compare July 3, 2023 12:07
@cballevre cballevre requested a review from JF-Cozy July 3, 2023 12:13
@cballevre cballevre force-pushed the feat/add-authentification-dialog branch 3 times, most recently from 288172d to 8c2da65 Compare July 4, 2023 13:32
@cballevre
Copy link
Contributor Author

@JF-Cozy j'ai fait toutes les modifications qu'on avait discutées. Après discussion avec Joël, le fond de l'icône restera blanc pour le moment j'ai donc opté pour Circle qui simplifie le code.

Est-ce qu'il y aurait moyen d'avoir une nouvelle review ? pls 🙏

react/CozyDialogs/PermissionDialog.jsx Outdated Show resolved Hide resolved
react/CozyDialogs/PermissionDialog.jsx Outdated Show resolved Hide resolved
react/CozyDialogs/styles.styl Outdated Show resolved Hide resolved
react/PasswordField/index.jsx Outdated Show resolved Hide resolved
@cballevre cballevre force-pushed the feat/add-authentification-dialog branch 2 times, most recently from 2a56764 to 742d808 Compare July 4, 2023 16:37
@cballevre cballevre requested a review from JF-Cozy July 4, 2023 16:38
@cballevre cballevre force-pushed the feat/add-authentification-dialog branch from 742d808 to 722e17f Compare July 5, 2023 06:49
react/CozyDialogs/PermissionDialog.jsx Show resolved Hide resolved
react/CozyDialogs/PermissionDialog.jsx Outdated Show resolved Hide resolved
react/CozyDialogs/Readme.md Outdated Show resolved Hide resolved
react/MuiCozyTheme/makeOverrides.js Show resolved Hide resolved
react/MuiCozyTheme/makeOverrides.js Outdated Show resolved Hide resolved
react/PasswordField/index.jsx Outdated Show resolved Hide resolved
@cballevre cballevre force-pushed the feat/add-authentification-dialog branch from 722e17f to 54a2fe9 Compare July 5, 2023 09:19
@cballevre cballevre requested a review from JF-Cozy July 5, 2023 09:19
@cballevre cballevre force-pushed the feat/add-authentification-dialog branch from 54a2fe9 to 807a26b Compare July 5, 2023 13:28
@cballevre cballevre force-pushed the feat/add-authentification-dialog branch from 807a26b to cde78f9 Compare July 5, 2023 14:05
@cballevre cballevre merged commit 52ee385 into master Jul 5, 2023
@cballevre cballevre deleted the feat/add-authentification-dialog branch July 5, 2023 14:44
@cozy-bot
Copy link

cozy-bot commented Jul 5, 2023

🎉 This PR is included in version 88.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants