Skip to content
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

chore: remove unused user_expire module #665

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

lazysoundsystem
Copy link
Contributor

Refs: RWR-354

The ticket says "Module to remain disabled until further notice. Content Squad will review users manually on a regular basis for now." but no sign of it being used again soon and if so we can add it again. This PR removes the module completely to keep things more tidy.

@github-actions
Copy link

Coverage Report

Totals Coverage
Statements: 45.95% ( 1566 / 3408 )
Methods: 26.14% ( 23 / 88 )
Lines: 46.48% ( 1543 / 3320 )

@github-actions
Copy link

Build output

Composer Validate `success`

PHP Lint `success`

Docker Build `success`

Environment Setup `success`

Site Install `success`

PHP Code Sniffer `success`

Software Versions

```PHP 8.2.11 (cli) (built: Oct 6 2023 09:47:18) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.2.11, Copyright (c) Zend Technologies
with Zend OPcache v8.2.11, Copyright (c), by Zend Technologies
with Xdebug v3.2.1, Copyright (c) 2002-2023, by Derick Rethans
Composer version 2.6.5 2023-10-06 10:11:52
```

Pusher: @lazysoundsystem, Action: `pull_request`, Workflow: `Run tests``;

Copy link
Contributor

@attiks attiks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@lazysoundsystem lazysoundsystem merged commit 5a59e35 into develop Oct 19, 2023
2 checks passed
@lazysoundsystem lazysoundsystem deleted the RWR-354-remove-user_expire-module branch October 19, 2023 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants