UserTeamsControllerService New User Request Unit tests

Related EPIC https://github.com/Aiven-Open/klaw/issues/2427
Use cases that require additional unit tests.
getNewUserRequests

Unauthoized missing ADD_EDIT_DELETE_USERS permission
No Requests found in the DB
Multiple Requests found in the DB
One Request Found in the DB

Check in Requests that return that password is not set, that the team is set as expected, and the tenant is set as expected

approveNewUserRequests

Unauthoized missing ADD_EDIT_DELETE_USERS permission
Where authentication type is Database password is set
Where authentication type is not Database e.g. LDAP password is not set
Try to create approve a user with a username that does not match the default pattern ("^[a-zA-Z0-9]{3,}$")
Create a user that use their email address as a username.
verify email is sent to new user
verify dbHandle is called to save user to database

declineNewUserRequests

Unauthoized missing ADD_EDIT_DELETE_USERS permission
Verify the the database function is called on declining the request
Verify exception is thrown when error occurs.

getRegistrationInfoFromId

Request User that does not exist
Request user that does exist, ensure expected fields are correct

Hey @khatibtamal I have added this new issue to tackle some of the New User Request functions that would greatly benefit from additional unit tests!
Thanks

@aindriu-aiven sure I will look into this.

@aindriu-aiven pr is open.

@aindriu-aiven pr is open.

Great thanks a million!


ci: rename master to main for jobs

Codecov Report

Merging #435 (72fb095) into main (dd634ac) will decrease coverage by 0.16%.
The diff coverage is n/a.

:exclamation: Current head 72fb095 differs from pull request most recent head 4b1515d. Consider uploading reports for the commit 4b1515d to get more accurate results

@@            Coverage Diff             @@
##             main     #435      +/-   ##
==========================================
- Coverage   60.30%   60.13%   -0.17%
==========================================
  Files          41       41
  Lines        1194     1194
  Branches      135      135
==========================================
- Hits          720      718       -2
- Misses        409      410       +1
- Partials       65       66       +1

Impacted Files
Coverage Δ

src/app/shared/add-device/add-device.component.ts
88.63% <0.00%> (-4.55%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd634ac...4b1515d. Read the comment docs.


KAN VS MLP for Motor Imagery BCI classification

I want to share this repo where we evaluate ACC performance between KAN and MLP in four scenarios: RAW Multiclass, RAW Binary, CSP Multiclass, and CSP Binary.
Draw your conclusions (see ours in the report).
Best!
https://github.com/JARS29/KAN_MLP_MI-BCI

Which report?
I could only find a notebook, inside "results", which shows approximately the same results, except for one which shows a bit more overfitting on KAN.


Move info about zLUX Plug-ins from Using to Extending

The content in https://zowe.github.io/docs-site/user-guide/usingmvd.html#using-zlux-application-plug-ins   is what I would expect to find in  "Extending Zlux" section.  In "Extending Zowe CLI", we talk about the plug-ins that are available for CLI and how to install them. So this would fit.
So, to address this issue I suggest:

Take "Using zLUX application plug-ins" section and move the content to "Extending zLUX" under a new heading named "Install Plug-ins to zLUX" or similar.
Expand on the topic with a procedure - it does not currently tell the user HOW to install a plug-in, it just lists the available plug-ins.
The content that you have under Extending zLUX right now could be moved to a new section for developers. I think that the User Guide should focus on the end-user of these applications who will be using them in daily work and possibly installing plug-ins to extend the functionality.  Then we should have a separate "Develop for Zowe" section that has your content about "creating zLUX plugins and apps" and the other subtopics that describe the framework for developing apps. We have similar content for Imperative Framework that doesn't fit under Tutorials but is still meant for developers. https://github.com/gizafoundation/imperative/wiki/Command-Definition-&-Processing

When the user downloads and installs the pax file, the application plug-ins (Hello World, IFrame, ZOS Subsystems, TN3270, VT Terminal ) are installed and ready to be used.  Users will need information about these apps.
The demonstration apps (Hello World and IFrame) are for people who will be creating applications, so I can see these apps moving to the developer area. I think that we might still want to include a brief description so when people open the desktop, they aren't wondering "What are those for?"
Yes, I agree that we should move the Extending zLUX topics to a developer section.

Ah, I see. I agree with you then - we should keep this "Using zLUX app plug-ins" section as you suggest.
I think we could clarify the section in the intro there where it says "Application plug-ins are applications that you can use to access the mainframe and to perform various tasks. Developers can create application plug-ins using a sample application as a guide.".      My suggestion is to replace with the following (feel free to adjust) - ""zLUX application plug-ins are applications that you can use to access the mainframe and perform various tasks through the web browser interface. Developers can create application plug-ins using a sample application as a guide.. The following plug-ins are installed to zLUX by default:"
I also suggest listing the subsystems, 3270, and VT apps first in the list since they are really the core apps, followed by the sample apps.
Sounds good on the developer section comment.
Eventually we will probably want a "Install plug-ins to zLUX" topic under "Extending zLUX" topic... although maybe that would belong in an Admin guide since that install happens on the back-end (while CLI users instill plugins themselves on their own PC, so it makes sense in the user guide).

Can we close this issue now?


Variable does not exist. Handing down variables from parent templates

Question
This is the first time I have used Sprig, so I might be understanding something fundamental  and I am not sure this is really a sprig issue. But perhaps a failing of my twig and handling of variables and how these are handed down and used in templates. But I am hoping you might be able to shed some light, or offer some guidance here.
My twig structure is based on this article Andrew Welch of NYStudio107.
So I am building a website that uses different colour scheme per service (section) and in order to control the tailwind classes. I have build a LUT (look up table) in a global-variables.twig file.

For example a /atoms/testimonial.twig component:

      {{ svg('@icons/bx/line/bx-heart.svg')|attr({ class:"w-32 h-32 flex-shrink-0 fill-current" }) }}

      {{ entry.testimonial }}

          — {{ entry.title }}

        {% if craft.app.request.pathInfo != 'testimonials' %}
          More Testimonials
        {% endif %}

If I was to include this on the main pages/testimonials.twig template. it would render fine and the classes would be replaced from the LUT table. I have no issues with using this directly in a template or within another include within another component even. The variables are accessible and are replaced.
However, if I try and use the testimonial include inside a sprig component I get the error:
Variable "class" does not exist.
See this as an example:
sprig/sprig-testimonial.twig
{# Sets a default value if not defined by the `s-vars` attribute on the button #}
{% set offset = offset ?? 0 %}

{% set entries = craft.entries.section('testimonials').offset(offset).limit(limit).all() %}

{% for entry in entries %}
  {% include '_includes/_atoms/testimonial.twig' %}
{% endfor %}

{% if entries %}
  {# Increments `offset` by the value of `limit` and swaps itself out on click #}
  <button sprig s-vars="offset: {{ offset + limit }}"
          s-target="this" s-swap="outerHTML"
  >
    Load another
  </button>
{% endif %}

then within the main testimonials template:
pages/testimonials.twig
{{ sprig('_includes/_sprig/sprig-testimonials.twig', {'limit': 1 }) }}
{{ sprig.script }}

ow can I ensure the global variables are available and accessible for including a component within the twig component.
I use the testimonial component individually within a matrix field. But I also want to include it on a full testimonials page where the user can click to load more. I don't really want to have separate components that are the the same design.
Any help will be greatly appreciated.
Thanks for reading this long thread.
Terry Upton

Variable "class" does not exist.

That's because the variable doesn't exist. Sprig renders the template sprig/sprig-testimonial.twig on its own, and it does not inherit from the global-variables.twig file.
I'm not sure if it'll work with Sprig, but can try adding something like:
{% extends "_layouts/global-variables.twig" %}

To your sprig/sprig-testimonial.twig template.

Thanks @khalwat - this does in fact work and has solved my issue.
@bencroker no side affects with this approach i assume?

No side effects, it's just Twig ™


Support decoupled shading from reservoirs

This is disabled by default because it clearly produces a lot of fireflies on the default scene:

getting this to work with pairwise MIS turns out to be challenging


ci: fix coverage SHA

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.965%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
463

Relevant Lines:
579

💛  - Coveralls


Tooltip "regionTooltip" prop cannot take multiple classes

Current Behavior
When trying to style a tooltip on a button, using multiple classes on a regionTooltip prop breaks things and yields Uncaught (in promise) DOMException: Failed to execute 'add' on 'DOMTokenList': The token provided ('text-sm uppercase') contains HTML space characters, which are not valid in tokens. in console.
Steps To Reproduce
<button class="btn variant-filled-primary" use:tooltip={{ content: 'Hello', regionTooltip: 'text-sm uppercase' }}>I'm a button</button>
Using a single regionTooltip class yields the expected result.
Anything else?
My apologies if I'm just making a dumb mistake here!

@noblepen8723 Just FYI we're in the midst of changes to how Utility features like Tooltips work:
https://github.com/skeletonlabs/skeleton/issues/922
It's very likely these will work in a different fashion in the next release. I'll circle back on this and try to test and confirm soon. But keep an eye out for the next release.

Please refer to: https://github.com/skeletonlabs/skeleton/issues/922
As of the last couple days we've completely replaced Menus/Tooltips with a new "Popups" feature, which you can preview here:
https://skeleton-docs-git-chore-utilities-polish-pass-skeleton-labs.vercel.app/utilities/popups
Tooltips now operate much more like Menus, where the popup UI itself is created locally (like a sibling to the trigger). This means all styles can be set directly on the UI you're generating. It's a lot simpler and more intuitive. I think folks are going to enjoy it.
Given this, the issue in the original post is no longer relevant, so I'm closing this out. Expect to see these changes alongside the new release on Tuesday (US time) next week! Thanks!


Link to guide for adding new scaler docs

Relates to https://github.com/kedacore/keda-docs/issues/51

@tomkerkhove the commit is not signed

That's what you get when you use the UI 😅


TypeScript error of 'prop-types'

Here is the error:
node_modules/@rmwc/base/index.d.ts:3:10 - error TS2305: Module '"/node_modules/@types/prop-types/index"' has no exported member 'default'.

3 export { default as PropTypes } from 'prop-types';
           ~~~~~~~

I think change export { default as PropTypes } from 'prop-types'; to the below can fix this problem:
import * as PropTypes from 'prop-types';
export { PropTypes };

Thanks for reporting, will put out a patch today

Apparently export {* as PropTypes} from 'prop-types' is considered experimental syntax? Anyways, implemented your fix, appears to have resolved the issue. Thanks again for reporting, will be live on NPM shortly as 4.0.1.


Failure Cannot read property 'Authentication' of null

setting is ok ?

click the push button then back a error

push is success on shell command

@pch18 what version of Portainer are you using?

1.19.2

confirmed, it's related to the structure of the registry url. we look only for a domain, while the url in this issue includes a path.
another issue is that when using localhost, it ignores the registry and tries to push to dockerhub.
look in https://github.com/portainer/portainer/blob/6b05a3588142a9279f4d363c8a679d3380243caf/app/portainer/services/api/registryService.js#L69
and in https://github.com/portainer/portainer/blob/develop/app/docker/helpers/imageHelper.js#L7-L20
it always separates the string based on the first slash. a quick fix will be to leave that as it is, and when search for repository by url (RegistryHelper.getRegistryByURL(data, imageDetails.registry);) we will do a check for existing substring instead of equality check.

@pch18, I made a fix, can you try running portainer with the image portainerci/portainer:fix2388-find-registry-by-url-segment

same question.

Thank you, the problem is solved in the test version from portainerci/portainer:fix2388-find-registry-by-url-segment

This problem still exists in the new version 1.22, doesn't it ❓
I'm using Harbor 😋

portainer\app\portainer\helpers\registryHelper.js

portainer\app\docker\helpers\imageHelper.js

This takes effect when modified

portainer-1.22.0\app\portainer\helpers\registryHelper.js

portainer-1.22.0\app\docker\helpers\imageHelper.js

After modification, it works

@chenyu-lau nice, would you mind opening a PR ?

@deviantony I created a new pull request #3086，But I don't know if I'm doing the right thing

Thanks, we'll add it into our contribution review pipeline.
cc @itsconquest

Awesome, I will push this forward. Great job @chenyu-lau! :+1:

As an update to this issue, the bug will be addressed via development of issue 3122 linked above

have seen this with NULL environment variable also

Closed via #3122


how to renew tls cert without taking down service

Looking for some guidance that I haven't been able to figure out from the documentation. When cert-manager renews a certificate, does it have to redo the http acme challenge?
I used this guide https://github.com/ahmetb/gke-letsencrypt to get my initial certificate, but my mistake was that I deleted cert-manager. So my certificate has been valid, but now I need to renew it. It's not clear how to renew my current certificate. Do I need to delete my old cert, temporarily take down my service, and generate a new cert?
I'm guessing there may be a way to launch cert-manager and renew my current one, but I can't seem to figure out how to do that properly.
Any advice would be much appreciated, thank you.

This is possible by either changing the certificate's spec.secretName, or through mangling the spec.renewBefore value to manually force a renewal.
Further down the line, I'd love to create a kubectl plugin such as kubectl cm certificate renew example-com-tls
Marking as help wanted as we don't currently have someone triaged to work on this.
/kind documentation
/help

Thanks for the advice, I'll play around with those and see what I can do

Was able to renew the certificates without taking down my site. Was a big struggle but I learned quite a bit about Kubernetes resources and Ingresses! The key was adding two annotations to my current site's Ingress:
certmanager.k8s.io/cluster-issuer: letsencrypt-prod
certmanager.k8s.io/acme-http01-edit-in-place: "true"
(full file is attached)
That first annotation just tells cert-manager to talk to the letsencrypt-prod issuer (which we'll configure next)
That second annotation is what allows you to do the challenge without taking down your website. Essentially it gives cert-manager the ability to complete the certificate challenge using your current ingress, rather than having to make a whole new one
example.txt
Then I also changed my secretName in the Ingress to something else, applied the Ingress, then changed it back again and applied the Ingress again (I'm not sure if that helped or not).
I also re-configured the clusterissuers:
export EMAIL=
curl -sSL https://rawgit.com/ahmetb/gke-letsencrypt/master/yaml/letsencrypt-issuer.yaml |     sed -e "s/email: ''/email: $EMAIL/g" |     kubectl apply -f-
I also deleted cert-manager and reinstalled it:
helm del --purge cert-manager
helm install --name cert-manager --version v0.6.0 --namespace cert-manager stable/cert-manager


the keyword bg  is not working in marp for vscode

when i use marp in my vscode, the picture will disappear when i use 'bg' keword ,such as :
![bg](/assets/1.png)
if it is:
![](/assets/1.png)
the picture will enmergence

TL;DR: Try using the relative path from Markdown file, ![bg](./assets/1.png) or ![bg](assets/1.png), instead of the absolute path ![bg](/assets/1.png).
Resolving absolute path is context-aware and unstable. We always recommend to use relative paths. e.g. Even if using ![](), /assets/1.png cannot render an image, when opening Markdown without workspace.

When rendering the preview of Markdown that is located in workspace, VS Code tries manipulating image paths specified in the regular Markdown images. For example, /assets/1.png will convert to https://file+.vscode-resource.vscode-cdn.net/xxx/xxxx/xxx/assets/1.png (/xxx/xxxx/xxx is the path for current workspace). This URL is virtual path, and required to render images in the preview panel correctly.
![bg]() syntax is Marp specific and not covered by VS Code's Markdown preview. So paths for background images will not manipulate. /assets/1.png will become url('/assets/1.png').

thank you ！it works！


Entering feed that is not selected

In the following example newsbeuter enters a feed that is not currently selected in the feed list.
config:
show-read-feeds no

urls:
https://github.com/emacs-mirror/emacs/releases.atom
https://github.com/vim/vim/releases.atom

Start newsbeuter: newsbeuter -c cache.db -C config -u urls
R to reload all feeds.
Down to select Vim feed.
Enter to open Vim feed.
A to mark all articles read.
q to go back to feeds view.
Note that only the Emacs feed is shown as the Vim feed is hidden by show-read-feeds no.
Enter to open the selected Emacs feed. Vim feed is shown instead.

Set toggle-show-read-feeds to hide read feeds

toggle-show-read-feeds is a key. I assume you mean show-read-feeds setting.
I checked current HEAD (a68ba16471800de7232d0815d5154e71ba9825e1), r2.9, r2.8 and r2.7, but couldn't reproduce the issue in any of them. What version of Newsbeuter are you running?

and read articles.

It only hid feeds for me. Read articles were still displayed. (This one checked only with HEAD.)

It is version 2.9. (Maybe add an issue template for the forgetful who fail to mention their version…)
I meant the key/operation but as you failed to reproduce it, I tried a minimal config. Please see the simplified steps in the updated first post.
(Unrelated to the issue: The toggle-show-read-feeds key hides read feeds if pressed in the feed view, if pressed in the article view, it hides read articles.)

Sorry for a delayed reply. Your new instructions reflect exactly what I did previously. Just in case, I tried this all again, both with HEAD (0da810b5) and r2.9. Still can't reproduce.
Asked people on #newsbeuter if any of them can reproduce this, just in case.
@Markus00000, how did you get your binary? Did you compile it yourself from Git? If not, can you? Maybe your distro is applying some patches?

Maybe add an issue template for the forgetful who fail to mention their version…

Oh, I didn't know GitHub supported those! Filed a separate issue about that. Thanks!

Unrelated to the issue

…thus split into #396.

It is this Arch package, which applies a patch, but I can also reproduce it with the newsbeuter-git package (765b053), which does not seem to apply any patches.

b0at on IRC confirms:

it seems to exhibit the bug for me. i'm using newsbeuter-git on arch, which i reinstalled the other week, and it reports as 2.10 and newsbeuter-git 1984.b961c66-1

So I found the cause. Arch already has STFL 0.24, while here in Funtoo I still have 0.23. I built Newsbeuter with 0.24 and immediately reproduced the bug with HEAD.
There's just a couple changes between 0.23 and 0.24, but it'll take time to figure out which one introduced the bug (or exposed one in our code).
Thanks for reporting this!


fix(textfield): diacritics & descenders cut off in some languages

🚨 DO NOT MERGE 🚨

Description
Test/fix branch for prototyping line-height changes.
How and where has this been tested?

How this was tested:
Browser(s) and OS(s) this was tested with:

Screenshots

To-do list

[ ] If my change impacts other components, I have tested to make sure they don't break.
[ ] If my change impacts documentation, I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have tested these changes in Windows High Contrast mode.
[ ] I have updated any relevant storybook stories and templates.
[ ] If my change(s) include visual change(s), a designer has reviewed and approved those changes.

[ ] This pull request is ready to merge.

Closed in favor of #2123


Example of Notify usage

Hi, I'm trying to use the Notify service and I always get
{:error,
 %{
   "code" => 20404,
   "message" => "The requested resource /Notifications was not found",
   "more_info" => "https://www.twilio.com/docs/errors/20404",
   "status" => 404
 }, 404}

It works using cURL, the Messaging Services is created under console and I'm using the right SID.
Code used:
ExTwilio.Notify.Notification.create(%{sid: "#{sid}", to_binding: "{\"binding_type\":\"sms\", \"address\":\"#{phone}\"}", body: "Message"})

I've also tried to send the sid as service_sid but same results.
Same 404 status code for Notify Binding resources too.

There is a second argument that you use to pass it as [service: service_id]. Wish this had been documented took awhile to figure out.


launch.json usewsl is not ignored on non-windows OS

VSCode Version: 1.19.3
OS Version: Windows 10 & MacOSx

Steps to Reproduce:

Set "usewsl": true in launch.json in VScode windows to run code in bash. Works perfectly.
Git commit
Open in Mac VScode and get message about not finding Windows Subsystem for linux.

Maybe usewsl should be ignored on non-windows machines?
Annoying since now I have to change that value every time I change from Windows to Mac.

Oops, this is a bug.
As a workaround you can just wrap the useWSL attribute in a windowssection like this:
			"windows": {
				"useWSL": true,
			},

With this it only takes effect on windows.


[BUG]: Arguments to chaincode has text of a function appended to it and fails with error about a '\n' in args

Describe the bug
For some reason the extention is appending a function onto the end of the list of my arguments when calling a chaincode.
To Reproduce
Steps to reproduce the behavior:
I tried my own .fabric files and just copying the example and they both caused the issue with it claimg there was a '\n' in the argument
[
    {
        "query": "ReadAsset",
        "args": ["asset1"]
    }
]

Expected behavior
A clear and concise description of what you expected to happen.
Environment:

OS: Windows 11
Docker version v24.0.7
Extension version 1.0.0

Screenshots
If applicable, add screenshots to help explain your problem.
Logger Output

2024-01-03T06:47:34.109Z [Error] Syntax error in query/invoke request. Chaincode arguments supplied: ['"asset1"','"function (old_index, new_index) {
    // Shortcut helper to move item to end of array
    if (-1 === new_index) {
        new_index = this.length - 1;
    }

    if (new_index >= this.length) {
        var k = new_index - this.length;
        while (k-- + 1) {
            this.push(undefined);
        }
    }
    this.splice(new_index, 0, this.splice(old_index, 1)[0]);
    return this; // for testing purposes
}"'].

@Nova38 , Thanks for reporting the issue.
This is pretty weird.. I am thinking that there could be some special character in the file which might be messing up with the parsing of the .fabric file.
Can you please attach the actual .fabric file to the issue so that I can take a look?


Swifter on server side (Mac) hangs

I am attempting to use Swifter in conjunction with the Vapor 3 server side web framework. It builds fine, but any attempt to use it just hangs. Even in test environment, no errors, just eerie stillness.
I am doing the connection in a background thread since Vapor doesn't allow blocking on the main thread, could this be causing any issues?
self.twitter = Swifter(consumerKey: key!, consumerSecret: secret!)
let async = context.container.eventLoop.newPromise(Void.self)
DispatchQueue.global(qos: .background).async {
    print("Marco?")
    let callback = URL(string: "https://asdf.horse/twitter")
    self.twitter.authorize(withCallback: callback!, success: { accessToken, response in
        print("Polo! (this never happens)")
        print(response)
        async.succeed(result: ())
    }, failure: { error in
        print("This never happens either.")
        async.fail(error: error)
    })
}
return async.futureResult.flatMap { _ -> Future<Void> in
    // go on to do some stuff like a users/search
}

… This appears to be because Swifter is doing its own async on the main thread, which Vapor doesn't like at all. Changing SwifterHTTPRequest.swift line 168 to…
DispatchQueue.global(qos: .background).async {

… causes it to go a bit further, but now Swifter isn't picking up the success/fail message (I think?)
At any rate, my impression is that Swifter is intended to be more of a client library and server side, so I may have to roll out a different solution. Closing this issue, but please let me know if I'm wrong and there's a solution to be found.

Having same issue with iOS


Run with HTML API only

Run widget without JS configuration.
Partially done in 2.0 release, but needs more testing.

API looks like:
<div id="github-card"
     data-username="YOUR_GITHUB_USERNAME"
     data-max-repos="3"
     data-sort-by="stars"
     data-header-text="Most starred repositories">
</div>


NETOBSERV-1020 Fix empty namespace list

cf https://github.com/observatorium/opa-openshift/pull/18
We need to prevent attempts to run queries when the matchers list exists but is empty, which corresponds to having 0 allowed namespaces

closing: amended the opa-openshift PR to make it self-contained, so there's no need to change something on the API side.


Invalid record is generated when double type additionalProperties has defined

Description:
$subject
Affected connectors : googleapis.bigquery
Sample OpenAPI schema property
labelClassWeights:
          additionalProperties:
            format: double
            type: number
          type: object

Generated bal type
record {|double...;|} labelClassWeights?;

Gives the error
ERROR [types.bal:(470:14,470:20)] unknown type 'double'
Steps to reproduce:
Affected Versions:
2201.4.1
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

@aneeshafedo lets send a duplicate PR to 1.5.x as well

Close with this PR https://github.com/ballerina-platform/openapi-tools/pull/1313


Update cuda and next release versions

What changes were proposed in this pull request?
Updates cuda and next release versions to latest.
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
Quick checklist
The following checklist helps ensure your PR is complete:

[ X] Eclipse Contributor Agreement signed, and signed commits - see IP Requirements page for details
[X ] Reviewed the Contributing Guidelines and followed the steps within.
[X ] Created tests for any significant new code additions.
[ X] Relevant tests for your changes are passing.

Removed all tmp files and updated gitignore to prevent regressions. Thanks for flagging!


rosetta-bitcoin and rosetta-ethereum return invalid response on /network/options request

The rosetta-bitcoin and rosetta-ethereum set the call_methods and balance_exemptions to null. The spec mandates a type of array.  It seems like the correct thing to do would be to set the fields to nullable in the spec or return empty arrays in the implementations. Alternatively it would be even better to make the fields optional, but that's a backwards compatibility issue.

Thanks for the callout. The rosetta-bitcoin and rosetta-ethereum should update call_methods and balance_exemptions to zero value (empty array) to comply rosetta spec. I created two issues for them accordingly:

https://github.com/coinbase/rosetta-bitcoin/issues/126
https://github.com/coinbase/rosetta-ethereum/issues/137

I will close this issue and the issue tracking and fix will be moved to the specific rosetta implementations.


Why can't I build boost in my computer?

I can't not see any .lib in lib folder, Why?

我换成1.83版本就编译过了

build_output.txt
This file recored the whole message I built boost 1.85 version.

我换成1.83版本就编译过了

我的还是不行，不知道哪里出现问题了

I have already fixed this issue.  Take the attach file to replace ".\boost_1_85_0\tools\build\src\tools\msvc.jam".

Or you can add the code as the figure show:

hi, @ZhikangLai. Can you please give me the entire steps to build boost  and can use it in my local machine ?
My system configuration are as follows:

Windows 11 OS
VS code and mingw installed.
Also Qt creator is installed.

hi, @ZhikangLai. Can you please give me the entire steps to build boost and can use it in my local machine ? My system configuration are as follows:

Windows 11 OS
VS code and mingw installed.
Also Qt creator is installed.

Hi, Mr. @chilkaditya , Which step are you stuck on? Could you share more details about your compiler？If you're using GCC, you might not encounter the same issue I did. However, if you're using MSVC 14.3, it's likely you're facing a similar problem.

@ZhikangLai sure,
I do not have Microsft visual studio IDE. I have vs code editor, mingw compiler downloaded separately. Can i build boost library without Microsft visual studio IDE ?

@ZhikangLai  sure,
I do not have Microsoft visual studio IDE. I have vscode editor and mingw compiler downloaded separately. After downloading boost 1.85.0, using command prompt i run bootstrap.bat and then ./b2 command, then one bin.v2 folder generated but inside this folder there was libs folder. I  attached image below.

inside libs folder all folders is empty and I also cant use boost from vs code editor. I add bin.v2 path to environment variable. I am stuck here.

2. mingw

when you run "b2", have you try this command:
b2 toolset=gcc
?

@ZhikangLai sure, I do not have Microsoft visual studio IDE. I have vscode editor and mingw compiler downloaded separately. After downloading boost 1.85.0, using command prompt i run bootstrap.bat and then ./b2 command, then one bin.v2 folder generated but inside this folder there was libs folder. I attached image below.
   inside libs folder all folders is empty and I also cant use boost from vs code editor. I add bin.v2 path to environment variable. I am stuck here.

when you run "b2", have you try this command:
b2 toolset=gcc
?


Show types in Intellisense during object initialization

During object initialization, there is no hint for what type a property has which makes working with unfamilliar classes difficult. There should be an indication of the type of a property.
How it is now:

Proposed solution:

Please file this against the c# extension


Set up repository with linters

Complete milestone 1
In this milestone, I

[x] Generated API boiler plate for the bike-rental app
[x] Set up linters

Thanks Ali for the review.


Document/sample for a single codebase that is different on iOS and Android

Flutter allows developers to build UIs that are mostly identical across iOS and Android, yet differ in important ways. Flutter doesn't force developers to build a single uniform UI if they don't want to.
This is a common question from our initial partner discussions, so we might want to document and show strategies for using Flutter in ways that give developers extreme code reuse and yet tweak for individual platforms.
e.g. we could:

Ensure the FAQ correctly answers this question ("Can I make two UIs that are mostly similar but differ slightly?" and "Can't I use OEM date picker on iOS?")
Ensure we have a good article on the website explaining the strategy and examples. Probably back that with sample code somewhere.

Does the memechat video address this sufficiently?

Great call! Yes, probably.
Before we close this out, @emshack @efortuna can you link to that codebase?

Yup. Here's the video: https://www.youtube.com/watch?v=w2TcYP8qiRI
And here's the repo: https://github.com/efortuna/memechat

Thanks! It's a great example of a single codebase that is tweaked for iOS and Android.


ts:likes-functionlity- tests likes functionality

What does this PR do?
This PR tests the like-functionality of staff users on posts. The like functionality needs to be tested and improved.

Still needs testing and improvement


Unclosed Socket

Describe the bug
A socket is opened in line 143 of the getmac.py file. However, this socket is not closed again.
This leads to a "ResourceWarning".
/Users/xxxx/projects/netmap/netmap/network.py:18: ResourceWarning: unclosed <socket.socket fd=3, family=AddressFamily.AF_INET, type=SocketKind.SOCK_DGRAM, proto=0, laddr=('<IP_ADDRESS>', 50240)>
  self.mac_address = get_mac_address(ip=str(self.ip_address.ip))
Object allocated at (most recent call last):
  File "/Users/xxxx/projects/netmap/venv/lib/python3.7/site-packages/getmac/getmac.py", lineno 143
    s = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)

To Reproduce
Run a unittest that contains a get_mac_address call that requests a mac through a ip.
Expected behavior
No Warning
System info
(please complete the following information):

OS name: macOS
OS Version: 10.14.4
Python version: 3.7.3 x64
getmac version: 0.8.0

Thanks for the report, and apologies for the delay in responding. This is definitely a issue and one I'm surprised has been overlooked until now. I've pushed a potential fix to master, please give it a try. You can install like so: pip install https://github.com/ghostofgoes/getmac/archive/master.tar.gz (that URL works in a requirements.txt/setup.py/etc. as well).
If that fixes it I'll get a point release cut and released as soon as I can.

The fix works. Thanks

Released on GitHub and uploaded to PyPI as 0.8.1.


Can't flash new firmware to ESP32-S3-BOX

Hello, I just received the ESP32-S3-BOX. Following the recommendations I was hoping I could upgrade the latest firmware according to these instructions: https://github.com/espressif/esp-box/blob/master/docs/firmware_update.md Although I have used this tool successfully multiple times on various other ESP devices, it does not work here. And yes, I also tried holding the boot button etc. It initializes the upload:
AP:  7CDFA1E1C781  STA:  7CDFA1E1C780
BT:  7CDFA1E1C782  ETHERNET:  7CDFA1E1C783
... and then I always get this error:
Uploading stub...
Running stub...
Stub running...
is stub and send flash finish
Is there any other thing I could try? It's not the cable or computer as I was also able to flash other firmware onto it (Tasmota via the Tasmota Web Installer for instance).
Thank you for your help!

You can try Disable download stub in menuconfig like the following diagram.

Wich software is that?

I realized that I needed to install the ESP IDF and found menuconfig command to see this screen and follow your advice. It worked.

Is it possible to flash through espressif IDE I don,t get how to do that
When i have run the command (git clone --recursive https://github.com/espressif/esp-box.git)
esp-box folder with required elements are created
so how can i run modify code using the IDE.
i able to flash using CMD  commands.

Is it possible to flash through espressif IDE I don,t get how to do that When i have run the command (git clone --recursive https://github.com/espressif/esp-box.git) esp-box folder with required elements are created so how can i run modify code using the IDE. i able to flash using CMD commands.

you want to do some modification in box, and don't know how to install IDF?
you can refer to this: installation-step-by-step

Is it possible to flash through espressif IDE I don,t get how to do that When i have run the command (git clone --recursive https://github.com/espressif/esp-box.git) esp-box folder with required elements are created so how can i run modify code using the IDE. i able to flash using CMD commands.

you want to do some modification in box, and don't know how to install IDF? you can refer to this: installation-step-by-step

currently working in espressif ide(windows) projects based on esp32
is it possible to modify the example code given inside the box
esp32 s3 board is already there.

does the location of the folder is wrong..?

C:\Espressif\frameworks\esp-idf-v4.4.2\esp-box

IDF is the same, you should press cmd to change build target:
idf.py set-target esp32s3
so, what's the problem now?  I'm not very clear

so, what's the problem now? I'm not very clear
Need to use ESPRESSIF IDE to flash the code and modify the code of esp-box other than using ESP-IDF command promt.
what should i do for that?
initial step followed
was (git clone --recursive https://github.com/espressif/esp-box.git)
then try to open the example for building it what may be the issue here.

Sorry, I didn’t use this IDE before.
I asked my colleagues, this is the IDE made by the previous agent,
I am not sure whether it has been adapted to the latest chip of S3, this is not our official recommended compilation method, if it is a window environment, you can execute it here [window install step]https://docs.espressif.com/projects/esp-idf/en/latest/esp32/get-started/windows-setup.html#get-started-windows-first-steps
And linux is recommended, window is too slow;
@.*** @.***
@.***
Best regards,
徐新
AE｜Espressif Systems
T:<PHONE_NUMBER>2
www.espressif.comhttp://espressif.com/
#204, Block 2, 690 Bibo Road,
Zhang Jiang High-Tech Park, Shanghai, China
发件人: @.>
发送时间: 2022年11月11日 19:40
收件人: @.>
抄送: Xu @.>; @.>
主题: Re: [espressif/esp-box] Can't flash new firmware to ESP32-S3-BOX (Issue #36)
[External: This email originated outside Espressif]
so, what's the problem now? I'm not very clear
Need to use ESPRESSIF IDE to flash the code and modify the code of esp-box other than using ESP-IDF command promt.
what should i do for that?
initial step followed
was (git clone --recursive https://github.com/espressif/esp-box.git)
then try to open the example for building it what may be the issue here.
[image]https://user-images.githubusercontent.com/83206454/201333081-eb728970-79eb-4cf0-9022-cee43e60ed1f.png
―
Reply to this email directly, view it on GitHubhttps://github.com/espressif/esp-box/issues/36#issuecomment-1311590592, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A2PUVSRBW67HTYK6753CJJTWHYWBDANCNFSM5ZZA6BZA.
You are receiving this because you commented.Message ID: @.***>

okay thanks, @espressif2022
so for development side you guys are using external editor and for building you use esp idf cmd right.
so there is no possible way to remodify compile the code using ide

using the ide is more easy way thats  why i asked for that to modify code of esp-box using ide since currently we have been using esp32 based projects using the ide.
is there any other ide that can be used for that such as visual studio code.

sorry, you did it with ESP-IDF Eclipse Plugin? ESP-IDF Eclipse Plugin

sorry, you did it with ESP-IDF Eclipse Plugin? ESP-IDF Eclipse Plugin

yes

I am not sure whether it has been adapted to the latest chip of S3

can you build it ok?

I am not sure whether it has been adapted to the latest chip of S3

can you build the examples successfully?

can you build the examples successfully?

yes,
need to know how to include box exampple code to build and flash using IDE rather than using ESP-IDF CMD

Hi, if you don't have any questions, please close this issue. Thank you!


Smart Start: iPhone - Fails to find a QR/DSK code

Version
Checklist:

[ x ] I am not using Home Assistant. Or: a developer has told me to come here.
[ x ] I have checked the troubleshooting section and my problem is not described there.
[ x ] I have read the changelog and my problem is not mentioned there.

Build/Run method

[ x ] Docker
[ ] PKG
[ ] Snap package
[ ] Manually built (git clone - yarn install - yarn run build )

Zwavejs2Mqtt version:  5.11.0
Z-Wave JS version: 8.7.5
Describe the bug
A clear and concise description of what the bug is.
SmartStart/+/(select qr/cam icon)/Import
On iphone clicking Drop... activates camera, on desktop a file can be dragged.
In either case - no QR code is found.
Expected behavior
QR Code is recognized

@doug62 Duplicate of #1936

To be clear - this bug is about how images acquired either by direct cam or file fail to be recognized - not the process of acquiring images

Humm do you get any error on console? I cannot reproduce it with my codes. Could you send me your qr code via email?


feat: add Moonriver, Metal L2 Testnet

Description
 Add Moonriver and Metal L2 Testnet using the projects' official docs.
Backward compatibility
 No
Testing
 CLI

@paulbalaji, sorry Paul, I made a note to remove the final '/', but I forgot... I'll triple-check next time I submit a PR.

@xeno097, can you please check this PR. Paul reviewed it and I corrected the errors.
Thank you.

Hey @guy93354 it seems that the prettier action is failing. Please  run yarn prettier and push the changes to make ci pass

Thank you @xeno097, I ran prettier and uploaded the corrected files.


Creating a Component

Summary
A user of Medulla should be able to create  a component within Medulla. Once the component has been designed by the user and the user saves it, an api request  (using GRPC) will be made to the corresponding endpoint within the backend. The request should contain the component the user made in a string format.
Motivation and goals
This allows the user to create components, and save it by interacting with the backend. Doing so, allows future reuse of the component.
In scope
Creating a component and attempting to save the component (only documenting and styling).
Out of scope
Interactively of the components, confirmation of component storage.
Risks / unknowns
There are currently no constraints in regards to creating components.
Examples
A user creates a component by inserting a label and a button. Once the design of this component is saved, the user will save it. After it has been saved, an api request will be made to the corresponding endpoint to store this component.

For now, we will have simple components with only HTML and CSS.

I believe we need to approach this in the following order of work:
Medulla:

Move previous code into the new front end framework we are using (Blazer)
Figure out an alternative to grape.js
Add the ability if not already implemented to create a component
Implement a method to take the designed component from the user of Medulla to convert it into a string or another requested format for the backend to store the component.
Implement a API request to send the component to the backend

@keithlogan94  @AndrewBabbitt97   thoughts on this order?

I will start looking into Blazer and see how drag and drop works along with what components we can use for this.

@babanfaraj Here's a good arrival on it, you use the standard HTML5 events for it: https://chrissainty.com/investigating-drag-and-drop-with-blazor/

I like how we are commenting within these issues. It keeps conversations scoped to the work.

@babanfaraj Here's a good article on it, you use the standard HTML5 events for it: https://chrissainty.com/investigating-drag-and-drop-with-blazor/

Thank you @AndrewBabbitt97 ! I will look into this and try to create a sample program going so we can go from there.
Once I figure out the jest of it, will there be a place in the front end for it @keithlogan94 ? I noticed you had the other pull request open for the start of the project


Update README.md

Fix syntax error; there was a newline between ] and ( so the link appeared twice. Also now https://.

Thanks!