[NV-2880] 🏘️ [Tenant] Update layout of the tenant card

LetItRock commented 1 year ago

What?

Creation dialogue on the right-side panel
Tenand card on the right-side panel

Note:

The tenancy feature is hidden behind the feature flag IS_MULTI_TENANCY_ENABLED. To enable this feature in the app you would need to hardcode the return value in the useIsMultiTenancyEnabled hook (don't commit this change).

Why? (Context)

We shipped some UI updates for multiple side cards in the product (such as provider card etc.), so we want to update the tenant card UI to make it consistent with other

Definition of Done

The with of the card is updated for both creation dialogue and a tenant card
Update button goes to the top of the card of a tenant
Updated at and Created at info go to the top of the card of a tenant
The icon "Create tenant" is added to a creation card (see creation dialogue)
The icon "Tenant" is added to a Tenant card (see tenant card)
Icon "X close" is updated
Update space between the header and the content on the tenant card
The hint on hover of the disabled button should display text in 2 lines (see mock)

Designs: Tenant list / Creation dialogue Tenant list / Creation dialogue

Substancia commented 1 year ago

Hey @LetItRock , I'd like to take this up. Can you please assign me? TIA!

ainouzgali commented 1 year ago

Hi @Substancia, assigning you

itssiddhantjain commented 1 year ago

Hello @LetItRock, please assign this issue to me as I have already worked on this kind of problem in past and has a great experience.

LetItRock commented 1 year ago

Hello @LetItRock, please assign this issue to me as I have already worked on this kind of problem in past and has a great experience.

hello @itssiddhantjain! 👋 thanks for your interest, but we have a person assigned to this ticket ;) if @Substancia won't be able to deliver the PR we can of course give it to you ;)

itssiddhantjain commented 1 year ago

@LetItRock thanks for letting me know, please assign me if possible in future.

Substancia commented 1 year ago

Hi @LetItRock , thanks for the opportunity, but unfortunately I will not be able to continue on this issue, apologies. Please feel free to assign this to @itssiddhantjain . If this remains open towards end of the month, I'll take this up then. Once again, thanks for the opportunity.

LetItRock commented 1 year ago

hey @itssiddhantjain! 👋 are you available to work on this ticket?

ItsCentric commented 1 year ago

Hey @LetItRock! If no one else is working on this I am willing to tackle this.

AkshitSinghal224 commented 1 year ago

hey @LetItRock i am right now working on deleting tenant issue and it is almost complete after that i can work on this one also as now i know the most of the code of tenant

AkshitSinghal224 commented 1 year ago

Hii, I have made all the UI changes in this issue should i raise a PR

itssiddhantjain commented 1 year ago

hey @itssiddhantjain! 👋 are you available to work on this ticket?

@LetItRock Yes. Kindly assign this issue to me so that I can send a PR.

davidsoderberg commented 1 year ago

@itssiddhantjain I have now assigned you :)

novuhq / novu