Skip to content

Fix/8343 8345 8344 8346#1313

Merged
rdenarie merged 4 commits intofeature/maintenancefrom
fix/8343-8345-8344-8346
Feb 4, 2025
Merged

Fix/8343 8345 8344 8346#1313
rdenarie merged 4 commits intofeature/maintenancefrom
fix/8343-8345-8344-8346

Conversation

@rdenarie
Copy link
Member

@rdenarie rdenarie commented Feb 4, 2025

No description provided.

…- MEEDS-8345

Before this fix, the notes move drawer use $ for jquery in closing event
This is no more needed, so this commit removes it

Resolves Meeds-io/meeds#2876
…not disabled - MEED-8344

Before this fix, when moving a note, and choosing the parent of the current note, the move button is enabled
This commit add a check to let this button deactivated in this case.
In addition we update the target breadcrumb to empty it when the parent note is selected

Resolves Meeds-io/meeds#2879
Before this fix, when accessing to the note app of a space the first time, 2 request getNotePage are launched in parallel.
As the wiki not exists, 2 requests to create it are launched
It make the note app not accessible
This commit move the publish extension load after the NotesOverview is not instead of in parallel

Resolves Meeds-io/meeds#2880
Before this fix, when opening a note not published, there is 2 requests resulting in 404 error.
This create also errors logs in server log.
This problem is due to the fact that, as the note is not published, there is no metadata associated.
This issue ensure to take care of this case.

Resolves Meeds-io/meeds#2868
@rdenarie rdenarie requested review from Jihed525 and mkrout February 4, 2025 17:38
@rdenarie rdenarie enabled auto-merge (rebase) February 4, 2025 17:38
@github-actions github-actions bot added the partialCIBuild Perform Partial CI Build label Feb 4, 2025
@rdenarie rdenarie merged commit ebb9881 into feature/maintenance Feb 4, 2025
6 checks passed
@rdenarie rdenarie deleted the fix/8343-8345-8344-8346 branch February 4, 2025 17:43
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 4, 2025

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

Labels

partialCIBuild Perform Partial CI Build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments