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

firebase token file sync #801

Merged
merged 9 commits into from
Apr 3, 2022
Merged

firebase token file sync #801

merged 9 commits into from
Apr 3, 2022

Conversation

dimacodota
Copy link
Contributor

No description provided.

@@ -3,7 +3,7 @@ import * as os from "os";

export const TABNINE_CONFIG_DIR = path.join(os.homedir(), ".config", "TabNine");

export const TABNINE_TOKEN_FILE_NAME = "tabnine.token";
export const TABNINE_TOKEN_FILE_NAME = ".refresh_token_v2";

Choose a reason for hiding this comment

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

Replace with

Suggested change
export const TABNINE_TOKEN_FILE_NAME = ".refresh_token_v2";
export const TABNINE_TOKEN_FILE_NAME = ".refresh_token";


async function openBinaryConfig(url: string): Promise<void> {
const config = await configuration({ quiet: true, source: StateType.AUTH });
if (config && config.message) {

Choose a reason for hiding this comment

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

Replace with

Suggested change
if (config && config.message) {
if (config?.message) {


async function openBinaryConfig(url: string): Promise<void> {
const config = await configuration({ quiet: true, source: StateType.AUTH });
if (config && config.message) {

Choose a reason for hiding this comment

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

Replace with

Suggested change
if (config && config.message) {
if (config?.message) {

Choose a reason for hiding this comment

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

amogus

const config = await configuration({ quiet: true, source: StateType.AUTH });
if (config && config.message) {
const localUri = await env.asExternalUri(Uri.parse(config.message));
const callback = `https://app.tabnine.com/auth/sign-in?tabnineUrl=${localUri.toString()}&sync=false`;
Copy link
Contributor

Choose a reason for hiding this comment

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

Bassa having it here. No?

@dimacodota dimacodota merged commit 98b9fad into master Apr 3, 2022
@dimacodota dimacodota deleted the fix/fix-firebase-token-sync branch April 3, 2022 08:24
Copy link

@concerete concerete left a comment

Choose a reason for hiding this comment

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

Just deform the .js files into a .html

ofekby pushed a commit that referenced this pull request Aug 1, 2023
Former-commit-id: ebcd3bed9b86d98a05c50cf81eb76b12ce6816ab [formerly b154b272dd7e3745b9b48c6baf9e45253d8f2195] [formerly 397f7fb72fafe709abdea764b7dd999169c141a6 [formerly 98b9fad]]
Former-commit-id: 1b2b53122191793ca5d5bdeadb932816ae8dd0c4 [formerly 7cc69fb7980652fef166f39c2ee805f02fb77cb6]
Former-commit-id: 7d52956dea64d3f477b3c957c8b115da4a31e109
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