Skip to content
This repository was archived by the owner on Mar 12, 2025. It is now read-only.

added tc-auth #5

Merged
merged 2 commits into from
Jun 16, 2018
Merged

added tc-auth #5

merged 2 commits into from
Jun 16, 2018

Conversation

veshu
Copy link
Contributor

@veshu veshu commented Jun 11, 2018

No description provided.

@callmekatootie
Copy link
Collaborator

@cwdcwd The Auth check for Lauscher. Do I merge this in right away? Or do we need to carry out additional steps for deployment

@@ -2,7 +2,10 @@
const config = {
API_URL: 'http://localhost:4000/api/v1',
WS_URL: 'ws://localhost:4000',
DEFAULT_MESSAGE_COUNT: 20
DEFAULT_MESSAGE_COUNT: 20,
TC_AUTH_URL: 'https://accounts.topcoder-dev.com',
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we get all these in Environmental Variables?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yup. Getting them in in a new PR.

@callmekatootie callmekatootie merged commit 4a6dc9f into topcoder-archive:develop Jun 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants