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

Fix email autofill in subscription popup #1394

Merged
merged 1 commit into from
Jul 1, 2022

Conversation

akellbl4
Copy link
Collaborator

@akellbl4 akellbl4 commented Jun 30, 2022

Depends on #1396

Fixes the first part of #1393
Persist email in local storage for autofill in subscription popup.

@akellbl4 akellbl4 requested review from paskal and Mavrin June 30, 2022 02:29
@akellbl4 akellbl4 force-pushed the autofill-email-for-subscription branch from 06dddb9 to 4b47343 Compare June 30, 2022 02:30
@akellbl4 akellbl4 marked this pull request as ready for review June 30, 2022 02:30
@akellbl4 akellbl4 requested a review from umputun as a code owner June 30, 2022 02:30
@codecov
Copy link

codecov bot commented Jun 30, 2022

Codecov Report

Merging #1394 (533c924) into master (1f1adba) will increase coverage by 0.14%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##           master    #1394      +/-   ##
==========================================
+ Coverage   57.74%   57.88%   +0.14%     
==========================================
  Files         130      130              
  Lines        2873     2885      +12     
  Branches      742      741       -1     
==========================================
+ Hits         1659     1670      +11     
- Misses       1210     1211       +1     
  Partials        4        4              
Impacted Files Coverage Δ
frontend/app/store/user/actions.ts 85.18% <66.66%> (+0.18%) ⬆️
frontend/app/components/auth/auth.utils.ts 86.95% <87.50%> (+0.28%) ⬆️
frontend/app/components/auth/auth.tsx 81.25% <100.00%> (+0.39%) ⬆️
...ribe-by-email/comment-form__subscribe-by-email.tsx 90.59% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1f1adba...533c924. Read the comment docs.

@akellbl4 akellbl4 added the bug label Jun 30, 2022
paskal
paskal previously approved these changes Jun 30, 2022
Copy link
Collaborator

@paskal paskal left a comment

Choose a reason for hiding this comment

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

It works, LGTM.

@@ -31,6 +31,7 @@ import styles from './auth.module.css';
export function Auth() {
const intl = useIntl();
const telegramParamsRef = useRef<null | { bot: string; token: string }>(null);
const emailRef = useRef('');
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

save email between rerenders

Mavrin
Mavrin previously approved these changes Jul 1, 2022
@akellbl4 akellbl4 dismissed stale reviews from Mavrin and paskal via 533c924 July 1, 2022 17:24
@akellbl4 akellbl4 force-pushed the autofill-email-for-subscription branch from 4b47343 to 533c924 Compare July 1, 2022 17:24
@github-actions
Copy link

github-actions bot commented Jul 1, 2022

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
public/embed.mjs 2.02 KB (0%) 41 ms (0%) 110 ms (+21.67% 🔺) 150 ms
public/remark.mjs 70.82 KB (+0.03% 🔺) 1.5 s (+0.03% 🔺) 208 ms (+28.2% 🔺) 1.7 s
public/remark.css 8.2 KB (-0.02% 🔽) 165 ms (-0.02% 🔽) 0 ms (+100% 🔺) 165 ms
public/last-comments.mjs 35.62 KB (+0.01% 🔺) 713 ms (+0.01% 🔺) 268 ms (+122.73% 🔺) 980 ms
public/last-comments.css 3.79 KB (+0.03% 🔺) 76 ms (+0.03% 🔺) 0 ms (+100% 🔺) 76 ms
public/deleteme.mjs 10.01 KB (0%) 201 ms (0%) 54 ms (-42.01% 🔽) 254 ms
public/counter.mjs 749 B (0%) 15 ms (0%) 77 ms (-6.65% 🔽) 91 ms

@umputun umputun merged commit 24e9404 into master Jul 1, 2022
@umputun umputun deleted the autofill-email-for-subscription branch July 1, 2022 17:33
@paskal paskal added this to the v1.10.2 milestone Jul 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants