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

Changes osmo.md to osmo.mdx, add FEE token DAO link #265

Merged
merged 3 commits into from
Jul 8, 2024

Conversation

Cordtus
Copy link
Contributor

@Cordtus Cordtus commented Jun 25, 2024

Adding fancy formatting

  • Adds Fee Token Whitelist Controller DAO
  • Changes page format from md >> mdx

Note

This should enable enhanced markdown and use the callout function as described in Docusaurus/Admonitions.

Additionally, fixes multiple spelling errors of "gauge" and "gauges" across several files, to keep the codespell check happy.

 - Adds Fee Token Whitelist Controller DAO
 - Changes page format from md >> mdx to enable enhanced markdown
Copy link

vercel bot commented Jun 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 4, 2024 0:57am
stage-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 4, 2024 0:57am

@Cordtus
Copy link
Contributor Author

Cordtus commented Jul 4, 2024

@mattverse or @JeremyParish69 I fixed a bunch of the spelling errors making the bot angry as well just now. Can you please review?

@JeremyParish69
Copy link
Collaborator

Looks OK to me.
The enforcement of a spell check seems a bit too strict since there is plenty of interaction with code-like naming...

@Cordtus
Copy link
Contributor Author

Cordtus commented Jul 5, 2024

Looks OK to me.
The enforcement of a spell check seems a bit too strict since there is plenty of interaction with code-like naming...

Yes I agree it is odd to have but it did catch a bunch.. Too bad it doesn't have a simpler way of correcting them.

@mattverse
Copy link
Member

Looks ok to me as well. Let's merge this for now, I think there was a way to add custom vocabs to CI

@mattverse mattverse merged commit 0d8c879 into osmosis-labs:main Jul 8, 2024
3 of 4 checks passed
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.

3 participants