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

✨ Rewrites + fix dep to hotfixes #3

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

floviolleau
Copy link

@floviolleau floviolleau commented Dec 9, 2024

This PR is for AndreMiras/edilkamin#4

It contains:

  • change from double quotes to simple quotes
  • indentation
  • transform MAC address before sending it to the API
  • add more types
  • upgrade some dependencies and re-arrange version numbers

Florent VIOLLEAU added 3 commits December 9, 2024 10:50
*/
async (jwtToken: string, macAddress: string) => {
const response = await axiosInstance.get<DeviceInfoType>(
`device/${transformMacAddress(macAddress)}/info`,
Copy link
Author

Choose a reason for hiding this comment

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

This is the main change. Else it is indentation + whitespaces

@floviolleau floviolleau changed the title ✨ change from double quotes to simple quotes + transform MAC address before sending it to the API + add more types + upgrade some dependencies and re-arrange version numbers ✨ Rewrites + fix dep to hotfixes Dec 9, 2024
@AndreMiras
Copy link
Owner

Thank you so much for taking the time to contribute to the library! I really appreciate the effort you put into this PR. 💖

While I can see you've made a variety of changes, it's generally best to focus on one specific improvement at a time in a pull request. For example, the changes to transformMacAddress() could potentially be meaningful and worth discussing further. However, combining that with changes to indentation rules, quoting, package versions, and other unrelated aspects makes it a bit challenging to review and merge.

For future contributions, I'd recommend keeping PRs small and focused on a single topic or feature. This helps ensure that the changes are easier to review and are more likely to get merged.

I'd love for you to amend the PR focusing just on the transformMacAddress() function (or any other specific improvement you'd like to propose). Let me know if you have any questions or need help with this—I'm here to help!

Thanks again for contributing, and I hope to see more of your work in the future! 🌟

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.

2 participants