Skip to content

WIP: Improve whole logic for approval workflow - #8

Open
sagargg wants to merge 37 commits into
masterfrom
refractor
Open

sagargg wants to merge 37 commits into
masterfrom
refractor

Conversation

@sagargg

@sagargg sagargg commented Apr 8, 2024

Copy link
Copy Markdown
Member
  • Avoided complex logic on approval flow.
  • Removed extra metadata publish status and use the core state for storing publishing status.
  • Users cannot edit dataset that are already submitted for review.
  • Avoided complex templates overriding making minimal changes.
  • Added unit tests.

Note: This only works for CKAN > v2.10.x

@sagargg sagargg changed the title Improve whole logic for approval workflow WIP: Improve whole logic for approval workflow Apr 10, 2024
@ivanrvpereira

Copy link
Copy Markdown

Any chance of this being merged soon?

@sagargg

sagargg commented Sep 15, 2025

Copy link
Copy Markdown
Member Author

@gavram Do you think we can merge this ?

@valentinmirchev

Copy link
Copy Markdown

@sagargg @gavram do you have any updates on this PR? Should we expect a merge in the following days? Weeks? Months?

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