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.

Sign up for GitHub

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

feat: Add new warning messages in Developer and Integrations components #124

Merged
merged 3 commits into from
Feb 2, 2025
Merged

feat: Add new warning messages in Developer and Integrations components #124

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
4d60cd4
feat: add new warning messages
SaiKumar2121 Jan 21, 2025
76c3f0d
refactor: change loadingStatus from null to false
SaiKumar2121 Jan 21, 2025
66f32d6
fix: adjust alert message layout for mobile responsiveness
SaiKumar2121 Jan 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions lib/web/assets/css/app.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -945,4 +945,12 @@ body {
left: 2.5% !important;
right: 2.5% !important;
}
}
@media (max-width: 768px) {
.responsive-alert {
margin-left: 15%;
width: 70% !important;
padding: 10px;
font-size: 14px;
}
}
2 changes: 1 addition & 1 deletion lib/web/src/components/DataRetention.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ class DataRetention extends React.Component {

render () {
const logsTTLDays = this.state.logsTTLDays || null;
const loadingStatus = this.state.loadingStatus || null;
const loadingStatus = this.state.loadingStatus || false;

return (
<>
Expand Down
8 changes: 7 additions & 1 deletion lib/web/src/components/Developers.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ import { bindActionCreators } from 'redux';
import * as userActions from 'actions/userActions.js';

/* Ante UI */
import { Layout, Divider, Form, Input, Row, Col, Button, Card, Spin, message } from 'antd';
import { Layout, Divider, Form, Input, Row, Col, Button, Card, Spin, message, Alert } from 'antd';
import { UserOutlined, LockOutlined, DeleteOutlined } from '@ant-design/icons';

const { Content } = Layout;
Expand Down Expand Up @@ -157,6 +157,12 @@ class Developers extends React.Component {
<Col span={20} offset={2}>
<Content className='all-users'>
<Content>
{isAdmin &&
<Alert
description='To add a member to the developer list, enter their email and set a temporary password. Share these details with the developer and instruct them to log in and update their password.'
type='warning'
showIcon
/>}
<h1>Developers</h1>
{isAdmin &&
<Form layout='inline'>
Expand Down
9 changes: 8 additions & 1 deletion lib/web/src/components/Integrations.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import * as appActions from 'actions/appActions.js';
import * as userActions from 'actions/userActions.js';
import { LoadingOutlined, SlackOutlined, MailOutlined, CheckCircleFilled, PauseCircleFilled } from '@ant-design/icons';
/* Ante UI */
import { Layout, Divider, Form, Input, Row, Col, Button, Tabs, Result, Select, Spin, message } from 'antd';
import { Layout, Divider, Form, Input, Row, Col, Button, Tabs, Result, Select, Spin, message, Alert } from 'antd';

const { Content } = Layout;
const { TabPane } = Tabs;
Expand Down Expand Up @@ -422,6 +422,13 @@ class Integrations extends React.Component {
{(!emailDetails || editEmailDetails) &&
<>
<Form layout='horizontal'>
<Alert
className='responsive-alert'
description="If a developer's email does not appear in the recipient list, note that recipients can only be selected from the developer list. Please ensure the member is added to the developer list before selecting them as a recipient."
type='warning'
showIcon
/>
<br />
<Form.Item>
<Input className='input-class-margin' value={host} style={{ width: '400px' }} name='host' placeholder='SMTP Host' onChange={this.handleFormChange.bind(this)} autocomplete='off' /><br />
<Input className='input-class-margin' value={port} style={{ width: '400px' }} name='port' placeholder='SMTP Port' onChange={this.handleFormChange.bind(this)} autocomplete='off' /><br />
Expand Down