Pull 2nd lot of develop changes into 0.9.0 #439

Merged

Vylpes merged 30 commits from develop into release/0.9.0 2025-05-18 11:05:56 +01:00

Conversation 0 Commits 30 Files changed 66 +3034 -772

Vylpes commented 2025-05-18 11:04:55 +01:00

Owner

Copy link

No description provided.

Vylpes added 30 commits 2025-05-18 11:04:55 +01:00

Create list effects command (#412) ... 3d143e7c73

# Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

#379

## Type of change

Please delete options that are not relevant.

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] This change requires a documentation update

# How Has This Been Tested?

Please describe the tests that you ran to verify the changes. Provide instructions so we can reproduce. Please also list any relevant details to your test configuration.

# Checklist

- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that provde my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged and published in downstream modules

Reviewed-on: #412
Reviewed-by: VylpesTester <tester@vylpes.com>
Co-authored-by: Ethan Lane <ethan@vylpes.com>
Co-committed-by: Ethan Lane <ethan@vylpes.com>

Use rsync -rvzP c4abf21013

Merge branch 'main' into develop 2b96c7c0d3

Use node v22 5db7cd9f11

Use node 20 1cace42983

Use node 20 ed52f3e3dc

Use node 20 d9d0243c3c

Implement ability to add images in the drop via an external url b8623398a6

0.8.3 c53e09f510

v0.8.3 ce0bc15c02

Merge branch 'main' into develop 7213703f66

Update id command from merge 3e81f8ce1d

Create use effect command (#419) ... a3248e978a

# Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

#380

## Type of change

Please delete options that are not relevant.

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] This change requires a documentation update

# How Has This Been Tested?

Please describe the tests that you ran to verify the changes. Provide instructions so we can reproduce. Please also list any relevant details to your test configuration.

# Checklist

- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that provde my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged and published in downstream modules

Reviewed-on: #419
Reviewed-by: VylpesTester <tester@vylpes.com>
Co-authored-by: Ethan Lane <ethan@vylpes.com>
Co-committed-by: Ethan Lane <ethan@vylpes.com>

Add friendly name and active effect to user effect list embed (#422) ... c6b458199c

# Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

- Add friendly name to list embed
- Add currently active effect to list

#379

## Type of change

Please delete options that are not relevant.

- [x] New feature (non-breaking change which adds functionality)

# How Has This Been Tested?

Please describe the tests that you ran to verify the changes. Provide instructions so we can reproduce. Please also list any relevant details to your test configuration.

# Checklist

- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that provde my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged and published in downstream modules

Reviewed-on: #422
Reviewed-by: VylpesTester <tester@vylpes.com>
Co-authored-by: Ethan Lane <ethan@vylpes.com>
Co-committed-by: Ethan Lane <ethan@vylpes.com>

Command to allow the user to buy more effects (#424) ... 5089ad6ab6

# Description

- Create a command to generate an embed for the user to be able to buy more effects
- This embed will contain the details about the effect as well as 2 buttons; Confirm and Cancel
- The confirm button will call the button event to:
    - Remove the currency from the user
    - Give the user the effect to their inventory
- The cancel button will just disable the buttons, so the user can't accidentally use it if they don't want to.

#381

## Type of change

Please delete options that are not relevant.

- [x] New feature (non-breaking change which adds functionality)

# How Has This Been Tested?

- Have created unit tests and tested locally

# Checklist

- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that provde my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged and published in downstream modules

Reviewed-on: #424
Reviewed-by: VylpesTester <tester@vylpes.com>
Co-authored-by: Ethan Lane <ethan@vylpes.com>
Co-committed-by: Ethan Lane <ethan@vylpes.com>

Fix remote image urls not showing up in image grids (#430) ... ad34cc7b7f

- Update the Image Grid helper to download images that are urls instead of trying (and failing) to load them locally

#425

Reviewed-on: #430
Reviewed-by: VylpesTester <tester@vylpes.com>
Co-authored-by: Ethan Lane <ethan@vylpes.com>
Co-committed-by: Ethan Lane <ethan@vylpes.com>

Fix droprarity and dropnumber commands (#431) ... 9b34bc65a7

- Fix the droprarity and dropnumber commands with the changes previously made regarding image urls
- Add a `CatchError` function to the app logger to easily send caught errors to the error logs
- Add choices to the droprarity command to make them easier to select

#429

Reviewed-on: #431
Reviewed-by: VylpesTester <tester@vylpes.com>
Co-authored-by: Ethan Lane <ethan@vylpes.com>
Co-committed-by: Ethan Lane <ethan@vylpes.com>

0.8.4 51a8e177ae

v0.8.4 211ef74410

Merge branch 'main' into develop 0eda06fe3d

Add error handler to series view command 1796f2519e

Merge branch 'main' into develop c3e328d538

Add catch to image grid generator 659906e562

Merge branch 'main' into develop 633e5fe95f

Fix linting c4e345cc46

Merge branch 'main' into develop f38451a7b4

Disable error for series helper temporarily 812e36329c

Merge branch 'main' into develop d8ed64e701

Fix linting 0669dfb0d0

Update the drop mechanic to take currency on drop instead of claim (#428) ... 730af871a0

# Description

- Update the drop mechanic to take currency on drop instead of claim
- Update the time until anyone can claim the card to 2 minutes
- Remove the restriction that the last drop can only be claimed by the user who dropped it
    - Now just controlled by the 2 minutes rule

#415

## Type of change

Please delete options that are not relevant.

- [x] New feature (non-breaking change which adds functionality)

# How Has This Been Tested?

Please describe the tests that you ran to verify the changes. Provide instructions so we can reproduce. Please also list any relevant details to your test configuration.

# Checklist

- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that provde my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged and published in downstream modules

Reviewed-on: #428
Reviewed-by: VylpesTester <tester@vylpes.com>
Co-authored-by: Ethan Lane <ethan@vylpes.com>
Co-committed-by: Ethan Lane <ethan@vylpes.com>

Vylpes merged commit f149aef444 into release/0.9.0 2025-05-18 11:05:56 +01:00

Vylpes referenced this pull request from a commit 2025-05-18 11:05:57 +01:00

Pull 2nd lot of develop changes into 0.9.0 (#439)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

blocked

  

duplicate

  

needs criteria

  

needs estimate

  

needs tests

  

question

  

requires documentation

  

type

admin

  

type

alert

  

type

bug

  

type

change

  

type

defect

  

type

dependencies

  

type

epic

  

type

spike

  

type

story

  

type

subtask

  

won't fix

No labels

blocked

duplicate

needs criteria

needs estimate

needs tests

question

requires documentation

type

admin

type

alert

type

bug

type

change

type

defect

type

dependencies

type

epic

type

spike

type

story

type

subtask

won't fix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: External/card-drop#439

No description provided.