Skip to content

Commit

Permalink
Merge pull request #1392 from research-software-directory/1391-safari…
Browse files Browse the repository at this point in the history
…-development-documentation

docs: add a note on Safari for developing the RSD
  • Loading branch information
ewan-escience authored Feb 7, 2025
2 parents 5a2e93a + 29be3b0 commit c2ae88a
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
SPDX-FileCopyrightText: 2021 - 2022 Dusan Mijatovic (dv4all)
SPDX-FileCopyrightText: 2021 - 2022 Jason Maassen (Netherlands eScience Center) <[email protected]>
SPDX-FileCopyrightText: 2021 - 2022 dv4all
SPDX-FileCopyrightText: 2021 - 2024 Ewan Cahen (Netherlands eScience Center) <[email protected]>
SPDX-FileCopyrightText: 2021 - 2024 Netherlands eScience Center
SPDX-FileCopyrightText: 2021 - 2025 Ewan Cahen (Netherlands eScience Center) <[email protected]>
SPDX-FileCopyrightText: 2021 - 2025 Netherlands eScience Center
SPDX-FileCopyrightText: 2021 Jesús García Gonzalez (Netherlands eScience Center) <[email protected]>
SPDX-FileCopyrightText: 2022 - 2024 Christian Meeßen (GFZ) <[email protected]>
SPDX-FileCopyrightText: 2022 - 2024 Helmholtz Centre Potsdam - GFZ German Research Centre for Geosciences
Expand Down Expand Up @@ -72,7 +72,13 @@ make stop
docker compose down
```

## Developing the frontend
## Developing the RSD

We discourage the use of Safari for developing the RSD. The reason is that it is not possible to sign in with Safari on localhost, because Safari handles [secure cookies differently](https://codedamn.com/news/web-development/safari-cookie-is-not-being-set) from other browsers.

For more details on developing the RSD, see the respective readmes in the varying directories.

### Developing the frontend

You can run frontend in development mode as docker a service (called frontend-dev) that enables hot reloading. By default this frontend-dev service will not be started automatically. For more detailed instructions see [frontend/README.md](frontend/README.md).

Expand All @@ -83,7 +89,7 @@ make frontend-dev
docker compose -f docker-compose.yml -f docker-compose.dev.yml up
```

It is possible to directly run the frontend too (without using a docker container). You must then have NodeJS installed, preferably v18.
It is possible to directly run the frontend too (without using a docker container). You must then have Node.js installed, see the `Dockerfile` in the frontend directory for the version in use.

```
# Build and install all dependencies.
Expand Down

0 comments on commit c2ae88a

Please sign in to comment.