Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Aerogear 10237 #55

Merged
merged 10 commits into from
Jun 8, 2020
Merged

Conversation

AindriuBrennan
Copy link
Contributor

@AindriuBrennan AindriuBrennan commented May 13, 2020

Motivation

https://issues.redhat.com/browse/AEROGEAR-10237

PR for creating the view for Renaming an Application on the UnifiedPush Server.

This is the dialog for the old Admin UI

oldRename

This is the Rename dialog for the new Admin UI

newRename

@AindriuBrennan AindriuBrennan marked this pull request as ready for review May 14, 2020 10:49
created Update App name View, WIP rename function not working

Renames app correctly

Aligns buttons to the right, Buttons hidden unless hovered over.
Copy link
Member

@ziccardi ziccardi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was trying the PR, but any button I click always opens the 'delete application' dialog.

@ziccardi ziccardi self-requested a review June 3, 2020 16:32
Copy link
Member

@ziccardi ziccardi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I try to rename an application, the dialog opens giving me the change to edit the application name. However the proposed name is not editable, but I have to rewrite it from scratch.

Moreover, if I try to rename another application, it always propose me the last name I wrote.

Copy link
Member

@ziccardi ziccardi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The issue we had with the edit dialog, now happens with the delete dialog

@AindriuBrennan AindriuBrennan merged commit fe3fc48 into aerogear:PF4 Jun 8, 2020
@AindriuBrennan AindriuBrennan deleted the AEROGEAR-10237 branch June 8, 2020 08:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants