Skip to content

Update README.md#3

Open
shiftedgears wants to merge 1 commit into
mainfrom
shiftedgears-patch-2
Open

Update README.md#3
shiftedgears wants to merge 1 commit into
mainfrom
shiftedgears-patch-2

Conversation

@shiftedgears
Copy link
Copy Markdown
Collaborator

removed some not so good info

removed info
Copy link
Copy Markdown
Owner

@Enprogames Enprogames left a comment

Choose a reason for hiding this comment

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

I made a few comments for changes that I think would improve the readme. It might be a good idea to remove a lot of the VIU-related instructions, but still keep instructions for running this locally without the VIU servers.

Comment thread README.md
DB_PASS=rAnhgtHT
DB_NAME=csci311h_tekku
PHRASE=tekku
DB_PORT=removed
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Hmm this is actually meant for if someone wanted to run the server and database locally, so I think we can probably keep the previous changes. The port, for example, is important, and shouldn't be set to "removed". But maybe the password should be set to something else, such as "change_me".

Comment thread README.md
@@ -130,21 +130,21 @@ Here is what it should look like when you make changes:
Using SSH tunnels into your CSCI account, you can access the Gitlab repos and webpage from anywhere. I'm doing this so that I can have my own setup away from the CSCI computers, since they cause me constant issues, such as with my quota. Here's how I do it.
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I think this entire section should just be removed.

Comment thread README.md
- `git clone git@127.0.0.1:<username>/tekku.git`
- Add remote to upstream: `git remote add git@127.0.0.1:oleander/tekku.git`

## View Dolphin Webpages from Off Campus
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I think this section, and "Connecting PHP Backend to Database" can be completely removed.

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