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

Rework eco + card components, sidebar 'For Users', add/mod user guides #52

Merged
merged 34 commits into from
May 10, 2024
Merged

Conversation

Cordtus
Copy link
Contributor

@Cordtus Cordtus commented May 6, 2024

important changes

Added text filter field on ecosystem/apps page.
This changes how we add app info. This no longer goes in ./components/EcosystemApps.tsx, it will go in ./data/appData.ts

New entries require another property "tags" which are used to search/filter the apps list as well as generate the text indicating what tags are currently possible to filter by.

// appData.ts

// .. imports .. 

// types definition
interface App {
    title: string;
    description: string;
    href: string;
    image: StaticImageData;
    tags: string[];
}
// list of apps
export const appData: App[] = [
    {
        title: "Compass",
        description: "The native DEX on SEI",
        href: "https://compasswallet.io/",
        image: compassLogo,
        tags: ["Wallet", "Native"]
    },
// additional entries

general structure changes

Reorganize left sidebar condense "for users" section and add, reorganize+condense other material, collapse all sidebar menus by default.

Cordtus added 4 commits May 3, 2024 09:43
	new file:   pages/getting-tokens.mdx
	deleted:    pages/getting-tokens/_meta.json
	deleted:    pages/getting-tokens/faucets.mdx
	new file:   pages/user-quickstart.mdx
	new file:   yarn.lock
	modified:   pages/_meta.json
	modified:   pages/index.mdx
	modified:   pages/interacting-with-sei/_meta.json
	renamed:    pages/block-explorers.mdx -> pages/interacting-with-sei/block-explorers.mdx
	renamed:    pages/getting-tokens.mdx -> pages/interacting-with-sei/getting-tokens.mdx
	renamed:    pages/setting-up-a-wallet.mdx -> pages/interacting-with-sei/setting-up-a-wallet.mdx
@Cordtus Cordtus requested a review from a team as a code owner May 6, 2024 14:53
goose and others added 7 commits May 6, 2024 23:12
- add FAQ page
- add User Guides folder
- move pages to User Guides folder
	modified:   pages/user-guides/block-explorers.mdx
	modified:   package.json
	modified:   pages/user-guides/block-explorers.mdx
	modified:   components/EcosystemApps/EcosystemApps.tsx
	new file:   data/appData.ts
	modified:   pages/user-guides/block-explorers.mdx
	modified:   tailwind.config.js
Copy link
Contributor Author

@Cordtus Cordtus left a comment

Choose a reason for hiding this comment

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

fixed any breaking changes regarding https://github.com/sei-protocol/sei-docs/pull/51/files

Copy link
Collaborator

@mj850 mj850 left a comment

Choose a reason for hiding this comment

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

Left some comments about small changes that might need to be made.

We should also update the title of this PR (Maybe: "Rework ecosystem component and add user guides").

components/Card/Card.tsx Outdated Show resolved Hide resolved
data/appData.ts Outdated Show resolved Hide resolved
data/appData.ts Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
pages/user-guides/getting-tokens.mdx Outdated Show resolved Hide resolved
pages/user-guides/setting-up-a-wallet.mdx Show resolved Hide resolved
pages/user-quickstart.mdx Outdated Show resolved Hide resolved
pages/user-guides/block-explorers.mdx Outdated Show resolved Hide resolved
pages/user-guides/linking-addresses.mdx Show resolved Hide resolved
pages/user-quickstart.mdx Outdated Show resolved Hide resolved
@Cordtus Cordtus changed the title Seiv2 Rework eco + card components, sidebar 'For Users', add/mod user guides May 9, 2024
Cordtus added 7 commits May 9, 2024 07:55
remove filename comment
added comment for filter method
add/remove comments
removed 'auto-build'
change 'warning' to 'info' callout
Remove page-breaking abandoned block
Removed unused theme elements
theme.config.tsx Outdated Show resolved Hide resolved
Copy link
Contributor Author

@Cordtus Cordtus left a comment

Choose a reason for hiding this comment

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

resolved all previous comments, added comments for any additional changes

Cordtus added 4 commits May 9, 2024 13:53
nfts2me
silo
stafi
superseiyanbot
replacing with proper dimensions
proper square dimensions
fixed pathing for last 3 images
Cordtus added 2 commits May 9, 2024 14:08
Quick final update  - re-order the user guide pages
Copy link
Collaborator

@mj850 mj850 left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for making these changes

Cordtus added 4 commits May 9, 2024 16:53
	modified:   components/EcosystemApps/EcosystemApps.tsx
	modified:   pages/user-guides/block-explorers.mdx
	modified:   pages/user-guides/getting-tokens.mdx
@mj850 mj850 merged commit b0bb6f5 into sei-protocol:seiv2 May 10, 2024
1 check passed
seiyan-writer pushed a commit that referenced this pull request Jan 9, 2025
#52)

* modified:   pages/_meta.json
	new file:   pages/getting-tokens.mdx
	deleted:    pages/getting-tokens/_meta.json
	deleted:    pages/getting-tokens/faucets.mdx
	new file:   pages/user-quickstart.mdx
	new file:   yarn.lock

* modified:   .gitignore
	modified:   pages/_meta.json
	modified:   pages/index.mdx
	modified:   pages/interacting-with-sei/_meta.json
	renamed:    pages/block-explorers.mdx -> pages/interacting-with-sei/block-explorers.mdx
	renamed:    pages/getting-tokens.mdx -> pages/interacting-with-sei/getting-tokens.mdx
	renamed:    pages/setting-up-a-wallet.mdx -> pages/interacting-with-sei/setting-up-a-wallet.mdx

* modified:   theme.config.tsx

* modified:   theme.config.tsx

* restructure For Users section

- add FAQ page
- add User Guides folder
- move pages to User Guides folder

* add FAQ, add linking addresses page, add images

* add to FAQ, refactor User quickstart

* modified:   .gitignore
	modified:   pages/user-guides/block-explorers.mdx

* modified:   .gitignore
	modified:   package.json
	modified:   pages/user-guides/block-explorers.mdx

* modified:   components/Card/Card.tsx
	modified:   components/EcosystemApps/EcosystemApps.tsx
	new file:   data/appData.ts
	modified:   pages/user-guides/block-explorers.mdx
	modified:   tailwind.config.js

* Update Card.tsx

* Adding Compass wallet to ecosystem
Resolves potential conflicts from
https://github.com/sei-protocol/sei-docs/pull/51/files

* Update .gitignore

* Update appData.ts

add would-be conflicting / deleted additions from https://github.com/sei-protocol/sei-docs/pull/51/files

* modified:   data/appData.ts

* Update Card.tsx

remove filename comment

* Update EcosystemApps.tsx

added comment for filter method

* Update appData.ts

add/remove comments

* Update package.json

* Update package.json

removed 'auto-build'

* Update block-explorers.mdx

change 'warning' to 'info' callout

* Update user-quickstart.mdx

Remove page-breaking abandoned block

* Remove unused elements - tailwind.config.js

Removed unused theme elements

* Add missing ecosystem images

nfts2me
silo
stafi
superseiyanbot

* Delete public/assets/ecosystem/superseiyanbot.jpeg

replacing with proper dimensions

* replace superseiyanbot image

proper square dimensions

* Update appData.ts

fixed pathing for last 3 images

* Update user-guides _meta.json

Quick final update  - re-order the user guide pages

* add compass logo

* modified:   .gitignore

* modified:   .gitignore

* modified:   components/Card/Card.tsx
	modified:   components/EcosystemApps/EcosystemApps.tsx
	modified:   pages/user-guides/block-explorers.mdx
	modified:   pages/user-guides/getting-tokens.mdx

---------

Co-authored-by: goose <[email protected]>
Co-authored-by: cordt-sei <[email protected]>
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.

4 participants