-
Notifications
You must be signed in to change notification settings - Fork 73
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
Ck documentation structure changes #167
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for all this- I did a pass through for some copy edits and tweaks and pushed several updates. I tried to leave comments along the way to take notes but lost track of some of them!
I still have some more files to edit and review but want to pause here for now and will pick this up again in a bit. We shouldn't merge this quite yet
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a few open items here that it'd be good to discuss @NevilleS / @iamkelllly and then we can hopefully finalize this.
I do need to get a copy of this into general privacy-taxonomy repo shortly too so it'd be helpful to bottom out.
Changes to Fides documentation including:
Code Changes
Steps to Confirm
Pre-Merge Checklist
Description Of Changes
This was a consolidation of a rebased master with some conflicts, please review carefully to ensure no outstanding issues.