Multiwoven / multiwoven

🔥🔥🔥 Open Source Alternative to Hightouch, Census, and RudderStack - Reverse ETL & Data Activation
https://squared.ai/multiwoven-reverse-etl
GNU Affero General Public License v3.0
1.14k stars 59 forks source link

User is enable to edit/update the connector credential data #404

Open xyfer17 opened 1 week ago

xyfer17 commented 1 week ago

Issue Description

Bug Summary

The user is unable to edit or update the credentials of an added connector after it has been successfully created.

Steps to Reproduce

  1. Log in to the application.
  2. Successfully create a source and destination connector.
  3. Attempt to edit the credentials of the created connector.

Screenshots

https://github.com/user-attachments/assets/2206954c-4efd-45a6-81c1-65a5b280a190

Actual Behaviour: User is enable to edit/update the connector credential data

Expected Behavior

The user should be able to edit and update the credentials for the added connector without any issues.

linear[bot] commented 1 week ago

FE-418 User is enable to edit/update the connector credential data

macintushar commented 1 week ago

@xyfer17 you need to test the connection before you can save changes

xyfer17 commented 1 week ago

@macintushar could you please check i have tried the test connection, even after that the details are haven't updated.

https://github.com/user-attachments/assets/044f4ab6-9644-4802-b45f-c57e9cca1ba6

macintushar commented 1 week ago

@xyfer17 you need to do the following steps in order:

  1. Change the details
  2. Test the connection
  3. Save the changes
xyfer17 commented 1 week ago

@macintushar It's working but can't we simplify these steps or add some kind of guide to help users understand the process better? This would improve the user experience and make it more intuitive.

macintushar commented 1 week ago

@xyfer17 I will bring this up with the team and we will update this. Meanwhile, I'll be closing this issue. Please stay tuned on Slack for updates on this.

macintushar commented 1 week ago

@xyfer17 can you raise a PR updating the docs explaining this exact process?

sumitd94 commented 3 hours ago

@xyfer17 can you please raise a PR to update the docs where these steps are clearly defined?