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

Correct the copying of e-mail addresses in the electron app #8124

Merged
merged 2 commits into from
Jan 15, 2019

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Jan 15, 2019

Fixes #7937

Changes the labels all the match casing of Chrome.
Fixes a small typo in a comment.

Signed-off-by: Michael Telatynski [email protected]

This was originally added in #7755, which pulled in the original source for the
module, breaking ancient browsers without support for classes (#8082).
@t3chguy t3chguy requested a review from a team January 15, 2019 08:31
@t3chguy t3chguy changed the base branch from master to develop January 15, 2019 08:37
@t3chguy t3chguy force-pushed the t3chguy/electron-mailto branch from d7e7180 to 8c895cd Compare January 15, 2019 08:37
@turt2live turt2live changed the base branch from develop to experimental January 15, 2019 15:37
@jryans
Copy link
Collaborator

jryans commented Jan 15, 2019

(It is a bit surprising that this PR shows my develop commit for SdkConfig, while other develop -> experimental merges did not...)

@turt2live
Copy link
Member

That change apparently hasn't made it to experimental I guess :(

I'm generally fine with ignoring the problem though and reviewing the actual changes at hand

@t3chguy
Copy link
Member Author

t3chguy commented Jan 15, 2019

I/someone can rebase if needs be.

@t3chguy
Copy link
Member Author

t3chguy commented Jan 15, 2019

Does it need rebasing to get out that other commit or not?

@turt2live
Copy link
Member

I don't think it does. It's a small change that should be on experimental anyways: this is just the vessel to get it there

@jryans
Copy link
Collaborator

jryans commented Jan 15, 2019

(The actual textual change made in the other commit is already present on experimental. It was just made via different commits for develop and experimental, which seems to be confusing things. Should be safe to ignore.)

@t3chguy t3chguy merged commit 67ba81e into experimental Jan 15, 2019
@t3chguy t3chguy deleted the t3chguy/electron-mailto branch May 12, 2022 09:03
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