Skip to content

Conversation

@Chartman123
Copy link
Collaborator

@Chartman123 Chartman123 commented May 13, 2025

This fixes #2740 by adding checks if the form is archived and not currently in the process of "un-archiving"

Signed-off-by: Christian Hartmann chris-hartmann@gmx.de

@Chartman123 Chartman123 added bug Something isn't working php PHP related ticket labels May 13, 2025
@Chartman123 Chartman123 self-assigned this May 13, 2025
@Chartman123 Chartman123 added the 3. to review Waiting for reviews label May 13, 2025
@Chartman123 Chartman123 added this to the 5.2 milestone May 13, 2025
@Chartman123 Chartman123 requested review from Koc and susnux May 13, 2025 16:22
…reApiController

Signed-off-by: Christian Hartmann <chris-hartmann@gmx.de>
@Chartman123 Chartman123 force-pushed the fix/checkFormArchived branch from e4b7ffb to 5ae62c9 Compare May 13, 2025 16:26
@codecov
Copy link

codecov bot commented May 13, 2025

Codecov Report

Attention: Patch coverage is 52.94118% with 8 lines in your changes missing coverage. Please review.

Project coverage is 45.58%. Comparing base (bacb41f) to head (5ae62c9).
Report is 7 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2741      +/-   ##
============================================
+ Coverage     45.54%   45.58%   +0.03%     
- Complexity      977      984       +7     
============================================
  Files            79       79              
  Lines          3548     3565      +17     
============================================
+ Hits           1616     1625       +9     
- Misses         1932     1940       +8     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Chartman123 Chartman123 merged commit b0c6b2b into main May 14, 2025
54 checks passed
@Chartman123 Chartman123 deleted the fix/checkFormArchived branch May 14, 2025 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug Something isn't working php PHP related ticket

Projects

None yet

Development

Successfully merging this pull request may close these issues.

updateForm and ShareApiController don't check if form is archived

3 participants