Skip to content

fix: library form submitting state#4594

Merged
mariojsnunes merged 4 commits intomasterfrom
fix/library-form-submitting-state
Feb 3, 2026
Merged

fix: library form submitting state#4594
mariojsnunes merged 4 commits intomasterfrom
fix/library-form-submitting-state

Conversation

@mariojsnunes
Copy link
Contributor

PR Checklist

  • - Unit and/or e2e tests for the changes that have been added (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix

What is the new behavior?

Fixes submitting state for drafts, prevents submitting multiple times by making the buttons disabled.

Does this PR introduce a DB Schema Change or Migration?

  • No

@mariojsnunes mariojsnunes requested a review from a team as a code owner February 3, 2026 15:54
@github-project-automation github-project-automation bot moved this to 🆕 Backlog in Core Team - BAU list Feb 3, 2026
@mariojsnunes mariojsnunes merged commit 4340326 into master Feb 3, 2026
2 checks passed
@mariojsnunes mariojsnunes deleted the fix/library-form-submitting-state branch February 3, 2026 16:04
@github-project-automation github-project-automation bot moved this from 🆕 Backlog to ✅ Done in Core Team - BAU list Feb 3, 2026
@cypress
Copy link

cypress bot commented Feb 3, 2026

onearmy-community-platform    Run #8480

Run Properties:  status check failed Failed #8480  •  git commit ba3e39fbd6: fix: news for save draft
Project onearmy-community-platform
Branch Review fix/library-form-submitting-state
Run status status check failed Failed #8480
Run duration 07m 29s
Commit git commit ba3e39fbd6: fix: news for save draft
Committer Mário Nunes
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 4
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 89
View all changes introduced in this branch ↗︎

Tests for review

Failed  src/integration/library/write.spec.ts • 1 failed test • ci-chrome

View Output Video

Test Artifacts
[Library] > [Create a project] > [By Authenticated] Test Replay Screenshots Video
Flakiness  profile.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test Artifacts
[Profile] > [By User] > [User directed to own profile] Test Replay Screenshots Video
Flakiness  settings.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test Artifacts
[Settings] > Can create member Test Replay Screenshots Video
Flakiness  news/write.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test Artifacts
[News.Write] > [Create a news item] > [By Authenticated] Test Replay Screenshots Video
Flakiness  map.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test Artifacts
[Map] > [Shows expected pins] Test Replay Screenshots Video

@onearmy-bot
Copy link
Collaborator

🎉 This PR is included in version 2.105.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants