Add user labels to user list and detail pages by stnguyen90 · Pull Request #457 · appwrite/console

stnguyen90 · 2023-06-02T01:02:18Z

What does this PR do?

Add user labels to user list and detail pages

Test Plan

Manual:

Users List

User Detail

without error

with error

Related PRs and Issues

Depends on:

Add a new labels attribute to the Users collection appwrite#5587
Updated Console SDK w/ User Labels

Have you read the Contributing Guidelines on issues?

Yes

vercel · 2023-06-02T01:02:25Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
console	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 27, 2023 0:17am
console-cloud	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 27, 2023 0:17am
console-next	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 27, 2023 0:17am

stnguyen90 · 2023-06-02T01:04:43Z

@chenparnasa, here's what I have so far for the user labels.

heyCarla

I think @chenparnasa left you some comments about this in Figma re: replacing the original helper text with the warning message if an invalid tag is added to the input field. Otherwise I like the idea of calling out the specific label in the warning!

chenparnasa · 2023-06-06T10:09:12Z

@stnguyen90 I left comments in Figma on how we should handle the helper texts :)

stnguyen90 · 2023-07-20T20:16:13Z

@TorstenDittmann, would you please review this when you get a chance?

stnguyen90 requested a review from chenparnasa June 2, 2023 01:03

vercelBotdeployed to Preview – console June 2, 2023 01:03 View deployment

stnguyen90 marked this pull request as ready for review June 2, 2023 01:04

stnguyen90 commented Jun 2, 2023
View reviewed changes

Comment threadsrc/routes/console/project-[project]/auth/+page.svelte Outdated

stnguyen90 commented Jun 2, 2023
View reviewed changes

Comment threadsrc/routes/console/project-[project]/auth/+page.svelte Outdated

stnguyen90 force-pushed the feat-user-labels branch from 32d8575 to ff2ae0aCompare June 2, 2023 19:12

vercelBotdeployed to Preview – console-cloud June 2, 2023 19:14 View deployment

vercelBotdeployed to Preview – console June 2, 2023 19:16 View deployment

stnguyen90 force-pushed the feat-user-labels branch from ff2ae0a to 684f01fCompare June 5, 2023 17:35

stnguyen90 requested review from TorstenDittmann and heyCarla June 5, 2023 17:36

vercelBotdeployed to Preview – console June 5, 2023 17:37 View deployment

vercelBotdeployed to Preview – console-cloud June 5, 2023 17:39 View deployment

heyCarla reviewed Jun 6, 2023
View reviewed changes

stnguyen90 force-pushed the feat-user-labels branch from 684f01f to ac80004Compare June 6, 2023 20:56

stnguyen90 requested a review from heyCarla June 6, 2023 20:56

vercelBotdeployed to Preview – console-cloud June 6, 2023 20:57 View deployment

vercelBotdeployed to Preview – console June 6, 2023 20:59 View deployment

stnguyen90 force-pushed the feat-user-labels branch from ac80004 to 385ff48Compare June 7, 2023 15:20

vercelBotdeployed to Preview – console-cloud June 7, 2023 15:21 View deployment

vercelBotdeployed to Preview – console June 7, 2023 15:23 View deployment

ArmanNik reviewed Jul 5, 2023
View reviewed changes

Comment threadsrc/routes/console/project-[project]/auth/user-[user]/updateLabels.svelte Outdated
Comment threadsrc/routes/console/project-[project]/auth/user-[user]/updateLabels.svelte Outdated
Comment threadsrc/routes/console/project-[project]/auth/user-[user]/updateLabels.svelte Outdated

stnguyen90 force-pushed the feat-user-labels branch from 385ff48 to a8d6f7bCompare July 5, 2023 19:42

stnguyen90 requested a review from ArmanNik July 5, 2023 19:42

vercelBotdeployed to Preview – console-cloud July 5, 2023 19:43 View deployment

vercelBotdeployed to Preview – console July 5, 2023 19:45 View deployment

ArmanNik approved these changes Jul 5, 2023
View reviewed changes

