Skip to content

Hotfix/solr setup improvements - #607

Merged
rubenvdlinde merged 4 commits into
developmentfrom
hotfix/solr-setup-improvements
Sep 16, 2025
Merged

Hotfix/solr setup improvements#607
rubenvdlinde merged 4 commits into
developmentfrom
hotfix/solr-setup-improvements

Conversation

@rubenvdlinde

Copy link
Copy Markdown
Contributor

Proper config naam

- Updated setupSolr() to create tenant-specific configSet instead of base configSet
- Modified ensureTenantCollectionExists() to use tenant-specific configSet
- Added getTenantConfigSetName() method for consistent configSet naming
- Removed ensureBaseCollectionExists() method as only tenant collections are needed
- Updated validation to check tenant-specific configSet and collection
- Updated error reporting and logging to reference tenant-specific resources
- All SOLR resources now follow tenant-specific naming pattern (e.g., openregister_nc_f0e53393)
…s tracking

- Added visual progress bar showing completed vs total steps
- Display detailed error information with primary error, error type, and operation
- Show SOLR configuration used during failed setup attempts
- Added troubleshooting steps section with numbered action items
- Enhanced step timeline with individual step status and timestamps
- Improved visual styling with color-coded progress indicators
- Failed steps now highlight in red with detailed error context
- Added error metadata display (error type, operation, configuration)
- Better responsive design for mobile devices
- Maintains backward compatibility with existing setup response format
- Added details about tenant-specific configSets and collections
- Documented enhanced error reporting with progress tracking
- Listed new UI improvements in setup dialog
- Fixed multi-tenant isolation improvements
@rubenvdlinde
rubenvdlinde merged commit 1157e2a into development Sep 16, 2025
0 of 12 checks passed
remko48 pushed a commit that referenced this pull request Sep 16, 2025
Hotfix/solr setup improvements

  Restored beta version to 0.2.7-beta.72
rubenvdlinde added a commit that referenced this pull request Dec 28, 2025
rubenvdlinde added a commit that referenced this pull request Aug 6, 2026
Carries ConductionNL/nextcloud-vue#608, which closes #607: the flow editor no
longer lets a user press Save before the store has a flow to save.

The window was real and wide. `emptyFlow()` has `name: ''`, only `open('new')`
supplies the default, and `open()` ran behind `await GET /api/flows` — a list
a blank flow does not need — while the sidebar was already rendered and Save
already enabled. Saving in that window posted `name: ""` and the API answered
400 "A flow needs a name." A refused save rendered nothing at all, so the user
saw the button flicker and no more. The same late `open('new')` also reset the
flow, wiping a step already placed on the canvas.

Verified against this bundle, in the window itself: clicking Save 120ms after
opening a blank flow — where a 400 was previously reproducible 9 times in 10 —
returns 201 with `name: "New flow"` and the route advances to the new uuid.

`vue` is pinned back to ^3.5.18 by hand again. `npm install` rewrites it to
^3.5.0 to match the new package's own range, which contradicts the `overrides`
entry, and CI's npm 10.8.2 refuses that with EOVERRIDE while local npm 11
accepts it silently. `npx npm@10.8.2 ci --dry-run` is clean with the range
restored. Worth automating; for now it is a hand check on every bump of this
package.
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.

1 participant