Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Merged by Bors] - check if resource for asset already exists before adding it #3560

Closed
wants to merge 6 commits into from
Closed

[Merged by Bors] - check if resource for asset already exists before adding it #3560

wants to merge 6 commits into from
+31 −1

Commits on Jan 24, 2022

  1. check if resource for asset already exists before adding it

    @mockersf
    mockersf committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    d9cc188 View commit details
    Browse the repository at this point in the history

  2. Add unit test

    @aevyrie @mockersf
    aevyrie authored and mockersf committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    437ce41 View commit details
    Browse the repository at this point in the history

  3. add a doc comment

    @mockersf
    mockersf committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    def1d01 View commit details
    Browse the repository at this point in the history

  4. code review 

    Co-Authored-By: Aevyrie <[email protected]>
    @mockersf @aevyrie
    mockersf and aevyrie committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    9ccd3de View commit details
    Browse the repository at this point in the history

  5. put test in test module behind a cfg

    @mockersf
    mockersf committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    a2a710e View commit details
    Browse the repository at this point in the history

  6. update dependency

    @mockersf
    mockersf committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    0481f5e View commit details
    Browse the repository at this point in the history