Skip to content

Conversation

@printminion-co
Copy link

@printminion-co printminion-co commented Jan 23, 2026

Summary

  • Start MailHog: $ .dev/mailer/run
  • Run PR check .dev/container/check_release.sh pr-197 ncw artifactory
    • Abort via "N" on "[?] Do you want to run live setup scripts now? [y/N]"
    • Ensure SMTP is configured for local usage
      • or run dev-setup-mail.sh config
      • Run occ user:welcome admin
        • Observe: mailhog has email
    • Observe ncw_app is enabled
      • occ app:list --enabled | grep ncw_tools
    • Admin user is created and it hast an email set:
      • Observe occ user:info admin
    • Is overwrite.cli.url is set occ config:system:get overwrite.cli.url
    • Check job is added
      • occ background-job:list -c OCA\\NcwTools\\BackgroundJob\\PostSetupJob
        • Observe output 42 | OCA\NcwTools\BackgroundJob\PostSetupJob | 1970-01-01T00:00:00+00:00 | "admin"
    • Check job status
      • occ config:app:get ncw_tools post_install
        • Observe "INIT"
    • Run simulated job php cron.php
      • Observe no - email

Test - domain not accesible

  • domain overwrite.cli.url is set to "http://localhost:8080" (accessible)
    • occ config:system:set --value 'http://localhost:8080' -- overwrite.cli.url
  • Run simulated job php cron.php
    • Observe - you got email
  • Check job is removed
    • occ background-job:list -c OCA\\NcwTools\\BackgroundJob\\PostSetupJob
      • Observe - no job
  • Check job status
    • occ config:app:get ncw_tools post_install
      • Observe "DONE"

Checklist

@printminion-co printminion-co added this to the ncw-3 milestone Jan 23, 2026
@printminion-co printminion-co changed the title integrate ncw tools app integrate ncw-tools app Jan 23, 2026
@printminion-co printminion-co changed the title integrate ncw-tools app Test: integrate ncw-tools app Jan 23, 2026
@printminion-co printminion-co force-pushed the dev/integrate_ncw_tools_app branch from 77d3b5f to 0e312c6 Compare January 26, 2026 14:55
@printminion-co printminion-co changed the title Test: integrate ncw-tools app integrate ncw-tools app Jan 26, 2026
@bromiesTM bromiesTM force-pushed the rc/nsw-3 branch 3 times, most recently from a0e7c2d to 6bfe201 Compare January 29, 2026 13:55
@printminion-co printminion-co force-pushed the dev/integrate_ncw_tools_app branch 2 times, most recently from 0d4dcb0 to 8a8ff4d Compare January 30, 2026 14:42
tanyaka and others added 7 commits February 3, 2026 12:27
https://github.com/IONOS-Productivity/ncw-tools/tree/v1.0.0-3af182e

git submodule add -f  git@github.com:IONOS-Productivity/ncw-tools.git apps-external/ncw_tools
cd apps-external/ncw_tools
git checkout v1.0.0-3af182emain
cd -
git submodule update --init apps-external/activity
git add .gitmodules apps-external/ncw_tools

Signed-off-by: Tatjana Kaschperko Lindt <kaschperko-lindt@strato.de>
…ooks

Add InstallationCompletedEvent class in public API (OCP namespace) that
provides installation details: data directory, admin username, and admin
email. Event will be dispatched after successful installation.

Include comprehensive unit tests covering all event scenarios.

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Integrate event dispatching into Setup class:
- Inject IEventDispatcher dependency
- Dispatch InstallationCompletedEvent after successful installation
- Add Setup tests for event integration
- Update composer autoload for new class

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Implements build number injection system for production traceability:
- Modified version.php to read .buildnumber file and append build ID as $OC_Version[4]
- Updated build-artifact.yml workflow to create .buildnumber file with GitHub run ID
- Added .buildnumber to .gitignore for CI/CD generated files

This preserves the original semantic version while adding build metadata
for tracking specific CI/CD runs in production environments.

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
…on for .buildnumber existence before packaging)

https://github.com/IONOS-Productivity/ncw-config/releases/tag/de85871

Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…dd ncw_tools to always enabled apps list)

https://github.com/IONOS-Productivity/ncw-config/releases/tag/c4488d6

Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@printminion-co printminion-co force-pushed the dev/integrate_ncw_tools_app branch from 59995cf to 01104c2 Compare February 3, 2026 11:27
@printminion-co printminion-co merged commit 5171586 into rc/nsw-3 Feb 3, 2026
17 of 19 checks passed
@printminion-co printminion-co deleted the dev/integrate_ncw_tools_app branch February 3, 2026 15:45
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.

3 participants