-
Notifications
You must be signed in to change notification settings - Fork 14
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
Initial version of Cloud Console and IDP modified for Cloud Console #298
Merged
Conversation
This file contains 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
BREAKING CHANGE: New client, additional option in fetcher, ui for new microservice
* feat(conversion): add mobile optimization to PDF pipeline * feat: working implementation with API endpoints * fix(api): OpenAPI docs * chore: revert unnecessary changes * fix(api): OpenAPI docs * docs: update DEVELOPMENT.md * chore: rename to segmentation * chore: update OpenAPI docs * fix: revert DTO changes * chore: update OpenAPI docs * fix(migrations): file header * chore(ui): add PDFProps field * feat: use ImageMagick to generate PDF thumbnails as JPEG * chore: generate OpenAPI docs
* chore: enrich S3Object * fix: router params * fix: router params * fix: error checking * feat: enrich snapshot.mosaic field * chore: improve S3Object data structure
feat: further enrichment of snapshot.original field with more properties (see JSON below) feat: further enrichment of snapshot.preview field with more properties (see JSON below) feat: further enrichment of snapshot.segmentation field with more properties (see JSON below) feat: further enrichment of snapshot.mosaic field with more properties (see JSON below) refactor: clients on API microservice look the same like other microservices (preparing for deduplication)
To prevent using outdated links to debian packages that were last refreshed during the docker build, perform an apt-update at the start.
* feat(api): endpoint to reprocess a file * refactor(api): no naked return * refactor(api): don't mark folders as rejected * fix: set snapshot.task_id to NULL when deleting a task * chore: format code * chore: fix linter error
* feat(mosaic): use bild * chore(conversion): format code * refactor(mosaic): delete not needed type conversion * feat(conversion): use bild * refactor(mosaic): remove more unnecessary type conversions
* fix: hybrid integration of bild and ImageMagick * chore: format code * fix: add support for .tif extension * fix: various issues * fix: base mosaic on preview
* feat(conversion): automatically trigger mosaic * refactor(conversion): adjustments * refactor(conversion): no need for a separate function * fix(conversion): delete output files correctly * fixup! fix(conversion): delete output files correctly * fix(conversion): correctly handle forwarding in pipelines
…service also works
BREAKING CHANGE: New client, additional option in fetcher, ui for new microservice
…service also works
…i-admin-panel # Conflicts: # ui/src/pages/admin/admin-panel-groups.tsx # ui/src/pages/admin/admin-panel-organizations.tsx # ui/src/pages/admin/admin-panel-workspaces.tsx # ui/src/pages/admin/admin-rename-modal.tsx
…n UI, Pagination option to disable rewind and jump to last buttons
bouassaba
reviewed
Sep 3, 2024
bouassaba
reviewed
Sep 3, 2024
bouassaba
reviewed
Sep 3, 2024
bouassaba
reviewed
Sep 3, 2024
bouassaba
reviewed
Sep 3, 2024
bouassaba
reviewed
Sep 3, 2024
bouassaba
reviewed
Sep 3, 2024
bouassaba
reviewed
Sep 3, 2024
bouassaba
reviewed
Sep 3, 2024
bouassaba
reviewed
Sep 3, 2024
bouassaba
reviewed
Sep 3, 2024
bouassaba
requested changes
Sep 3, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check the review comments.
bouassaba
reviewed
Sep 4, 2024
bouassaba
reviewed
Sep 4, 2024
bouassaba
reviewed
Sep 4, 2024
bouassaba
reviewed
Sep 4, 2024
bouassaba
reviewed
Sep 4, 2024
bouassaba
reviewed
Sep 4, 2024
bouassaba
approved these changes
Sep 4, 2024
loboda4450
approved these changes
Sep 4, 2024
dsonck92
reviewed
Sep 5, 2024
Comment on lines
+86
to
+97
.col( | ||
ColumnDef::new(User::IsAdmin) | ||
.boolean() | ||
.not_null() | ||
.default(false), | ||
) | ||
.col( | ||
ColumnDef::new(User::IsActive) | ||
.boolean() | ||
.not_null() | ||
.default(true), | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Never change existing migrations. This needs to be reverted asap
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.
Closes: #151