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

2025-01-24-Adapt-and-update-trigger-and-functions #80

Merged
merged 6 commits into from
Jan 27, 2025

Conversation

sjib
Copy link
Contributor

@sjib sjib commented Jan 24, 2025

  • re-group modification_functions.sql as in tww
  • update oid_functions.sql as in tww
  • Adapt organisation_functions.sql as in tww
  • Add default values as in tww

@sjib
Copy link
Contributor Author

sjib commented Jan 24, 2025

 Traceback (most recent call last):
  File "/src/datamodel/scripts/../app/create_app.py", line 175, in <module>
    create_app(
  File "/src/datamodel/scripts/../app/create_app.py", line 64, in create_app
    run_sql_file("functions/oid_functions.sql", pg_service, variables)
  File "/src/datamodel/scripts/../app/create_app.py", line 23, in run_sql_file
    run_sql(sql, pg_service, variables)
  File "/src/datamodel/scripts/../app/create_app.py", line 28, in run_sql
    sql = psycopg.sql.SQL(sql).format(**variables)
  File "/opt/venv/lib/python3.9/site-packages/psycopg/sql.py", line 277, in format
    rv.append(args[int(name)])
IndexError: tuple index out of range

@sjib sjib added the enhancement New feature or request label Jan 24, 2025
@sjib sjib self-assigned this Jan 24, 2025
ponceta and others added 3 commits January 27, 2025 14:48
Don't pass variables unless required and used
Set default values
@ponceta ponceta merged commit 56f10d4 into main Jan 27, 2025
5 checks passed
@ponceta ponceta deleted the 2025-01-24-Adapt-and-update-trigger-and-functions branch January 27, 2025 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants