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

Support Terra 2.0 #285

Merged
merged 21 commits into from
May 26, 2022
Merged

Support Terra 2.0 #285

merged 21 commits into from
May 26, 2022

Conversation

hanjukim
Copy link
Contributor

@hanjukim hanjukim commented May 26, 2022 •
edited
Loading

Description

This PR includes support for Terra 2.0 and Terra Classic by adding "isClassic" flag in the LCDClient constructor

e.g.

// connect to columbus-5 terra classic network
const client = new LCDClient({
  URL: 'https://columbus-lcd.terra.dev',
  chainID: 'columbus-5',
  isClassic: true  // *set to true to connect terra-classic chain*
});

isClassic and its related codes will be deprecated in next major version (v4.x) update.

Geoff Lee and others added 19 commits May 23, 2022 15:21
@hanjukim hanjukim requested a review from octalmage May 26, 2022 04:36
Merge pull request #286 from terra-money/chrore/rename-flag
f8c1404 
chore: remove comment and update wasm smart query
Vritra4
Vritra4 approved these changes May 26, 2022
View reviewed changes
Copy link
Contributor

@Vritra4 Vritra4 left a comment

Choose a reason for hiding this comment

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

LGTM

@hanjukim hanjukim merged commit 870e74d into main May 26, 2022
@hanjukim hanjukim deleted the v2 branch May 26, 2022 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Vritra4 Vritra4 Vritra4 approved these changes

@octalmage octalmage Awaiting requested review from octalmage

@simcheolhwan simcheolhwan Awaiting requested review from simcheolhwan

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hanjukim @Vritra4 @JSHan94