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

use spl-governance serializers #9

Merged
merged 2 commits into from
Oct 6, 2022
Merged

Conversation

arielsegura
Copy link
Contributor

@arielsegura arielsegura commented Oct 5, 2022

To move faster, let's just use spl-governance serializers under the hood.

See commits for details

@arielsegura

This comment was marked as outdated.

@arielsegura

This comment was marked as outdated.

setting GENERATE_SOURCEMAP=false due to facebook/create-react-app#11752

also using react-app-rewired due to facebook/create-react-app#12021 and solana-labs/oyster#538
"build": "react-scripts build",
"test": "react-scripts test",
"eject": "react-scripts eject"
"start": "GENERATE_SOURCEMAP=false react-app-rewired start",
Copy link
Contributor

Choose a reason for hiding this comment

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

There was a comment somewhere that says this is not super nice because you kind of like to have sourcemap later in dev mode. Later we can do another fix for that. I think it's just a "avoid sending errors/warning from this file/folder" i think.

Copy link
Contributor

@lucaslencinas lucaslencinas left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@arielsegura arielsegura merged commit b830ebb into main Oct 6, 2022
@arielsegura arielsegura deleted the use-spl-governance-serializers branch October 6, 2022 19:44
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