(domains) enhance required action modal content

Enhance DNS required actions and explanations
This commit is contained in:
Sabrina Demagny
2025-03-14 13:32:09 +01:00
parent 4c033d7262
commit db3185e16b
3 changed files with 12 additions and 8 deletions

View File

@@ -10,6 +10,7 @@ and this project adheres to
### Added ### Added
- ✨(domains) enhance required action modal content
- ✨(domains) add periodic tasks to fetch domain status - ✨(domains) add periodic tasks to fetch domain status
- 🧑‍💻(docker) add celery beat to manage periodic tasks - 🧑‍💻(docker) add celery beat to manage periodic tasks
- ✨(organization) add metadata field #790 - ✨(organization) add metadata field #790

View File

@@ -89,7 +89,7 @@ describe('<MailDomainView />', () => {
// Verify DNS configuration section // Verify DNS configuration section
expect( expect(
screen.getByText(/DNS Configuration Required:/i), screen.getByText(/Diagnostic status performed by the server/i),
).toBeInTheDocument(); ).toBeInTheDocument();
expect(screen.getByText(/imap.ox.numerique.gouv.fr./i)).toBeInTheDocument(); expect(screen.getByText(/imap.ox.numerique.gouv.fr./i)).toBeInTheDocument();
expect( expect(

View File

@@ -84,12 +84,12 @@ export const MailDomainView = ({ mailDomain, onMailDomainUpdate }: Props) => {
onClose={() => setShowModal(false)} onClose={() => setShowModal(false)}
title={t('Required actions on domain')} title={t('Required actions on domain')}
> >
<p> <pre>
{t( {t(
'The domain is currently in action required status. Please take the necessary actions to resolve those following issues.', 'The domain is currently in action required status. Please take the necessary actions to resolve those following issues.',
)} )}
</p> </pre>
<h3>{t('Actions required detail')}</h3> <h3>{t('Diagnostic status performed by the server')}</h3>
<pre> <pre>
{mailDomain.action_required_details && {mailDomain.action_required_details &&
@@ -105,7 +105,6 @@ export const MailDomainView = ({ mailDomain, onMailDomainUpdate }: Props) => {
</pre> </pre>
{mailDomain.expected_config && ( {mailDomain.expected_config && (
<Box $margin={{ bottom: 'medium' }}> <Box $margin={{ bottom: 'medium' }}>
<h3>{t('DNS Configuration Required:')}</h3>
<pre> <pre>
<div <div
style={{ style={{
@@ -113,7 +112,9 @@ export const MailDomainView = ({ mailDomain, onMailDomainUpdate }: Props) => {
overflowWrap: 'break-word', overflowWrap: 'break-word',
}} }}
> >
{t('Add the following DNS values:')} {t(
'You can resolve the above issues by applying the following configurations on your domain:',
)}
<ul> <ul>
{mailDomain.expected_config.map((item, index) => ( {mailDomain.expected_config.map((item, index) => (
<li <li
@@ -122,10 +123,12 @@ export const MailDomainView = ({ mailDomain, onMailDomainUpdate }: Props) => {
> >
{item.target && ( {item.target && (
<> <>
<b>{item.target.toUpperCase()}</b> -{' '} <b>{item.target}</b>{' '}
</> </>
)} )}
<b>{item.type.toUpperCase()}</b> {t('with value:')}{' '} <b style={{ color: 'purple' }}>
IN {item.type.toUpperCase()}
</b>{' '}
<span style={{ backgroundColor: '#d4e5f5' }}> <span style={{ backgroundColor: '#d4e5f5' }}>
{item.value} {item.value}
</span> </span>