-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Audit Trail module #4620
Comments
I don't know how the other company progressed on it. First to ship wins. |
And if we don't do the same work twice everybody wins :D. If you could get in touch with them somehow that would be great. |
Hi @Piedone I'm hoping to get started on this late next week! |
Great then @devlife! Let me know if you'd like to exchange notes or something. |
@devlife Did you work on this in the end? |
Hi @Piedone I have been super swamped at work and haven't even looked at this, unfortunately. I do have a client asking for it but am not sure when I'll have time. |
@Piedone would you be willing to share your thoughts on this issue and how you might approach it? I might be able to wrangle some time at work over the next few weeks. |
@barthamark @domonkosgabor please chime in. It's for a client so we'll need to check what we can disclose. |
We've started working on this feature with @domonkosgabor . In the following days Gabor or I will write some software design ideas that we can discuss. All of your help will be appreciated. |
That sounds great! I'd be super happy to help. |
I am thinking about implementing the Audit Trail module based on the module that can be found in Orchard 1, but I found some interesting behavior when trying to add an implementation of the Let's imagine a Page content type that has a
This will record three different kinds of Audit Trail event, but if you get a closer look, you will see that there is no need to record the action in the first step, because every time when we update/publish/etc. the page, we have the I know that there are scenarios available to see the parent or the root element of a given content item. You can use the However, there are different ways to find out that when we need to log the event or not:
My problem is that these are just different kinds of workarounds, maybe adding the stereotype to the JSON structure of the content item could be a solution, but I'm not sure about that either. What do you think about that? How can we filter what audit trail events should be recorded in terms of content items? |
Let's try the VersionedAsync event. |
I might have a need for this in an upcoming project. Does anyone have a repository where this is being worked on? I should be able to contribute code to the process. |
Not yet but we're working on it (again, since it's for a client, we can't just publish it right away). |
Still on it. |
@Piedone I am working on a client application that requires auditing as per Orchard 1 What is the status of this module, could we assist? |
Thanks, but not at the moment, this now is mostly an administrative issue that we're working through with the client of ours. We aim to have this ready in a few weeks. What we have BTW is mostly a port of the O1 module, adapted and updated to Orchard Core concepts and features. |
This is great news! Super excited to see it in action! |
Someone should make a betteropen source version with a restriction for this customer to not be able to use it. |
They're a nice customer and they agreed to open it up :). In the meantime those issues were resolved and it's up to us to make it ready for public release. We're on it. |
Just to say that in #6054 i introduced a Normally So |
Hi all. Just checking to see the status of the audit trail. I have several clients excited for it. |
I just noticed it's not on the roadmap. What I'd imagine for an Audit Trail module is similar to what's in Orchard 1, but let's discuss what we'd like to see if not a re-implementation of that.
What I've seen with our clients is that the most requested feature, and what I'd imagine as a first implementation, is the ability to track content changes:
What I'd imagine as needed eventually, but not necessarily right away:
I'd imagine, just in O1, this would be designed in an extensible way that user modules can define and record their own events (and other but built-in modules would contribute their own events in the same way).
Related: #506. What not do this time: #178.
@sebastienros here you mentioned that "some company is already working on it". Can you tell more?
The text was updated successfully, but these errors were encountered: