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

fix(bridge-ui): renamed configuredCustomToken to configuredCustomTokens #15905

Merged
merged 5 commits into from
Feb 23, 2024

Conversation

arularmstrong
Copy link
Contributor

This PR addresses an inconsistency in the project's configuration file naming. The file in the config directory was originally named configuredCustomToken, while references elsewhere used configuredCustomTokens. To maintain consistency, I've renamed the file to configuredCustomTokens, aligning it with the Readme and other references.

@arularmstrong arularmstrong changed the title Renamed configuredCustomToken to configuredCustomTokens fix(bridge-ui-v2): renamed configuredCustomToken to configuredCustomTokens Feb 18, 2024
@KorbinianK KorbinianK closed this Feb 18, 2024
@KorbinianK KorbinianK reopened this Feb 21, 2024
Copy link
Contributor

@KorbinianK KorbinianK left a comment

Choose a reason for hiding this comment

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

@arularmstrong I have reopened the PR again as you were right after all. It seems due to some rebase the fix has been reverted. But the package has since moved to bridge-ui now, please rebase your PR to the latest main and then we can merge it.

@arularmstrong arularmstrong force-pushed the example-config-name-fix branch from 02f83f1 to f6ff7bf Compare February 22, 2024 09:54
@arularmstrong arularmstrong changed the title fix(bridge-ui-v2): renamed configuredCustomToken to configuredCustomTokens fix(bridge-ui): renamed configuredCustomToken to configuredCustomTokens Feb 22, 2024
@arularmstrong arularmstrong force-pushed the example-config-name-fix branch from f6ff7bf to 5737418 Compare February 22, 2024 10:05
@arularmstrong
Copy link
Contributor Author

@KorbinianK the PR is rebased with latest main

Copy link
Contributor

@KorbinianK KorbinianK left a comment

Choose a reason for hiding this comment

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

@arularmstrong You need to remove the incorrectly named ones as well please.

@davidtaikocha davidtaikocha added this pull request to the merge queue Feb 23, 2024
Merged via the queue into taikoxyz:main with commit a9f60b8 Feb 23, 2024
2 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