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

Fixes #724 #725

Merged
merged 2 commits into from
Nov 4, 2024
Merged

Fixes #724 #725

merged 2 commits into from
Nov 4, 2024

Conversation

dheid
Copy link

@dheid dheid commented Nov 4, 2024

New Pull Request

For more information, see the CONTRIBUTING guide.

Prerequisites

Please make sure you can check the following boxes:

  • I have read the CONTRIBUTING document
  • My code follows the code style of this project
  • All new and existing tests passed

Type(s) of Changes

Contribution Type

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue, please reference the issue id)
  • New feature (non-breaking change which adds functionality, make sure to open an associated issue first)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • I have updated the documentation accordingly
  • I have added tests to cover my changes

Description

See #724

@CLAassistant
Copy link

CLAassistant commented Nov 4, 2024

CLA assistant check
All committers have signed the CLA.

Copy link

github-actions bot commented Nov 4, 2024


Error: Error while trying to collect info before merging develop into develop.

Error: Command failed: npx lerna run build:release
lerna notice cli v8.1.7
lerna info ci enabled

    at ChildProcess.exithandler (node:child_process:422:12)
    at ChildProcess.emit (node:events:514:28)
    at maybeClose (node:internal/child_process:1105:16)
    at ChildProcess._handle.onexit (node:internal/child_process:305:5)

Generated by @jsenv/file-size-impact during check-bundle-size#11667980847 on 445bc89

Copy link
Contributor

@FlorianRappl FlorianRappl left a comment

Choose a reason for hiding this comment

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

LGTM - maybe also consider backwards compatibility to avoid running into the original problem in other / older browsers.

@FlorianRappl FlorianRappl added bug Something isn't working blazor Concerns the Blazor integration piral-blazor. labels Nov 4, 2024
@FlorianRappl FlorianRappl added this to the 1.8.0 milestone Nov 4, 2024
@dheid
Copy link
Author

dheid commented Nov 4, 2024

@FlorianRappl I agreed multiple times to the contributor license, but it still says "Contributor License Agreement is not signed yet.". Do you have an idea, what I can do?

@FlorianRappl
Copy link
Contributor

FlorianRappl commented Nov 4, 2024

@FlorianRappl I agreed multiple times to the contributor license, but it still says "Contributor License Agreement is not signed yet.". Do you have an idea, what I can do?

It either may be a problem with the bot or with the settings. Keep in mind that the bot is operating on your GitHub account, so it only has access to your registered email addresses. To verify that the commits are done by you the email you entered in your local git instance is compared to the email addresses of your GitHub account. Is the email matching? If not - you may need to enter that email address in your alias list.

(I guess this might be the problem @dheid as the commit is associated with another user @eliadheid - so maybe a different email address was used in the commits?)

@dheid
Copy link
Author

dheid commented Nov 4, 2024

Nice! That solved the license agreement issue. Thanks!

@FlorianRappl FlorianRappl merged commit 007e055 into smapiot:develop Nov 4, 2024
5 of 6 checks passed
@FlorianRappl
Copy link
Contributor

Awesome - thanks again for your contribution!

There are a couple of things that should enter in 1.8.0, but in general the version should come out this week.

@dheid
Copy link
Author

dheid commented Nov 4, 2024

Thank you so much for accepting this PR, merging it and for the release! This bugfix will help a lot of people, I suppose. It was a really nice experience contributing to Piral. Thx!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blazor Concerns the Blazor integration piral-blazor. bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants