Skip to content

feat: add single sample project publish confirmation#1342

Merged
vcnainala merged 14 commits intodevelopmentfrom
issue-#1262
Feb 11, 2026
Merged

feat: add single sample project publish confirmation#1342
vcnainala merged 14 commits intodevelopmentfrom
issue-#1262

Conversation

@NishaSharma14
Copy link
Copy Markdown
Contributor

@NishaSharma14 NishaSharma14 commented Jan 23, 2026

Add modal to prompt users when publishing a project with only one sample, allowing them to choose between publishing as an individual sample or as a project. Includes tests for enableProjectMode parameter handling.

Key Changes:

  • Added a new modal component to prompt users when publishing single-sample projects
  • Implemented publishAsSample() and publishAsProject() methods to handle user choices
  • Added tests to verify the enableProjectMode parameter is correctly processed
  • Updated confirmation modal text to differentiate between sample and project publishing

fixes #13

image image

Add modal to prompt users when publishing a project with only one sample,
allowing them to choose between publishing as an individual sample or as
a project. Includes tests for enableProjectMode parameter handling.
publish as sample is already chosen.
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds a confirmation modal for single-sample project publishing, allowing users to choose whether to publish as an individual sample or as a project containing one sample. The feature enhances user experience by making the publishing choice explicit rather than implicit.

Changes:

  • Added a new modal component to prompt users when publishing single-sample projects
  • Implemented publishAsSample() and publishAsProject() methods to handle user choices
  • Added tests to verify the enableProjectMode parameter is correctly processed
  • Updated confirmation modal text to differentiate between sample and project publishing

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 4 comments.

File Description
tests/Feature/Project/PublishProjectTest.php Added 5 new tests covering single-sample publishing scenarios with enableProjectMode parameter
resources/js/Pages/Publish.vue Added single-sample modal UI, handler methods, and conditional logic to display modal for single-sample projects
public/build/manifest.json Auto-generated build artifact reflecting frontend changes

Comment thread resources/js/Pages/Publish.vue
Comment thread resources/js/Pages/Publish.vue Outdated
Comment thread tests/Feature/Project/PublishProjectTest.php
Comment thread tests/Feature/Project/PublishProjectTest.php
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.90%. Comparing base (a3e6931) to head (6472a90).
⚠️ Report is 1 commits behind head on development.

Additional details and impacted files
@@              Coverage Diff               @@
##             development    #1342   +/-   ##
==============================================
  Coverage          72.90%   72.90%           
  Complexity          2358     2358           
==============================================
  Files                209      209           
  Lines               9091     9091           
==============================================
  Hits                6628     6628           
  Misses              2463     2463           
Flag Coverage Δ
unittests 72.90% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

@vcnainala vcnainala merged commit 45bbc87 into development Feb 11, 2026
5 checks passed
@vcnainala vcnainala deleted the issue-#1262 branch February 11, 2026 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Suggest publishing as samples in case of projects with one sample

3 participants