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

[NEW] Warning Popup reinforcing Zapier deprecation on Integrations Admin settings #27744

Merged
merged 24 commits into from
Feb 3, 2023

Conversation

Educg550
Copy link
Contributor

@Educg550 Educg550 commented Jan 12, 2023

Proposed changes (including videos or screenshots)

Warning being displayed on Zapier settings, above Zapier itself:

warning

Issue(s)

Steps to test or reproduce

On an Admin Workspace:

  1. Go to workspace administration settings
  2. Click on "Integrations" in the sidebar
  3. Go to the "Zapier" tab
  4. The new Popup should show

Further comments

A new message (in English) was written on the translation JSON used by the useTranslation() hook, and it still needs to be written on other languages as well. Note that the proposed warning popup it's following Fuselage's current design pattern, so it may differ from the initial request and from the storybook design SHOWN HERE.

@Educg550 Educg550 requested a review from a team as a code owner January 12, 2023 19:24
@yash-rajpal yash-rajpal added this to the 6.0.0 milestone Jan 12, 2023
@codecov
Copy link

codecov bot commented Jan 12, 2023

Codecov Report

Merging #27744 (dfdc190) into develop (9a25093) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #27744      +/-   ##
===========================================
+ Coverage    41.89%   41.92%   +0.02%     
===========================================
  Files          841      841              
  Lines        17624    17624              
  Branches      2071     2071              
===========================================
+ Hits          7384     7388       +4     
+ Misses        9963     9959       -4     
  Partials       277      277              
Flag Coverage Δ
e2e 41.92% <ø> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

yash-rajpal
yash-rajpal previously approved these changes Jan 13, 2023
yash-rajpal
yash-rajpal previously approved these changes Jan 18, 2023
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Jan 23, 2023
@kodiakhq kodiakhq bot merged commit 91600e1 into develop Feb 3, 2023
@kodiakhq kodiakhq bot deleted the tc-363 branch February 3, 2023 15:39
gabriellsh added a commit that referenced this pull request Feb 6, 2023
….Chat into matrixSearch

* 'feat/federation-feat-2' of github.com:RocketChat/Rocket.Chat: (31 commits)
  fix: removing file wrongly merged on update merge
  [NEW] Federation search public rooms in the Matrix network (#27434)
  Chore: Refactor other kinds of message - Phase 2 (#27927)
  Chore: improve some e2e selectors (#27936)
  [NEW] Warning Popup reinforcing Zapier deprecation on Integrations Admin settings (#27744)
  Chore: Notify Marketplace on App Install (#27925)
  i18n: Language update from LingoHub 🤖 on 2023-01-23Z (#27814)
  [FIX][ENTERPRISE] DDP streamer sending data to destroyed streams (#27929)
  Chore: Migrate cached collections to TypeScript (#27909)
  i18n: Language update from LingoHub 🤖 on 2023-01-09Z (#27705)
  [FIX] Reduce re-subscription on `notify-room/user-activity` stream event (#27911)
  [FIX] Unread Message count not displayed for new messages in Omni-Rooms (#27539)
  Chore: Improve service's licence check (#27872)
  [IMPROVE] Livechat Persian translation (#27825)
  Chore(deps): Bump thehanimo/pr-title-checker from 1.3.4 to 1.3.6 (#27884)
  Chore: use useQuery in place of deprecated useEndpointData (#27673)
  Chore: Update Some Composer Icons (#27862)
  Regression: Undefined default message context (#27870)
  Chore: Remove OS version for device management (#27786)
  [BREAK] Remove less theming (#27647)
  ...
@sampaiodiego sampaiodiego mentioned this pull request Feb 17, 2023
@sampaiodiego sampaiodiego mentioned this pull request Mar 9, 2023
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stat: ready to merge PR tested and approved waiting for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants