Skip to content
This repository has been archived by the owner on Mar 24, 2021. It is now read-only.

JBrowse screenshots #251

Merged
23 commits merged into from
Mar 28, 2017
Merged

Conversation

nathandunn
Copy link
Contributor

No description provided.

@nathandunn
Copy link
Contributor Author

WIP - for #244

@nathandunn nathandunn changed the title Test jbrowse tempfix (don't merge) Test jbrowse tempfix Mar 28, 2017
@nathandunn nathandunn self-assigned this Mar 28, 2017
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

After the IP is replaced with the domain I think it will be ready to submit

render() {
// let externalPrefix = 'http://bw.scottcain.net/jbrowse/?data=data%2F';
// let externalPrefix = 'http://34.208.22.23/jbrowse/overview.html?data=data%2F';
let externalPrefix = 'http://52.43.125.139/jbrowse/overview.html?data=data%2F';
Copy link

Choose a reason for hiding this comment

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

This hard coded IP should be "jbrowse.alliancegenome.org

// let internalPrefix = 'http://localhost/jbrowse/overview.html?data=data%2F';
// let visualizationPrefix = 'http://dev.alliancegenome.org:8891/?url=';
// let visualizationPrefix = 'http://localhost:8891/?url=';
let visualizationPrefix = 'http://52.43.125.139:8891/?url=';
Copy link

Choose a reason for hiding this comment

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

Change IP here too

@nathandunn nathandunn changed the title Test jbrowse tempfix JBrowse screenshots Mar 28, 2017
@nathandunn
Copy link
Contributor Author

@a8wright I made most of the changes and cleaned out some stuff. You saw my note that: jbrowse.alliancegenome.org doesn't resolve?

If we can get that in today then great. Otherwise, we should just use these IP's until next week so that I can demo.

@ghost ghost merged commit 0afbf73 into alliance-genome:development Mar 28, 2017
This pull request was closed.
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.

1 participant