Skip to content

Comments

Added Profile#198

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

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

Conversation

@Pdzly
Copy link
Contributor

@Pdzly Pdzly commented Nov 7, 2025

No description provided.

@Pdzly
Copy link
Contributor Author

Pdzly commented Nov 7, 2025

image

@Pdzly
Copy link
Contributor Author

Pdzly commented Nov 7, 2025

The "confused" dude is the role icon placement

Copy link
Member

@bristermitten bristermitten left a comment

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

const xpProgress = Math.min(Number(xp) / xpForNextLevel, 1);

// Draw XP bar background
canvas.fillStyle = "#444444";
Copy link
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
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 )

- Register new model in storage
- Add hasMany relation from DDUser to DDUserAchievements
- Minor formatting and whitespace cleanup in DDUser model
@Pdzly
Copy link
Contributor Author

Pdzly commented Nov 12, 2025

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