Skip to content

tweak: refactor code to follow firefox guidelines#31

Merged
warengonzaga merged 5 commits intoWeb3Philippines:devfrom
JBenedicto13:dev
Aug 5, 2023
Merged

tweak: refactor code to follow firefox guidelines#31
warengonzaga merged 5 commits intoWeb3Philippines:devfrom
JBenedicto13:dev

Conversation

@JBenedicto13
Copy link
Member

No description provided.

@warengonzaga warengonzaga self-requested a review July 16, 2023 14:13
@warengonzaga warengonzaga added the tweak Enhancements (Issue/PR) label Jul 16, 2023
@warengonzaga warengonzaga added this to the v0.1.4 🌱 milestone Jul 16, 2023
@warengonzaga warengonzaga linked an issue Jul 16, 2023 that may be closed by this pull request
@warengonzaga warengonzaga added the under review Under Review Status (Issue/PR) label Jul 16, 2023
@warengonzaga
Copy link
Contributor

Hey @JBenedicto13 let me take a look into your changes.

Copy link
Contributor

@warengonzaga warengonzaga left a comment

Choose a reason for hiding this comment

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

Just need a few things to change, after your changes, I'll test it in my local to see if the gas trackers are working properly. Thank you for your contribution!

@warengonzaga warengonzaga self-requested a review August 5, 2023 13:52
Copy link
Contributor

@warengonzaga warengonzaga left a comment

Choose a reason for hiding this comment

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

@JBenedicto13 All is good except the icons.

@warengonzaga warengonzaga self-requested a review August 5, 2023 14:46
Copy link
Contributor

@warengonzaga warengonzaga left a comment

Choose a reason for hiding this comment

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

@JBenedicto13 perfect! Looks good to me, merging the changes to the dev branch. Thank you for your contribution!

@warengonzaga warengonzaga merged commit 463afd1 into Web3Philippines:dev Aug 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tweak Enhancements (Issue/PR) under review Under Review Status (Issue/PR)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor script.js to pass firefox review

2 participants