Skip to content

Commit

Permalink
Merge pull request #65 from AMT-Project/l-updateREADME
Browse files Browse the repository at this point in the history
Updated README
  • Loading branch information
bonzonlu authored Oct 30, 2020
2 parents 9b65e32 + 2c8893c commit 35a43b7
Showing 1 changed file with 25 additions and 7 deletions.
32 changes: 25 additions & 7 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,17 @@
# StackOverflow
# S7ack Application

Stack is a question and answer site for all, based on the famous programming Q&A website Stack Overflow. It's built using the MVC pattern on the server side, using Java EE APIs (Servlets, JSPs, JSTL, JDBC), runs on Open Liberty Application server and runs in a Dockerised environment.

#### Students

| Student | GitHub username |
| ---------------- | --------------- |
| Ludovic Bonzon | bonzonlu |
| Claire Delhomme | Kleeer |
| Pablo Mercado | pabloheigvd |
| Vitor Vaz Afonso | vitorva |

## Prerequisites

Do not forget to edit your `/etc/hosts` file to add:
```
Expand Down Expand Up @@ -38,9 +51,7 @@ Before starting the test, deploy a website by `cd`'ing into `/scripts` folder an
```
And checkout out the votes at stack.ch:9080/stack/

# S7ack Application

Stack is a question and answer site for all, based on the famous programming Q&A website Stack Overflow. It's built using the MVC pattern on the server side, using Java EE APIs (Servlets, JSPs, JSTL, JDBC), runs on Open Liberty Application server and runs in a Dockerised environment.
## Application

### Pages list

Expand Down Expand Up @@ -79,11 +90,18 @@ We choose not to implement a tagging mechanism, or a searching bar, the only nav

### Password policy

Passwords are hashed when stored in the DB, and need to be a minimum of 3 characters.
Passwords are hashed when stored in the DB, and need to be a minimum of 3 characters (we know it's a bit weak, but we could easily add more security if necessary/asked).

# What doesn't function quite right
# Possible improvements
To run integration tests, you need an instance of the website before running IT tests inside intelliJ. You can deploy this website by `cd`'ing in `\scripts` and run:
```bash
./runDocker.sh
```
then run the tests.
then run the tests.

Through our various test plans, we didn't find any major issues or problems, but there is always things to tweak and improve, other than the bugs that might have made their way through, we can think of the following improvements :

- Adding more tests (especially JMeter load tests, but all kinds of tests are always welcome, and also tests using Arquillian are not yet implemented) and thus improving coverage.
- Factoring JSP components into fragments (we didn't have the time to do that before the project_1 deadline).
- Implementing a tagging mechanism nor a search bar for easier navigation through large amount of data.
- Improving the CSS.

0 comments on commit 35a43b7

Please sign in to comment.