Skip to content
Snippets Groups Projects

Added template with configurable message

Merged Pavel Břoušek requested to merge github/fork/BaranekD/new_template into master

Created by: BaranekD

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
17 19 class Perunauthorize extends ProcessingFilter
18 20 {
19 21
22 const SERVICE_NAME_PLACEHOLDER = '%SERVICE_NAME%';
23 const SERVICE_EMAIL_PLACEHOLDER = '%SERVICE_EMAIL%';
24 const SP_METADATA = 'SPMetadata';
25 const ADMINISTRATION_CONTACT = 'administrationContact';
  • Pavel Břoušek
  • Pavel Břoušek
  • Pavel Břoušek
  • Pavel Břoušek
    Pavel Břoušek @433364 started a thread on commit 260ee06c
  • 2 2 All notable changes to this project will be documented in this file.
    3 3
    4 4 ## [Unreleased]
    5 #### Added
    6 Added another template with option of a configurable message
  • Author Contributor

    Created by: melanger

    Review: Dismissed

    Seems good to me. I recommend adding an explanation into the README, why we have this module and we don't use authorize from SSP. I think the only difference are the placeholders available in the message text.

  • Author Contributor

    Created by: BaranekD

    I've added a small change based on a left comment. It removed approval of this PR. Sorry for that.

  • Author Contributor

    Created by: melanger

    Review: Approved

  • Pavel Vyskočil approved this merge request

    approved this merge request

  • Ghost User approved this merge request

    approved this merge request

  • :tada: This MR is included in version 2.1.1 :tada:

    The release is available on GitLab release.

    Your semantic-release bot :package: :rocket:

  • Please register or sign in to reply
    Loading