Skip to content

Commit

Permalink
Merge pull request #2748 from PnX-SI/delete-app-config-ts-frontend
Browse files Browse the repository at this point in the history
Remove app.config.ts and associated commands
  • Loading branch information
jacquesfize authored Oct 12, 2023
2 parents 242ff94 + 418e1d9 commit f7e973c
Show file tree
Hide file tree
Showing 7 changed files with 1 addition and 61 deletions.
4 changes: 0 additions & 4 deletions .github/workflows/cypress.yml
Original file line number Diff line number Diff line change
Expand Up @@ -97,13 +97,9 @@ jobs:
- name: generate frontend config
run: |
cp ./config/settings.ini.sample ./config/settings.ini
geonature generate_frontend_config
./install/05_install_frontend.sh --ci
env:
GEONATURE_CONFIG_FILE: "${{ github.workspace }}/config/test_config.toml"
- name: Show generated frontend config
run: |
cat ./frontend/src/conf/app.config.ts
- name: Install core modules
run: |
geonature install-gn-module contrib/occtax OCCTAX --build=false
Expand Down
23 changes: 0 additions & 23 deletions backend/geonature/core/command/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
from geonature.utils.config import config
from geonature.utils.config_schema import GnGeneralSchemaConf, GnPySchemaConf
from geonature.utils.command import (
create_frontend_config,
create_frontend_module_config,
build_frontend,
)
Expand Down Expand Up @@ -61,27 +60,6 @@ def dev_back(ctx, host, port):
ctx.invoke(run_command, host=host, port=port)


@main.command()
@click.option(
"--input",
"input_file",
type=click.File("r"),
)
@click.option(
"--output",
"output_file",
type=click.File("w"),
)
def generate_frontend_config(input_file, output_file):
"""
Génération des fichiers de configurations pour javascript
"""
create_frontend_config(input_file, output_file)
click.echo(
"Configuration générée. Pensez à rebuilder le frontend pour la production.", err=True
)


@main.command()
@click.argument("module_code")
@click.option(
Expand Down Expand Up @@ -113,7 +91,6 @@ def update_configuration(modules, build):
"""
click.echo("Génération de la configuration du frontend :")
click.echo(" GeoNature … ", nl=False)
create_frontend_config()
click.secho("OK", fg="green")
if modules:
for dist in iter_modules_dist():
Expand Down
19 changes: 0 additions & 19 deletions backend/geonature/utils/command.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,25 +19,6 @@
from geonature.utils.module import get_dist_from_code, get_module_config


def create_frontend_config(input_file=None, output_file=None):
if input_file is None:
input_file = (FRONTEND_DIR / "src/conf/app.config.ts.sample").open("r")
else:
input_file = nullcontext(input_file)
with input_file as f:
template = Template(f.read())

parameters = json.dumps(config_frontend, indent=True)
app_config_template = template.render(parameters=parameters)

if output_file is None:
output_file = (FRONTEND_DIR / "src/conf/app.config.ts").open("w")
else:
output_file = nullcontext(output_file)
with output_file as f:
f.write(app_config_template)


def create_frontend_module_config(module_code, output_file=None):
"""
Create the frontend config
Expand Down
9 changes: 0 additions & 9 deletions docs/development.rst
Original file line number Diff line number Diff line change
Expand Up @@ -252,15 +252,6 @@ Debugger avec un navigateur
L'extension `Angular DevTools <https://angular.io/guide/devtools>`_ permettra de debugger l'application dans la console du navigateur.
Pour utiliser l'extension vous devez l'installer et passer obligatoirement en mode ``development``.

Ouvrez le fichier ``frontend/src/conf/app.config.ts`` et modifiez la valeur ``PROD_MOD`` pour avoir :

.. code-block:: javascript
:linenos:
"PROD_MOD": false
Si le mode production (PROD_MOD) est à true, alors vous n'êtes pas en mode production lors du lancement de la commande ``npm run start``.

.. _dev-backend:

Développement Backend
Expand Down
1 change: 0 additions & 1 deletion frontend/src/conf/app.config.ts.sample

This file was deleted.

4 changes: 0 additions & 4 deletions install/05_install_frontend.sh
Original file line number Diff line number Diff line change
Expand Up @@ -69,10 +69,6 @@ echo $api_end_point
sed -i 's|"API_ENDPOINT": .*$|"API_ENDPOINT" : "'${api_end_point}'"|' src/assets/config.json
cat src/assets/config.json

echo "Création de la configuration du frontend depuis 'config/geonature_config.toml'..."
# Generate the app.config.ts
geonature generate-frontend-config

echo "Désactivation du venv..."
deactivate

Expand Down
2 changes: 1 addition & 1 deletion install/install_all/install_all.sh
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ my_domain=$(echo $my_url | sed -r 's|^.*\/\/(.*)$|\1|')
my_domain=$(echo $my_domain | sed s'/.$//')
export DOMAIN_NAME="$my_domain"

# Check OS and versions
# Check OS and versions
if [ "$OS_NAME" != "debian" ]
then
echo -e "\e[91m\e[1mLe script d'installation n'est prévu que pour la distribution Debian\e[0m" >&2
Expand Down

0 comments on commit f7e973c

Please sign in to comment.