-
Notifications
You must be signed in to change notification settings - Fork 150
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
Bug on adding a record to a community #2821
Closed
jrcastro2 opened this issue
Sep 2, 2024
· 0 comments
· Fixed by #2822 or inveniosoftware/invenio-rdm-records#1806
Closed
Bug on adding a record to a community #2821
jrcastro2 opened this issue
Sep 2, 2024
· 0 comments
· Fixed by #2822 or inveniosoftware/invenio-rdm-records#1806
Labels
bug
Something isn't working
Comments
jrcastro2
added a commit
to jrcastro2/invenio-app-rdm
that referenced
this issue
Sep 2, 2024
jrcastro2
added a commit
to jrcastro2/invenio-app-rdm
that referenced
this issue
Sep 2, 2024
jrcastro2
added a commit
to jrcastro2/invenio-rdm-records
that referenced
this issue
Sep 2, 2024
jrcastro2
added a commit
to jrcastro2/invenio-rdm-records
that referenced
this issue
Sep 2, 2024
* Removes response_handler to avoid allowing to process the response with the schema * closes inveniosoftware/invenio-app-rdm#2821
jrcastro2
added a commit
to jrcastro2/invenio-app-rdm
that referenced
this issue
Sep 2, 2024
jrcastro2
added a commit
to inveniosoftware/invenio-rdm-records
that referenced
this issue
Sep 3, 2024
* Removes response_handler to avoid allowing to process the response with the schema * closes inveniosoftware/invenio-app-rdm#2821
github-project-automation
bot
moved this from In review
to To release 🤖
in Sprint Q4/2024 🎄
Sep 3, 2024
jrcastro2
added a commit
that referenced
this issue
Sep 3, 2024
slint
added a commit
to slint/zenodo-rdm
that referenced
this issue
Sep 23, 2024
📁 invenio-accounts (5.1.1 -> 5.1.2 🐛) release: v5.1.2 security: filter out current session if it's set to `None` * also, rename it to avoid controversial terminology dependencies: bump flask-security-invenio minimum 📁 invenio-app-rdm (13.0.0b1.dev3 -> 13.0.0b1.dev4 🌈) release: v13.0.0b1.dev4 config: make OAI-PMH record index dynamic modal: fix record adding to a community * closes inveniosoftware/invenio-app-rdm#2821 📁 invenio-banners (3.1.0 -> 3.1.1 🐛) 📦 release: v3.1.1 admin: adjust descriptions of fields 📁 invenio-files-rest (2.2.0 -> 2.2.1 🐛) release: v2.2.1 fix: fix downloading for some weird filenames the following filenames couldn't be downloaded before: - is not `latin-1` encodable, ... - ...and contains a comma (`,`) character fix: apply invenio-accounts api changes 📁 invenio-jobs (0.5.0 -> 0.5.1 🐛) release: v0.5.1 fix: add compatibility layer to move to flask>=3 * flask-sqlalchemy moved pagination. * this change has been added to have a smooth migration to flask>=3.0.0 without creating a hard cut and major versions release. 📁 invenio-oaiserver (2.2.1 -> 2.2.2 🐛) 📦 release: v2.2.2 percolator: allow lazy strings from config 📁 invenio-oauthclient (4.0.0 -> 4.0.2 🐛) release: v4.0.2 views: fix boolean logic for checking remote app visibility release: v4.0.1 global: explicitly set and use hide attribute for config fix: add flag for auth in keycloak realm url 📁 invenio-pages (4.1.0 -> 4.1.1 🐛) release: v4.1.1 fix: add compatibility layer to move to flask>=3 * flask-sqlalchemy moved pagination. * this change has been added to have a smooth migration to flask>=3.0.0 without creating a hard cut and major versions release. 📁 invenio-rdm-records (12.1.0 -> 12.2.1 🌈) release: v12.2.1 deposit: check permission and set disable tooltip for publish button Refactor: Enhance Warnings Handling * Add the missing translation tags. * Refactor the logic to display multiple warning messages, while allowing other files to upload, except for case where exceeding the file size limit. UI: Handle empty file uploads in FileUploader * Display a warning for empty files, indicating they won't be included and listing file names. * Feature controlled by `records-resources-allow-empty-files` config value. * Continue uploading other files while showing the warning message. serializers: ensure values are not None before access release: v12.1.1 fixtures: allow to load new vocab resource: fix add record to community * Removes response_handler to avoid allowing to process the response with the schema * closes inveniosoftware/invenio-app-rdm#2821 controls: refactored isDisabled function * Refactored isDisabled function in PublishButton and SubmitReviewButton components to check if all files are finished uploading before enabling the button. * Passed filesState instead of just the number of files to the function. * Added a check for non-finished files. * Added early returns to avoid unnecessary checks on possible large objects / arrays. 📁 invenio-requests (5.0.0 -> 5.1.0 🌈) release: v5.1.0 assets: optimise detection if timeline was updated timeline feed: render mathematical content in the request comments * closes CERNDocumentServer/cds-rdm#133 📁 invenio-theme (3.3.0 -> 3.4.0 🌈) release: v3.4.0 config: add MathJax support * closes CERNDocumentServer/cds-rdm#133 navbar: fix logo size in non-desktop devices css: updated table border style 📁 invenio-users-resources (6.0.0 -> 6.1.0 🌈) release: v6.1.0 users: services: Add can_read_all permission for admin search services: permissions: Use search_all config for admin user search CI: Switch To Centralised Workflows * Change branches type from string to array * Switch to centralised workflows 📁 invenio-vocabularies (5.0.2 -> 5.0.3 🐛) 📦 release: v5.0.3 services: skip index rebuilding
slint
added a commit
to slint/zenodo-rdm
that referenced
this issue
Sep 25, 2024
📁 invenio-accounts (5.1.1 -> 5.1.2 🐛) release: v5.1.2 security: filter out current session if it's set to `None` * also, rename it to avoid controversial terminology dependencies: bump flask-security-invenio minimum 📁 invenio-app-rdm (13.0.0b1.dev3 -> 13.0.0b1.dev5 🌈) 📦 release: v13.0.0b1.dev5 deposit: Add allow-empty-files config available for deposit page * Expose `RECORDS_RESOURCES_ALLOW_EMPTY_FILES` for UI control * Related to: inveniosoftware/invenio-rdm-records#1802 deposit: provide permissions to publish button config: add group resolver for notifications admin-records: add reference to gh issue admin-records: account for system owned records migration: account for deleted communities and draft concept DOI css: added class for padding removal theme: fix accordion rotation template: mathjax remove from javascript block templates: add mathjax only to parent template landing page: support different MathJax delimeters search-result: namespace overridable id for community search results search-result: provide key to part of community array element fix: the following / prevents rspack from building release: v13.0.0b1.dev4 config: make OAI-PMH record index dynamic modal: fix record adding to a community * closes inveniosoftware/invenio-app-rdm#2821 📁 invenio-banners (3.1.0 -> 3.1.1 🐛) 📦 release: v3.1.1 admin: adjust descriptions of fields 📁 invenio-files-rest (2.2.0 -> 2.2.1 🐛) release: v2.2.1 fix: fix downloading for some weird filenames the following filenames couldn't be downloaded before: - is not `latin-1` encodable, ... - ...and contains a comma (`,`) character fix: apply invenio-accounts api changes 📁 invenio-jobs (0.5.0 -> 0.5.1 🐛) release: v0.5.1 fix: add compatibility layer to move to flask>=3 * flask-sqlalchemy moved pagination. * this change has been added to have a smooth migration to flask>=3.0.0 without creating a hard cut and major versions release. 📁 invenio-oaiserver (2.2.1 -> 2.2.2 🐛) 📦 release: v2.2.2 percolator: allow lazy strings from config 📁 invenio-oauthclient (4.0.0 -> 4.0.2 🐛) release: v4.0.2 views: fix boolean logic for checking remote app visibility release: v4.0.1 global: explicitly set and use hide attribute for config fix: add flag for auth in keycloak realm url 📁 invenio-pages (4.1.0 -> 4.1.1 🐛) release: v4.1.1 fix: add compatibility layer to move to flask>=3 * flask-sqlalchemy moved pagination. * this change has been added to have a smooth migration to flask>=3.0.0 without creating a hard cut and major versions release. 📁 invenio-rdm-records (12.1.0 -> 12.2.1 🌈) release: v12.2.1 deposit: check permission and set disable tooltip for publish button Refactor: Enhance Warnings Handling * Add the missing translation tags. * Refactor the logic to display multiple warning messages, while allowing other files to upload, except for case where exceeding the file size limit. UI: Handle empty file uploads in FileUploader * Display a warning for empty files, indicating they won't be included and listing file names. * Feature controlled by `records-resources-allow-empty-files` config value. * Continue uploading other files while showing the warning message. serializers: ensure values are not None before access release: v12.1.1 fixtures: allow to load new vocab resource: fix add record to community * Removes response_handler to avoid allowing to process the response with the schema * closes inveniosoftware/invenio-app-rdm#2821 controls: refactored isDisabled function * Refactored isDisabled function in PublishButton and SubmitReviewButton components to check if all files are finished uploading before enabling the button. * Passed filesState instead of just the number of files to the function. * Added a check for non-finished files. * Added early returns to avoid unnecessary checks on possible large objects / arrays. 📁 invenio-requests (5.0.0 -> 5.1.0 🌈) release: v5.1.0 assets: optimise detection if timeline was updated timeline feed: render mathematical content in the request comments 📁 invenio-theme (3.3.0 -> 3.4.0 🌈) release: v3.4.0 config: add MathJax support navbar: fix logo size in non-desktop devices css: updated table border style 📁 invenio-users-resources (6.0.0 -> 6.1.0 🌈) release: v6.1.0 users: services: Add can_read_all permission for admin search services: permissions: Use search_all config for admin user search CI: Switch To Centralised Workflows * Change branches type from string to array * Switch to centralised workflows 📁 invenio-vocabularies (5.0.2 -> 5.0.3 🐛) 📦 release: v5.0.3 services: skip index rebuilding
slint
added a commit
to slint/zenodo-rdm
that referenced
this issue
Sep 25, 2024
📁 invenio-accounts (5.1.1 -> 5.1.2 🐛) release: v5.1.2 security: filter out current session if it's set to `None` * also, rename it to avoid controversial terminology dependencies: bump flask-security-invenio minimum 📁 invenio-app-rdm (13.0.0b1.dev3 -> 13.0.0b1.dev5 🌈) 📦 release: v13.0.0b1.dev5 deposit: Add allow-empty-files config available for deposit page * Expose `RECORDS_RESOURCES_ALLOW_EMPTY_FILES` for UI control * Related to: inveniosoftware/invenio-rdm-records#1802 deposit: provide permissions to publish button config: add group resolver for notifications admin-records: add reference to gh issue admin-records: account for system owned records migration: account for deleted communities and draft concept DOI css: added class for padding removal theme: fix accordion rotation template: mathjax remove from javascript block templates: add mathjax only to parent template landing page: support different MathJax delimeters search-result: namespace overridable id for community search results search-result: provide key to part of community array element fix: the following / prevents rspack from building release: v13.0.0b1.dev4 config: make OAI-PMH record index dynamic modal: fix record adding to a community * closes inveniosoftware/invenio-app-rdm#2821 📁 invenio-banners (3.1.0 -> 3.1.1 🐛) 📦 release: v3.1.1 admin: adjust descriptions of fields 📁 invenio-files-rest (2.2.0 -> 2.2.1 🐛) release: v2.2.1 fix: fix downloading for some weird filenames the following filenames couldn't be downloaded before: - is not `latin-1` encodable, ... - ...and contains a comma (`,`) character fix: apply invenio-accounts api changes 📁 invenio-jobs (0.5.0 -> 0.5.1 🐛) release: v0.5.1 fix: add compatibility layer to move to flask>=3 * flask-sqlalchemy moved pagination. * this change has been added to have a smooth migration to flask>=3.0.0 without creating a hard cut and major versions release. 📁 invenio-oaiserver (2.2.1 -> 2.2.2 🐛) 📦 release: v2.2.2 percolator: allow lazy strings from config 📁 invenio-oauthclient (4.0.0 -> 4.0.2 🐛) release: v4.0.2 views: fix boolean logic for checking remote app visibility release: v4.0.1 global: explicitly set and use hide attribute for config fix: add flag for auth in keycloak realm url 📁 invenio-pages (4.1.0 -> 4.1.1 🐛) release: v4.1.1 fix: add compatibility layer to move to flask>=3 * flask-sqlalchemy moved pagination. * this change has been added to have a smooth migration to flask>=3.0.0 without creating a hard cut and major versions release. 📁 invenio-rdm-records (12.1.0 -> 12.2.1 🌈) release: v12.2.1 deposit: check permission and set disable tooltip for publish button Refactor: Enhance Warnings Handling * Add the missing translation tags. * Refactor the logic to display multiple warning messages, while allowing other files to upload, except for case where exceeding the file size limit. UI: Handle empty file uploads in FileUploader * Display a warning for empty files, indicating they won't be included and listing file names. * Feature controlled by `records-resources-allow-empty-files` config value. * Continue uploading other files while showing the warning message. serializers: ensure values are not None before access release: v12.1.1 fixtures: allow to load new vocab resource: fix add record to community * Removes response_handler to avoid allowing to process the response with the schema * closes inveniosoftware/invenio-app-rdm#2821 controls: refactored isDisabled function * Refactored isDisabled function in PublishButton and SubmitReviewButton components to check if all files are finished uploading before enabling the button. * Passed filesState instead of just the number of files to the function. * Added a check for non-finished files. * Added early returns to avoid unnecessary checks on possible large objects / arrays. 📁 invenio-requests (5.0.0 -> 5.1.0 🌈) release: v5.1.0 assets: optimise detection if timeline was updated timeline feed: render mathematical content in the request comments 📁 invenio-theme (3.3.0 -> 3.4.0 🌈) release: v3.4.0 config: add MathJax support navbar: fix logo size in non-desktop devices css: updated table border style 📁 invenio-users-resources (6.0.0 -> 6.1.0 🌈) release: v6.1.0 users: services: Add can_read_all permission for admin search services: permissions: Use search_all config for admin user search CI: Switch To Centralised Workflows * Change branches type from string to array * Switch to centralised workflows 📁 invenio-vocabularies (5.0.2 -> 5.0.3 🐛) 📦 release: v5.0.3 services: skip index rebuilding
slint
added a commit
to slint/zenodo-rdm
that referenced
this issue
Sep 25, 2024
📁 invenio-accounts (5.1.1 -> 5.1.2 🐛) release: v5.1.2 security: filter out current session if it's set to `None` * also, rename it to avoid controversial terminology dependencies: bump flask-security-invenio minimum This is for v12 as invenio-users-resources at least needs it https://github.com/inveniosoftware/invenio-users-resources/blob/master/invenio_users_resources/resources/users/resource.py#L15 and invenio-accounts is the controlling distribution package for that dependency. 📁 invenio-app-rdm (13.0.0b1.dev3 -> 13.0.0b1.dev5 🌈) 📦 release: v13.0.0b1.dev5 deposit: Add allow-empty-files config available for deposit page * Expose `RECORDS_RESOURCES_ALLOW_EMPTY_FILES` for UI control * Related to: inveniosoftware/invenio-rdm-records#1802 deposit: provide permissions to publish button config: add group resolver for notifications admin-records: add reference to gh issue admin-records: account for system owned records migration: account for deleted communities and draft concept DOI css: added class for padding removal theme: fix accordion rotation template: mathjax remove from javascript block templates: add mathjax only to parent template landing page: support different MathJax delimeters * closes CERNDocumentServer/cds-rdm#133 search-result: namespace overridable id for community search results search-result: provide key to part of community array element fix: the following / prevents rspack from building release: v13.0.0b1.dev4 config: make OAI-PMH record index dynamic modal: fix record adding to a community * closes inveniosoftware/invenio-app-rdm#2821 📁 invenio-banners (3.1.0 -> 3.1.1 🐛) 📦 release: v3.1.1 admin: adjust descriptions of fields 📁 invenio-files-rest (2.2.0 -> 2.2.1 🐛) release: v2.2.1 fix: fix downloading for some weird filenames the following filenames couldn't be downloaded before: - is not `latin-1` encodable, ... - ...and contains a comma (`,`) character fix: apply invenio-accounts api changes * This commit addresses compatibility issues caused by the recent API changes in the invenio-accounts module that is introduced in [this commit](inveniosoftware/invenio-accounts@5e1c583). It removes the usage of accounts_blueprint from invenio_files_rest and related tests, as it is no longer required and causes failures in the continuous integration (CI) tests. The change is aligned with the new release notes of invenio-accounts where this modification has been documented. * Upgrades Sphinx to >=5.0.0,<6.0.0 to address compatibility issues with Python 3.11. Removes deprecated 'formatargspec' from inspect, critical for Python 3.11 support and stability in our Invenio platform. 📁 invenio-jobs (0.5.0 -> 0.5.1 🐛) release: v0.5.1 fix: add compatibility layer to move to flask>=3 * flask-sqlalchemy moved pagination. * this change has been added to have a smooth migration to flask>=3.0.0 without creating a hard cut and major versions release. 📁 invenio-oaiserver (2.2.1 -> 2.2.2 🐛) 📦 release: v2.2.2 percolator: allow lazy strings from config 📁 invenio-oauthclient (4.0.0 -> 4.0.2 🐛) release: v4.0.2 views: fix boolean logic for checking remote app visibility release: v4.0.1 global: explicitly set and use hide attribute for config fix: add flag for auth in keycloak realm url 📁 invenio-pages (4.1.0 -> 4.1.1 🐛) release: v4.1.1 fix: add compatibility layer to move to flask>=3 * flask-sqlalchemy moved pagination. * this change has been added to have a smooth migration to flask>=3.0.0 without creating a hard cut and major versions release. 📁 invenio-rdm-records (12.1.0 -> 12.2.1 🌈) release: v12.2.1 deposit: check permission and set disable tooltip for publish button Refactor: Enhance Warnings Handling * Add the missing translation tags. * Refactor the logic to display multiple warning messages, while allowing other files to upload, except for case where exceeding the file size limit. UI: Handle empty file uploads in FileUploader * Display a warning for empty files, indicating they won't be included and listing file names. * Feature controlled by `records-resources-allow-empty-files` config value. * Continue uploading other files while showing the warning message. serializers: ensure values are not None before access release: v12.1.1 fixtures: allow to load new vocab resource: fix add record to community * Removes response_handler to avoid allowing to process the response with the schema * closes inveniosoftware/invenio-app-rdm#2821 controls: refactored isDisabled function * Refactored isDisabled function in PublishButton and SubmitReviewButton components to check if all files are finished uploading before enabling the button. * Passed filesState instead of just the number of files to the function. * Added a check for non-finished files. * Added early returns to avoid unnecessary checks on possible large objects / arrays. 📁 invenio-requests (5.0.0 -> 5.1.0 🌈) release: v5.1.0 assets: optimise detection if timeline was updated timeline feed: render mathematical content in the request comments * closes CERNDocumentServer/cds-rdm#133 📁 invenio-theme (3.3.0 -> 3.4.0 🌈) release: v3.4.0 config: add MathJax support * closes CERNDocumentServer/cds-rdm#133 navbar: fix logo size in non-desktop devices css: updated table border style 📁 invenio-users-resources (6.0.0 -> 6.1.0 🌈) release: v6.1.0 users: services: Add can_read_all permission for admin search services: permissions: Use search_all config for admin user search CI: Switch To Centralised Workflows * Change branches type from string to array * Switch to centralised workflows 📁 invenio-vocabularies (5.0.2 -> 5.1.0 🌈) 📦 release: v5.1.0 funders: tune search boost for acronyms * Add and `acronym.keyword` field to the funders mapping. * Apply to funders the same field boosting as in affiliations. 📦 release: v5.0.3 services: skip index rebuilding
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
On adding a record to a community, we get an internal server error
The text was updated successfully, but these errors were encountered: