[D8] Crisis Mode [#2955142]

This module provides a predefined and configurable block which can be easily enabled and disabled in case of a crisis or any other situation which needs immediate communication. The block will not be visible after the module installation. The block only appears if the "Crisis Situation" is active. To activate a crisis you either you the Settings page (/admin/config/system/crisis_mode) or drush crisis-mode on. To ensure the block will be shown it will also clear the caches.

Project Page:
https://www.drupal.org/project/crisis_mode

Repo:
git clone --branch 8.x-1.x https://git.drupal.org/project/crisis_mode.git

Comment	File	Size	Author
#2	fixed_coding_standard_issues-2955142-2.patch	9.15 KB	Deepthi kumari
#2	coding_standard_errors-2955142-2.txt	5.52 KB	Deepthi kumari

Comments

Comment #1

22 March 2018 at 10:11

dalanz created an issue. See original summary.

Comment #2

Deepthi kumari commented 23 March 2018 at 12:05

Status	File	Size
new	coding_standard_errors-2955142-2.txt	5.52 KB
new	fixed_coding_standard_issues-2955142-2.patch	9.15 KB

Hi,

Fixed coding standard issues. Applied the patch. Needs review.

Comment #3

dalanz commented 26 March 2018 at 07:12

I applied the patch and retested the module. All works fine. Thank you very much.

Comment #4

dalanz commented 28 March 2018 at 08:25

I merged the patch #2 and the module is ready for testing.

git clone --branch 8.x-1.x https://git.drupal.org/project/crisis_mode.git

Thanks

Comment #5

altagrade commented 31 March 2018 at 00:03

The https://pareview.sh/pareview/https-git.drupal.org-project-crisis_mode.git is showing:

FILE: ...00000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/README.txt
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------
 14 | WARNING | Line exceeds 80 characters; contains 177 characters
 33 | WARNING | Line exceeds 80 characters; contains 81 characters
--------------------------------------------------------------------------

Time: 527ms; Memory: 6Mb

Comment #6

dalanz commented 3 April 2018 at 08:04

I fixed the line length issue in readme.txt and committed the changes back. Again ready for testing.

Thank you.

Comment #7

avpaderno

he/him

Italian

Brescia, 🇮🇹 🇪🇺

commented 18 August 2018 at 13:25

Priority:

Normal

» Critical

To the reviewers: Please set the priority to Normal after reviewing the project.

Comment #8

sleitner commented 21 August 2018 at 12:54

Priority:	Critical	» Normal
Status:	Needs review	» Reviewed & tested by the community

Automated Review

Review of the 8.x-1.x branch (commit c6cb8b8):

No automated test cases were found, did you consider writing PHPUnit tests? This is not a requirement but encouraged for professional software development.

This automated report was generated with PAReview.sh, your friendly project application review script.

Manual Review

Individual user account: Yes: Follows the guidelines for individual user accounts.
No duplication: Yes: Does not cause module duplication and/or fragmentation.
Master Branch: Yes: Follows the guidelines for master branch.
Licensing: Yes: Follows the licensing requirements.
3rd party assets/code: Yes: Follows the guidelines for 3rd party assets/code.
README.txt/README.md: Yes: Follows the guidelines for in-project documentation and/or the README Template.
Code long/complex enough for review: Yes: Follows the guidelines for project length and complexity.
Secure code: Yes: Meets the security requirements.
Coding style & Drupal API usage: no issues
The starred items (*) are fairly big issues and warrant going back to Needs Work. Items marked with a plus sign (+) are important and should be addressed before a stable project release. The rest of the comments in the code walkthrough are recommendations.

If added, please don't remove the security tag, we keep that for statistics and to show examples of security problems.

This review uses the Project Application Review Template.

Comment #9

avpaderno

he/him

Italian

Brescia, 🇮🇹 🇪🇺

commented 21 August 2018 at 17:32

Assigned:	Unassigned	» avpaderno
Status:	Reviewed & tested by the community	» Fixed

Thank you for your contribution!

I am going to update your account so you can opt into security advisory coverage now.
These are some recommended readings to help with excellent maintainership:

You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, also, for your patience with the review process.
Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

I thank all the dedicated reviewers as well.

Comment #10

4 September 2018 at 17:34

Status:

Fixed

» Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

[D8] Crisis Mode