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

Make defaults generic #687

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Make defaults generic #687

wants to merge 4 commits into from

Conversation

dlebauer
Copy link
Member

It is confusing enough that there are so many instances of betydb. It does not make sense to have the default settings look like the database at betydb.org:

image

One additional step would be to add a table of all of the databases that people might be looking for.

@robkooper
Copy link
Member

@dlebauer can you add an image of what it looks like now with the changes.

@dlebauer
Copy link
Member Author

@robkooper working on it ... once I run

docker-compose -p bety up -d postgres
docker run --rm --network bety_bety pecan/bety initialize
docker-compose -p bety up -d

Should the interface be at localhost:8000/bety?

@robkooper
Copy link
Member

Yes, should be. Make sure that port is exposed in bh the docker-compose file

@dlebauer
Copy link
Member Author

@robkooper I have the same docker-compose.yml as the repo ... looks like the port should be exposed

services:
  # BETY rails frontend to the database
  bety:
    image: pecan/bety:${BRANCH:-latest}
    networks:
      - bety
    ports:
      - 8000
    environment:
      - UNICORN_WORKER_PROCESSES=1
      - SECRET_KEY_BASE=thisissomereallllllylongsecretkeyandshouldbelongerthanthis
    depends_on:
      - postgres
    restart: unless-stopped

@robkooper
Copy link
Member

correct at that point it is exposed, not as port 8000 though. You will need to do a docker ps to see the port, or easier docker port bety 8000 You will see something like: 0.0.0.0:32769 meaning you need to connect to http://localhost:32769.

- URL: http://illinois.edu
title: "University of Illinois at Urbana-Champaign"
text: "University of Illinois at Urbana-Champaign"
- URL: http://example.com
Copy link
Member

Choose a reason for hiding this comment

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

how about we replace this with icons and names of all universities involved that contributed to BETY

  • Illinois
  • BU
  • AU

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.

2 participants