Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow for metadata to be added to change history (supporting workflow bot changes) #464

Open
nigel-bmlt opened this issue Jun 28, 2022 · 3 comments

Comments

@nigel-bmlt
Copy link
Contributor

Tracking issue Re older thread: https://bmlt.slack.com/archives/C11N1P0J0/p1646516933789499

Intent is to allow for some metadata to be added when a change is submitted, with the intent of allowing third party interactors (like the workflow plugin) to add information about the change such as the originating user.

@nigel-bmlt
Copy link
Contributor Author

Have looked through the change schema and it appears 'change_description_text'(defined as SQL text type) is not used but is stored away on db read via $this->SetLocalDescription on a c_comdef_change object.

Would it be a valid use of this column to use it to store a free text change metadata value? It would save a schema update, but I'm not sure of the original intention of the column.

@nigel-bmlt
Copy link
Contributor Author

is this something we could add into the 3.x admin api as part of meeting api endpoint - make it an optional field for create and update calls

@jbraswell
Copy link
Collaborator

Possibly. I want to get 3.x out without any new features first, and then consider stuff like this once folks have started upgrading and we are confident we are stable.

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

No branches or pull requests

2 participants