Skip to content

Migrate all messages antd for mui#1694

Merged
thesocialdev merged 4 commits intostagefrom
Change-all-messages-antd-for-mui
Jan 14, 2025
Merged

Migrate all messages antd for mui#1694
thesocialdev merged 4 commits intostagefrom
Change-all-messages-antd-for-mui

Conversation

@LuizFNJ
Copy link
Collaborator

@LuizFNJ LuizFNJ commented Jan 8, 2025

Description

In this PR, I created a message component to replace the native message from Ant Design, thus changing all the places where messages (success, error, warning, info) are used. The message component uses a Node.js class and provides a way to emit and listen to events in a decoupled manner, making it possible to use it globally across the platform, just like the Ant Design message. A natural style change was made, so it doesn't affect the user experience

style change:
Screenshot from 2024-12-24 17-55-45Screenshot from 2024-12-24 19-36-21

Testing

All types of messages that appear on the site, whether error, success, warnings, or info, were changed, so I need to trigger all these situations to test the messages properly. Apart from that, I took the opportunity to change col, row, typography, and avatar, which I will separate here to make it easier.

API components

Various components

closes #1497 #1496 #1495 #1494 #1493 #1492 #1491 #1490 #1489 #1488 #1487 #1501 #1500 #1499 #1542 #1686 #1687 #1688 #1690 #1511

@LuizFNJ LuizFNJ requested a review from thesocialdev as a code owner January 8, 2025 22:02
@LuizFNJ LuizFNJ force-pushed the Change-all-messages-antd-for-mui branch from f182cf6 to 2100c2c Compare January 11, 2025 15:35
@sonarqubecloud
Copy link

@thesocialdev thesocialdev merged commit 02e944c into stage Jan 14, 2025
7 checks passed
@thesocialdev thesocialdev deleted the Change-all-messages-antd-for-mui branch January 14, 2025 17:10
@LuizFNJ LuizFNJ changed the title Change all messages antd for mui Migrate all messages antd for mui Jan 22, 2025
@thesocialdev thesocialdev mentioned this pull request Jan 26, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: QA

Development

Successfully merging this pull request may close these issues.

Migrate component sentenceApi.ts - Import { message }

2 participants