stnguyen90 mentioned this pull request Jul 17, 2023
Add a new labels attribute to the Users collection appwrite/appwrite#5587
Merged
2 tasks

stnguyen90 force-pushed the feat-user-labels branch from a8d6f7b to 82d6a80Compare July 20, 2023 19:53

vercelBotdeployed to Preview – console-cloud July 20, 2023 19:54 View deployment

vercelBotdeployed to Preview – console July 20, 2023 19:56 View deployment

stnguyen90 mentioned this pull request Jul 22, 2023
Create GH workflow to generate github metrics appwrite/.github#23
Merged

This was referenced Jul 25, 2023
PRs Opened in June 2023 appwrite/.github#37
Closed
PRs Opened in June 2023 appwrite/.github#43
Closed
PRs Opened in June 2023 appwrite/.github#49
Closed
PRs Opened in June 2023 appwrite/.github#54
Closed

stnguyen90 force-pushed the feat-user-labels branch from 82d6a80 to 126c6eaCompare July 26, 2023 00:07

vercelBotdeployed to Preview – console-next July 26, 2023 00:09 View deployment

vercelBotdeployed to Preview – console-cloud July 26, 2023 00:11 View deployment

vercelBotdeployed to Preview – console July 26, 2023 00:13 View deployment

Add user labels to user list and detail pages
d6d960f

stnguyen90 force-pushed the feat-user-labels branch from 126c6ea to d6d960fCompare July 26, 2023 17:11

vercelBotdeployed to Preview – console-next July 26, 2023 17:13 View deployment

vercelBotdeployed to Preview – console July 26, 2023 17:15 View deployment

vercelBotdeployed to Preview – console-cloud July 26, 2023 17:17 View deployment

TorstenDittmann approved these changes Jul 27, 2023
View reviewed changes

Merge branch 'main' into feat-user-labels
2b98065

vercelBotdeployed to Preview – console-next July 27, 2023 12:15 View deployment

vercelBotdeployed to Preview – console-cloud July 27, 2023 12:16 View deployment

vercelBotdeployed to Preview – console July 27, 2023 12:17 View deployment

eldadfux merged commit bd6a94d into mainJul 28, 2023

stnguyen90 deleted the feat-user-labels branch July 29, 2023 00:26

github-actionsBot mentioned this pull request Aug 1, 2023
PRs Closed in July 2023 appwrite/.github#62
Closed

stnguyen90 mentioned this pull request Aug 15, 2023
Update user labels based on design review #503
Merged

stnguyen90 mentioned this pull request Sep 5, 2023
Add label as a selectable permission #531
Merged

Navigation Menu

Uh oh!

Add user labels to user list and detail pages - #457

Add user labels to user list and detail pages#457
eldadfux merged 2 commits into
mainfrom
feat-user-labels

stnguyen90 commented Jun 2, 2023•
edited
Loading

Uh oh!

vercelBot commented Jun 2, 2023•
edited
Loading

Uh oh!

stnguyen90 commented Jun 2, 2023

Uh oh!

Uh oh!

Uh oh!

heyCarla left a comment

Uh oh!

chenparnasa commented Jun 6, 2023

Uh oh!

Uh oh!

Uh oh!

Uh oh!

stnguyen90 commented Jul 20, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Uh oh!

Conversation

stnguyen90 commented Jun 2, 2023• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What does this PR do?

Test Plan

Users List

User Detail

without error

with error

Related PRs and Issues

Have you read the Contributing Guidelines on issues?

Uh oh!

vercelBot commented Jun 2, 2023• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

stnguyen90 commented Jun 2, 2023

Uh oh!

Uh oh!

Uh oh!

heyCarla left a comment

Choose a reason for hiding this comment

Uh oh!

chenparnasa commented Jun 6, 2023

Uh oh!

Uh oh!

Uh oh!

Uh oh!

stnguyen90 commented Jul 20, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

stnguyen90 commented Jun 2, 2023•
edited
Loading

vercelBot commented Jun 2, 2023•
edited
Loading