Skip to content

Added Profile#198

Merged
bristermitten merged 8 commits into
TheDeveloperDen:masterfrom
Pdzly:feature/profile
Nov 18, 2025
Merged

Added Profile#198
bristermitten merged 8 commits into
TheDeveloperDen:masterfrom
Pdzly:feature/profile

Conversation

@Pdzly

@Pdzly Pdzly commented Nov 7, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@Pdzly

Pdzly commented Nov 7, 2025

Copy link
Copy Markdown
Contributor Author
image

@Pdzly

Pdzly commented Nov 7, 2025

Copy link
Copy Markdown
Contributor Author

The "confused" dude is the role icon placement

@bristermitten bristermitten left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A great start! Only a few small changes to make

Comment thread src/modules/user/user.ts Outdated
Comment thread src/modules/user/user.ts
const xpProgress = Math.min(Number(xp) / xpForNextLevel, 1);

// Draw XP bar background
canvas.fillStyle = "#444444";

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

could we make the colour of the xp bar based on the user's highest role colour? i think that would make it more visually interesting

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The background or foreground the bar ( green currently )

Comment thread src/util/canvas.ts Outdated
- Register new model in storage
- Add hasMany relation from DDUser to DDUserAchievements
- Minor formatting and whitespace cleanup in DDUser model
@Pdzly

Pdzly commented Nov 12, 2025

Copy link
Copy Markdown
Contributor Author
image

If the user has either the background color or just pitch black as role color it uses the developer den accent color

# Refactor starboard.listener.ts for improved formatting, error handling, and readability
- Enhanced code formatting and indentation
- Optimized error handling mechanisms
- Improved overall readability and consistency
```
@bristermitten
bristermitten merged commit 4157fab into TheDeveloperDen:master Nov 18, 2025
10 of 13 checks passed
@Pdzly
Pdzly deleted the feature/profile branch November 18, 2025 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants