Skip to content

Add Tags tutorial + small typo fix#78

Open
OmgRod wants to merge 23 commits intogeode-sdk:mainfrom
OmgRod:main
Open

Add Tags tutorial + small typo fix#78
OmgRod wants to merge 23 commits intogeode-sdk:mainfrom
OmgRod:main

Conversation

@OmgRod
Copy link

@OmgRod OmgRod commented Jun 29, 2025

1/2 made on my phone; enjoy!

@HJfod
Copy link
Member

HJfod commented Jul 7, 2025

These are already listed in the Popup tutorial, although I agree they should probably be moved to their own dedicated tutorial. Can you update this PR to change popup.md to link to the new tutorial?

@OmgRod
Copy link
Author

OmgRod commented Jul 8, 2025

These are already listed in the Popup tutorial, although I agree they should probably be moved to their own dedicated tutorial. Can you update this PR to change popup.md to link to the new tutorial?

ok

@OmgRod
Copy link
Author

OmgRod commented Aug 12, 2025

@HJfod Sorry for bothering you but I updated the docs (a while ago now lol). How is it?

HJfod
HJfod approved these changes Aug 13, 2025
View reviewed changes
tutorials/popup.md Outdated
Copy link
Member

@HJfod HJfod Aug 13, 2025

Choose a reason for hiding this comment

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

Should prolly say "are listed [here]" instead of available

RayDeeUx
RayDeeUx suggested changes Aug 13, 2025
View reviewed changes
RayDeeUx
RayDeeUx suggested changes Aug 17, 2025
View reviewed changes
Copy link
Contributor

@RayDeeUx RayDeeUx left a comment

Choose a reason for hiding this comment

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

found a few more things to change that i missed the first time around :P

@OmgRod
Copy link
Author

OmgRod commented Aug 19, 2025

found a few more things to change that i missed the first time around :P

Finished!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@HJfod HJfod HJfod approved these changes

+1 more reviewer

@RayDeeUx RayDeeUx RayDeeUx requested changes

Reviewers whose approvals may not affect merge requirements

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

@OmgRod @HJfod @RayDeeUx

Comments