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

add dclogin scheme to qr code scan #2366

Merged
merged 5 commits into from
Sep 29, 2022
Merged

add dclogin scheme to qr code scan #2366

merged 5 commits into from
Sep 29, 2022

Conversation

Simon-Laux
Copy link
Member

@Simon-Laux Simon-Laux commented Aug 19, 2022

see deltachat/interface#48 for spec.

needs a core with deltachat/deltachat-core-rust#3541 merged.

I started with android implementation mainly to add the strings that they are swiftly discussed and then translated.

Tasks

  • scan as first account
  • scan as second account
  • strings
  • register DC as dclogin: scheme handler systemwide

those intents are beyond me, I can't figure out how to register DC as dclogin: scheme handler systemwide. sure it needs to go into AndroidManifest.xml but then the java code besides that is to hard for me. (I strongly dislike coding java) so it's better done by someone with real android experience, maybe together with #2365, as those two tasks are very similar.

so feel free to just push java stuff to this branch, I'm tired and annoyed of writing java.

@Simon-Laux Simon-Laux added the core Networking, Messages, IMAP, etc. that is handled by the core label Aug 19, 2022
@Simon-Laux Simon-Laux removed the core Networking, Messages, IMAP, etc. that is handled by the core label Aug 19, 2022
@github-actions
Copy link

To test the changes in this pull request, install this apk:
📦 app-preview.apk

@@ -791,6 +791,8 @@
<string name="qraccount_success_enter_name">Login successful—your e-mail address is %1$s\n\nIf you like, you can now enter a name and an profile image that will be displayed to people you write to.</string>
<string name="qraccount_qr_code_cannot_be_used">The scanned QR code cannot be used to set up a new account.</string>
<string name="qraccount_use_on_new_install">The scanned QR code is for setting up a new account. You can scan the QR code when setting up a new Delta Chat installation.</string>
<string name="qrlogin_ask_login">Login to \"%1$s\"?</string>
Copy link
Member

@r10s r10s Aug 20, 2022

Choose a reason for hiding this comment

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

"Login" is the noun/adjective and "Log in" the verb. this is also how we use that (mostly :) elsewhere.

Copy link
Member

@r10s r10s Sep 29, 2022

Choose a reason for hiding this comment

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

ah, that bit is not fixed, will push that directly on master before uploading the strings to transifex.

EDIT: done.

@github-actions
Copy link

To test the changes in this pull request, install this apk:
📦 app-preview.apk

Copy link
Member

@r10s r10s left a comment

Choose a reason for hiding this comment

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

in general, this looks good, some minors.

let's wait for core :)

@github-actions
Copy link

To test the changes in this pull request, install this apk:
📦 app-preview.apk

@adbenitez
Copy link
Member

let's wait for core :)

also first #2367 should be merged into this branch

@github-actions
Copy link

To test the changes in this pull request, install this apk:
📦 app-preview.apk

@r10s
Copy link
Member

r10s commented Sep 29, 2022

also first #2367 should be merged into this branch

i just rebased.

@github-actions
Copy link

To test the changes in this pull request, install this apk:
📦 app-preview.apk

@r10s r10s merged commit 05fcac8 into master Sep 29, 2022
@r10s r10s deleted the dc_login_scheme branch September 29, 2022 16:34
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