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

Small link update request for the rucss_firewall_ips admin notice #6684

Closed
camilamadronero opened this issue May 30, 2024 · 2 comments · Fixed by #7086
Closed

Small link update request for the rucss_firewall_ips admin notice #6684

camilamadronero opened this issue May 30, 2024 · 2 comments · Fixed by #7086
Assignees
Labels
effort: [XS] < 1 day of estimated development time module: user interface quick win
Milestone

Comments

@camilamadronero
Copy link

Is your feature request related to a problem? Please describe.

We can improve the link used in the admin notice that's displayed when a firewall block is detected:

It seems a security plugin or the server's firewall prevents WP Rocket from accessing the SaaS features. IPs listed here in our documentation should be added to your allowlists

Describe the solution you'd like

We now have a doc specific to our IPs, so we could update it
From: https://docs.wp-rocket.me/article/1529-remove-unused-css#basic-requirements
To: https://docs.wp-rocket.me/article/1628-which-ip-do-i-need-to-allow-for-wp-rocket

Describe alternatives you've considered
NA

Additional context
It seems to be over here

@piotrbak piotrbak added the needs: documentation Issues which need to create or update a documentation label Jul 1, 2024
@piotrbak
Copy link
Contributor

piotrbak commented Aug 5, 2024

@Miraeld
Copy link
Contributor

Miraeld commented Oct 21, 2024

Scope a solution

In here

Change

			'rucss_firewall_ips'         => [
				'en' => [
					'id'  => '6076083ff8c0ef2d98df1f97',
					'url' => 'https://docs.wp-rocket.me/article/1529-remove-unused-css?utm_source=wp_plugin&utm_medium=wp_rocket#basic-requirements',
				],
				'fr' => [
					'id'  => '60d499a705ff892e6bc2a89e',
					'url' => 'https://fr.docs.wp-rocket.me/article/1577-supprimer-les-ressources-css-inutilisees?utm_source=wp_plugin&utm_medium=wp_rocket#basic-requirements',
				],
			],

to

	'rucss_firewall_ips'         => [
				'en' => [
					'id'  => '60ed8bde00fd0d7c253ff547',
					'url' => 'https://docs.wp-rocket.me/article/1628-which-ip-do-i-need-to-allow-for-wp-rocket?utm_source=wp_plugin&utm_medium=wp_rocket',
				],
				'fr' => [
					'id'  => '6246fe1a2ce7ed0fb091c543',
					'url' => 'https://fr.docs.wp-rocket.me/article/1690-quelles-adresses-ip-url-autoriser-pour-wp-rocket?utm_source=wp_plugin&utm_medium=wp_rocket',
				],
			],

Effort

XS

@Miraeld Miraeld added the effort: [XS] < 1 day of estimated development time label Oct 21, 2024
@Miraeld Miraeld self-assigned this Oct 23, 2024
@piotrbak piotrbak added this to the 3.17.3 milestone Oct 23, 2024
@hanna-meda hanna-meda self-assigned this Nov 4, 2024
@hanna-meda hanna-meda removed the needs: documentation Issues which need to create or update a documentation label Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort: [XS] < 1 day of estimated development time module: user interface quick win
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants