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.

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

[ASSET-9] Add info.json and tests #7

Merged
merged 3 commits into from
Jan 18, 2024
Merged

[ASSET-9] Add info.json and tests #7

merged 3 commits into from
Jan 18, 2024

Conversation

jormal
Copy link
Collaborator

@jormal jormal commented Jan 17, 2024

Pull Request

Description

info.json을 추가하고, 그 형식에 대한 test를 추가합니다.

Related issue: ASSET-9

Changes

Types of Changes (multiple options can be selected)

  • Create asset information
  • Update asset information
  • Delete asset information
  • Other Add settings

Checklist

  • Did you pass the tests?
  • Did you run the pre-process?
  • Have you added and run tests to validate the changes?

* ASSET-9 Add ethereum info

* ASSET-9 Add optimism info

* ASSET-9 Edit git ignore file

Signed-off-by: jormal <[email protected]>

* ASSET-9 Reformat the svg codes

Signed-off-by: jormal <[email protected]>

* ASSET-9 Fix to fit with JSON scheme

Signed-off-by: jormal <[email protected]>

* ASSET-9 Add bsc info

Signed-off-by: jormal <[email protected]>

* ASSET-9 Add polygon info

Signed-off-by: jormal <[email protected]>

* ASSET-9 Fix the property name

Signed-off-by: jormal <[email protected]>

* ASSET-9 Add Klaytn info

Signed-off-by: jormal <[email protected]>

* ASSET-9 Add Bifrost info

Signed-off-by: jormal <[email protected]>

* ASSET-9 Add Base info

Signed-off-by: jormal <[email protected]>

* ASSET-9 Add Arbitrum info

Signed-off-by: jormal <[email protected]>

* ASSET-9 Add Avalanche info

Signed-off-by: jormal <[email protected]>

* ASSET-9 Add protocol info

Signed-off-by: jormal <[email protected]>

* ASSET-9 Add unknown info and add asset name

Signed-off-by: jormal <[email protected]>

* ASSET-9 Add enum values and descriptions

Signed-off-by: jormal <[email protected]>

* ASSET-9 Fix optimism icon

Signed-off-by: jormal <[email protected]>

* ASSET-9 Fix URL of Bifrost Bridge  to apply @yjkellyjoo's comment

Signed-off-by: jormal <[email protected]>

* ASSET-9 Add protocol tag enum to apply @yjkellyjoo's comment

Signed-off-by: jormal <[email protected]>

---------

Signed-off-by: jormal <[email protected]>
* ASSET-9 Add model of information

Signed-off-by: jormal <[email protected]>

* ASSET-9 Add validation test for each model

Signed-off-by: jormal <[email protected]>

* ASSET-9 Fix wrong info

Signed-off-by: jormal <[email protected]>

* ASSET-9 Add CI workflow

Signed-off-by: jormal <[email protected]>

---------

Signed-off-by: jormal <[email protected]>
* ASSET-9 Refactor image preprocess

Signed-off-by: jormal <[email protected]>

* ASSET-9 Refine image validation

Signed-off-by: jormal <[email protected]>

* ASSET-9 Refine ID's regex pattern

Signed-off-by: jormal <[email protected]>

* ASSET-9 Chore edit

Signed-off-by: jormal <[email protected]>

---------

Signed-off-by: jormal <[email protected]>
@jormal jormal added asset_create Create the asset information settings Set this repository labels Jan 17, 2024
@jormal jormal self-assigned this Jan 17, 2024
@jormal jormal changed the title [Asset-9] Add info.json and tests [ASSET-9] Add info.json and tests Jan 17, 2024
@jormal jormal removed the asset_create Create the asset information label Jan 17, 2024
@jormal jormal marked this pull request as ready for review January 17, 2024 08:17
@jormal jormal requested a review from a team as a code owner January 17, 2024 08:17
@jormal jormal merged commit bdd21e7 into main Jan 18, 2024
1 check passed
@jormal jormal deleted the feature/ASSET-9 branch January 18, 2024 04:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
settings Set this repository
Development

Successfully merging this pull request may close these issues.

2 participants