aws-amplify / amplify-backend

Home to all tools related to Amplify's code-first DX (Gen 2) for building fullstack apps on AWS
Apache License 2.0
167 stars 56 forks source link

Optional Amplify Console UI changes #1454

Open hackmajoris opened 4 months ago

hackmajoris commented 4 months ago

How did you install the Amplify CLI?

No response

If applicable, what version of Node.js are you using?

No response

Amplify CLI Version

11.0.5

What operating system are you using?

Mac, Windows, Linux

Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.

No

Describe the bug

The new Amplify Console design should be optional and not mandatory. Some features such as: build stages, Cypress integration are gone, and it's inadmissible to push such changes without noticing them in advance.

Expected behavior

The new UI is optional in respect for the projects which uses Amplify Gen 1 and the migration to Gen 2 is impossible because of unstableness of Amplify.

Reproduction steps

  1. Check Amplify Console.

Project Identifier

No response

Log output

``` # Put your logs below this line ```

Additional information

No response

Before submitting, please confirm:

dorutiuga commented 4 months ago

In the new UI, there are some issues when trying to open a previous deploy.

https://github.com/aws-amplify/amplify-cli/assets/107680344/6b288fbf-9ed1-4a74-9f0a-7312325a1339

dorutiuga commented 4 months ago

Another issue: Openning custom headers on an application is not possible 328423643-6fcb4d6f-4749-45e8-a231-60c0962d29d2

then, trying to navigate to all apps is not working. It changes only the url, staying on the same page

dorutiuga commented 4 months ago

Another issue: image this is randomly appearing on different pages

swaminator commented 4 months ago

Another issue: Openning custom headers on an application is not possible 328423643-6fcb4d6f-4749-45e8-a231-60c0962d29d2

then, trying to navigate to all apps is not working. It changes only the url, staying on the same page

@dorutiuga are you using Safari?

Jay2113 commented 4 months ago

Another issue: Openning custom headers on an application is not possible 328423643-6fcb4d6f-4749-45e8-a231-60c0962d29d2

then, trying to navigate to all apps is not working. It changes only the url, staying on the same page

@dorutiuga we are unable to reproduce this behavior. Can you share some reproduction steps for this issue? Also, are you consistently observing it?

TenetMax commented 4 months ago

Another issue: Openning custom headers on an application is not possible 328423643-6fcb4d6f-4749-45e8-a231-60c0962d29d2

then, trying to navigate to all apps is not working. It changes only the url, staying on the same page

I am also seeing this issue. As far as I can tell the steps to reproduce are simply to have had custom headers set up prior to the new UI having rolled out. As mentioned, simply clicking the link to view the headers results in this error:

Screenshot 2024-05-07 at 4 48 47 PM Screenshot 2024-05-07 at 4 48 52 PM

The console error itself is:

TypeError: JSON.parse(...).map is not a function

This maps to the following code:

            var t = {
                customHeaders: JSON.parse(e).map((function(e) {

This is happening because e here is an object, not an array:

Screenshot 2024-05-07 at 4 52 27 PM

It seems that after parsing it is expecting to match the application, then key in on .customHeaders

Jay2113 commented 4 months ago

@TenetMax @dorutiuga Can you share your Amplify app id on which you are observing the custom headers issue?

dorutiuga commented 4 months ago

@TenetMax @dorutiuga Can you share your Amplify app id on which you are observing the custom headers issue?

Sure, d6myjg10lg61f

hackmajoris commented 4 months ago

Navigating through the Amplify console with Safari is nearly impossible. After clicking on different sections (such as attempting to view previous builds and checking the logs), the UI freezes, and only refreshing the page helps.image

TenetMax commented 4 months ago

@TenetMax @dorutiuga Can you share your Amplify app id on which you are observing the custom headers issue?

d1rmomz2jsmjjk

gongora92 commented 4 months ago

I can't change the production branch in the new interface. I don't get any error, it just does not update

https://github.com/aws-amplify/amplify-backend/assets/44581880/701ed176-96d5-46e7-98b2-99d083223061

hackmajoris commented 4 months ago

@Jay2113 Is there a reason to not have the new Amplify Console UI as optional, at least for a while?. Every AWS service provided an option to choose between new UI and old one. Why Amplify can't?

alacret commented 1 month ago

@Jay2113 @hackmajoris (With a lot of frustration on me) This UI is not as good as the old one, I wish there were an option to use the old one

hackmajoris commented 1 month ago

@Jay2113 @hackmajoris (With a lot of frustration on me) This UI is not as good as the old one, I wish there were an option to use the old one

After few months it gets better, but still miss the old one