Migrate deprecated bs-callout to alert component in pages

What does this MR do and why?

Migrate deprecated bs-callout to Pajamas::AlertComponent in app/views/projects/pages/*

It's a part of Pajamas adoption project

Changed according to migration guide available in next epic: &5213 (closed)

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Before	After

How to set up and validate locally

1. Enable external_http for gitlab pages in config/gitlab.yml:

development:
  pages:
    enabled: true
    access_control: false
    host: 127.0.0.1.nip.io
    port: 3010
    https: false # Set to true if you serve the pages with HTTPS
    external_http: ["1.1.1.1:80", "[2001::1]:80"] # If defined, enables custom domain support in GitLab Pages

2. Open some project
3. Goto Deploy -> Pages
4. Setup pages(if required).
5. Add new domain.
6. Verify warning messages. You can mock next methods to see warnings:

app/presenters/pages_domain_presenter.rb#L6

app/presenters/pages_domain_presenter.rb#L10

7. Modify account namespace name (to include dot .) to see the HTTPS with subdomains of subdomains warning message.

8. Disable pages for this project or mock it here app/models/project.rb#L2123 to see the pages are disabled warning message.

Related to #441112 (closed)

added Category:Source Code Management component:alert devopscreate groupsource code pajamasintegrate sectiondev typemaintenance workflowin dev labels

assigned to @ttaday-ext

added backend frontend labels

	2 Warnings
	e95e072d: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
	This merge request does not refer to an existing milestone.

	1 Message
	CHANGELOG missing: If this merge request needs a changelog entry, add the Changelog trailer to the commit message you want to add to the changelog. If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.

Reviewer roulette

Category	Reviewer	Maintainer
backend	@thutterer (UTC+1)	@pedropombeiro (UTC+1)
frontend	@thomasrandolph (UTC-7)	@zcuddy (UTC-6)
test for spec/features/*	@thutterer (UTC+1)	Maintainer review is optional for test for spec/features/*

Please check reviewer's status!

• Reviewer is available!
• Reviewer is unavailable!

Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.

If needed, you can retry the danger-review job that generated this comment.

Generated by Danger

added Community contribution label

added linked-issue label

This has been identified as a contribution from a Temporary Service Provider to GitLab. It is encouraged to seek review from at least one domain expert during the review process.

added Contractor Contribution label

added 1 commit

• e95e072d - Migrate deprecated bs-callout to alert component for gitlab pages

Compare with previous version

changed title from Migrate deprecated bs-callout to alert component in pages domains list to Migrate deprecated bs-callout to alert component in pages

changed the description

@thomasrandolph can you please review this MR? Thanks!

requested review from @thomasrandolph

approved this merge request

requested review from @zcuddy and removed review request for @thomasrandolph

@thomasrandolph, thanks for approving this merge request.

This is the first time the merge request has been approved. To ensure we don't only run predictive pipelines, and we don't break master, please start a new pipeline before merging.

Please wait for the pipeline to start before resolving this discussion and set auto-merge for the new pipeline. See merging a merge request for more details.

added pipeline:mr-approved label

approved this merge request

resolved all threads