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

Remove Units Per Second #1074

Merged
merged 12 commits into from
Jan 22, 2025
Merged

Remove Units Per Second #1074

merged 12 commits into from
Jan 22, 2025

Conversation

themacexpert
Copy link
Contributor

@themacexpert themacexpert commented Jan 14, 2025

Description

Removes unneeded units per second references and directs to new relative price where applicable.

Checklist

  • I have added a label to this PR 🏷️
  • I have run my changes through Grammarly

Corresponding PRs

Please link to any corresponding PRs here.

After Translation Requirements

Items to be Updated

Please list any of the items that will need to be added or deleted after the translations are done here.

@themacexpert themacexpert marked this pull request as draft January 14, 2025 23:28
@themacexpert themacexpert marked this pull request as ready for review January 15, 2025 03:30
@themacexpert themacexpert added A1 - Maintenance Major Pull request contains major updates to an existing page (i.e., adding a new section, reorgs, etc.) B0 - Needs Review Pull request is ready for review labels Jan 15, 2025
@themacexpert themacexpert requested a review from eshaben January 15, 2025 03:30
Copy link
Contributor

@eshaben eshaben left a comment

Choose a reason for hiding this comment

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

Nice!

@themacexpert themacexpert requested a review from eshaben January 17, 2025 19:44
@dawnkelly09 dawnkelly09 self-requested a review January 21, 2025 19:56
Copy link
Contributor

@dawnkelly09 dawnkelly09 left a comment

Choose a reason for hiding this comment

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

🚀 thank you!

@themacexpert themacexpert added B1 - Ready to be Merged Pull request is ready to be merged and removed B0 - Needs Review Pull request is ready for review labels Jan 21, 2025
@eshaben eshaben merged commit a413939 into master Jan 22, 2025
1 check passed
@eshaben eshaben deleted the themacexpert/removeups branch January 22, 2025 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A1 - Maintenance Major Pull request contains major updates to an existing page (i.e., adding a new section, reorgs, etc.) B1 - Ready to be Merged Pull request is ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants