Skip to content

Conversation

@carlinmack
Copy link
Contributor

@carlinmack carlinmack commented Mar 12, 2025

❤️ Thank you for your contribution!

Description

On initial draft creation, no errors were being forwarded for empty files while on successive saves/updates they do.

Before and after of pressing save draft once (to create it). With my change the files not being there is also shown as an error, as both create and update check for the files

image

I recommend reviewing by reading the commits in order

  • first commit I add a test case that there should be an error on draft create
  • second commit I extract assign_files_enabled into a common function as the implementations slightly differed but should be the same
  • third commit I extract check_files_exist and add this to the create handler
  • fourth commit is responding to christoph's review

Checklist

Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

Frontend

Reminder

By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

  1. You license your contribution under the same terms as the current repository’s license.
  2. You agree that you have the right to license your contribution under the current repository’s license.

@carlinmack carlinmack changed the title Carlin patch 2 bug: if files.enabled, check whether there are files Mar 12, 2025
@carlinmack carlinmack changed the title bug: if files.enabled, check whether there are files files: added errors on draft create Mar 13, 2025
@carlinmack carlinmack mentioned this pull request Mar 13, 2025
10 tasks
@carlinmack carlinmack force-pushed the carlin-patch-2 branch 2 times, most recently from 8b89382 to 3a3ce70 Compare March 13, 2025 14:36
@carlinmack carlinmack marked this pull request as ready for review March 13, 2025 14:36
@carlinmack carlinmack moved this to In review 🔍 in Sprint Q1/2026 Mar 13, 2025
@0einstein0 0einstein0 requested review from kpsherva and zzacharo March 14, 2025 09:27
@carlinmack carlinmack added this to vNext Mar 27, 2025
@carlinmack carlinmack moved this to Triage in vNext Mar 27, 2025
@slint slint moved this to In progress in Sprint Q1/2026 May 14, 2025
@carlinmack carlinmack moved this from In progress to In review 🔍 in Sprint Q1/2026 Jun 30, 2025
@sakshamarora1
Copy link
Contributor

Do these errors show up in the UI? If so, can you please add a screenshot?
Not sure why the tests for 3.12 are failing as the logs aren't available anymore.

@carlinmack
Copy link
Contributor Author

carlinmack commented Jul 21, 2025

just rebased on master so the tests can run again, and added a picture

@slint slint moved this from In review 🔍 to Ready in Sprint Q1/2026 Aug 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready
Status: Triage

Development

Successfully merging this pull request may close these issues.

Deposit Form: revamp errors/warning in banner and form sections

3 participants