Skip to content
Snippets Groups Projects

Security related documentation

Merged homotechsual requested to merge github/fork/seamuslee001/security_chapter into master

Created by: seamuslee001

@totten @seanmadsen open for some review, I think this meets the balance we were looking for but i like the concept of a Security chapter for reporting bug + ACL and permissions but then in the coding standards have a thing on doing this all correctly.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • homotechsual
  • homotechsual
  • homotechsual
  • homotechsual
  • Created by: seancolsen

    @seamuslee001 Want to take a look over these additions from me?

  • homotechsual
  • homotechsual
  • Created by: seamuslee001

    I think the changes make sense @seanmadsen i would like @totten to have a look over them too,

  • Created by: seancolsen

    I'll merge this on 2017-08-17 if we don't hear from anyone else.

  • Created by: seancolsen

    @seamuslee001 your changes from today look good :white_check_mark:

    Still on-track to merge on 2017-08-17 if no further comments.

  • Created by: MegaphoneJon

    I reviewed this today.

    There is overall a great addition, and there's nothing I see that would block a merge. If time permits, then some time down the line, I might consider rewriting the intro (use it to introduce the other pages a bit more) and there's some small punctuation things, but I think we merge now and improve later.

  • Created by: seamuslee001

    thanks @MegaphoneJon :-)

  • Created by: seancolsen

    Merging now since we got a review from @MegaphoneJon :tada:

  • Please register or sign in to reply
    Loading