Skip to content

Conversation

@etimr
Copy link
Contributor

@etimr etimr commented Jan 16, 2025

No description provided.

@etimr etimr added the ☠️ do not merge ☠️ Do not merge PR label Jan 16, 2025
@cla-assistant
Copy link

cla-assistant bot commented Jan 16, 2025

CLA assistant check
All committers have signed the CLA.

@cla-assistant
Copy link

cla-assistant bot commented Jan 16, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@etimr etimr marked this pull request as ready for review March 6, 2025 08:02
@etimr etimr changed the title WIP: Modification of Hierarchies Modification of Hierarchies Mar 6, 2025
@etimr etimr removed the ☠️ do not merge ☠️ Do not merge PR label Mar 6, 2025
@eugene-andreev
Copy link
Contributor

Some suggestions #446

@@ -0,0 +1,64 @@
ID;parent_ID;name;page;book_ID
f846b0b9-01d4-4f6d-82a4-d79204f62514;;Foreword;3;9b084139-0b1e-43b6-b12a-7b3669d75f02
Copy link
Contributor

Choose a reason for hiding this comment

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

Also here would be nice, if UUIDs would not look so similar everywhere, but that is optional.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll change them after the product day, together with the dynamic generation of the ui.hidden annotation for Contents entity.

@etimr etimr merged commit 5947b74 into main Mar 10, 2025
1 of 2 checks passed
@etimr etimr deleted the d070615-modify-hierarchy branch March 10, 2025 13:33
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.

5 participants