-
Notifications
You must be signed in to change notification settings - Fork 0
New Release v0.2.0 - #minor #63
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Otherwise the file is only created when the venv is deactivated again. As the "pipenv shell" command takes into account what is present in .env, we additionally have the env variables properly set from the start.
Same as in service-stac.
This avoids error "django.core.management.base.CommandError: You must set settings.ALLOWED_HOSTS if DEBUG is False." when debugging. Same as in service-stac.
This avoids error "psycopg2.OperationalError: connection to server at "localhost" (127.0.0.1), port 15432 failed: server closed the connection unexpectedly" when debugging. Same as service-stac.
Same as service-stac.
They are produced by "manager.py collectstatic" in "make test-debug".
Created wIth "manage.py startapp distributions". This app is to manage attributions and datasets.
This addresses a warning shown by pylint: "UserWarning: 'BaseException' is not a proper value for the 'overgeneral-exceptions' option. Use fully qualified name (maybe 'builtins.BaseException' ?) instead. This will cease to be checked at runtime when the configuration upgrader is released."
Removed the MetaClass.verbose_name(_plural) because here they merely write out what Django does already by itself (take the class name as lower case, add an "s" for plural).
Unpack dict into arguments. Call output object "actual" to in line with usual naming in unit tests.
PB-1158: Add package distribution model
Update packages
Fix some bugs in the user admin UI and model
The idea is to create a superuser via terraform and share it with SSM to the cluster. That way we can make sure the user is always existing and up-to-date with the parameter entry
Add a little command to create or update our superuser
Create superuser with is_staff=True obviously
Create automatci superuser with is_superuser=True
This allows for coverage reports
Install pytest-cov
Add coverage report
PB-1288 Set e-mail as validated in cognito
Use XML for coverage report
Add branch coverage
Fix updating user in cognito
asteiner-swisstopo
approved these changes
Dec 19, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Creating a release to go on INT
This release
There are various smaller adjustments made, such as the setup of pytest, superuser creation, ...