Compatibility with Greasemonkey

Per your invitation on https://github.com/lisonge/vite-plugin-monkey/issues/166 documenting an issue and will look to put together a PR for it, based on your preference(s) here.
There are a couple of options here, so figured a ticket would be best place to discuss before putting together a PR. In the meantime, I forked the code to make these modifications, so I'm not blocked on this issue. Having this implementation worked out great otherwise, so thank you!
For context: I was using this library + vite-plugin-monkey, yet with JS instead of TS. The resulting userscript worked on Tampermonkey but not on Greasemonkey. So the issues described and following recommendations are aimed at getting where my local fork got to -- a solution that seems to work on both. If these changes are made (or satisfactory alternatives), then I should be able to switch from a local fork back to pulling this library from npm directly.

GM.xmlHttpRequest is GM API 4 and GM_xmlhttpRequest is GM API 3. Userscript extensions like Tampermonkey appear to support both APIs, whereas Greasemonkey dropped support for GM API 3. The JS thread was halting when coming across the undefined references to GM API 3, so the existing code does not appear to be sufficient for supporting both API 3 and 4.

Recommendation: remove support for GM_xmlhttpRequest and require use of GM.xmlHttpRequest. Tampermonkey supports GM API 4 just fine, so I don't see a particular reason to support GM API 3 anymore. API 4 has existed for ~7 years now.

Greasemonkey's implementation of GM.xmlHttpRequest appears to have a TODO around returning an abortable object. It doesn't return anything instead, so the use of this library currently fails due to this issue.

Recommendation: remove use of { abort } = on the xhr result and delete references to the variable. You could branch the logic based on GM.info.scriptHandler if it were important to support both, but for my purposes I didn't need abortable support.

The "binary: true" flag was causing issues processing the response from the service I was using this library to call. Removing it resolved the issue.

Recommendation: remove use of binary flag or support taking in a parameter that activates it.

Wow, thanks, I appreciate all these detailed proposals!
re 1:

The JS thread was halting when coming across the undefined references to GM API 3

Instead of removing GM_xmlhttpRequest completely, I'd like to know the exact reason why this happens. The way I see it, this code should prevent the script from failing if GM_xmlhttpRequest is not defined:
var _GM_xmlhttpRequest = /* @__PURE__ */ (() => typeof GM_xmlhttpRequest != "undefined" ? GM_xmlhttpRequest : void 0)();

Unless GM_xmlhttpRequest is aliased to another thing in GM API 4. In that case I think switching the order here is enough: https://github.com/Sec-ant/gm-fetch/blob/f59b5e450ebcf3b25f78bdf5ac6f71a5cba2a323/src/index.ts#L24
re 2:
I prefer to keep the abort mechanism for script handlers that support it. But yeah this one is tricky, it seems different script handlers have different behaviors: https://github.com/trim21/gm-fetch/issues/56#issuecomment-1492859094
re 3:
I unified all kinds of request payload into a blob and use binary mode to send them because I believe there were some issues about formdata payload when I developed this script. Deleting binary: true can break many things. Maybe now it's time to revisit current options. https://github.com/Sec-ant/gm-fetch/blob/f59b5e450ebcf3b25f78bdf5ac6f71a5cba2a323/src/index.ts#L37-L42
Anyway, these problems are tougher than I thought. Thanks for bringing up all the problems you encountered in Greasemonkey. I don't think I'll accept a PR to fix them the way you did, but I think I can take a further look into them later when I have time.


[RFE] Add the ability to add a delay

I've setup one of my keys to lock the screen as 'cmd-ctrl-q' but I usually like to lock the screen and suspend it, so I need to press escape after that. It would be nice to add a "delay" so the 'cmd-ctrl-q<>escape' shortcut works.
I don't think such delay can be implemented tho, but just in case :)
Thanks!

I don't think such delay can be implemented tho

You are right, I'm limited by what hardware can do and what "official" software can. If you find any software which can do it, I'll copy it. Otherwise let's think it's impossible.


docs: add README.md in demo folder

Awesome project, just a small one to help others running kcp themselves. This PR adds a README.md in the demo folder as an index to the existing demos. Might be obsolete with #373, but fixes #410 for now. I hope I got all the demos right.

@ReToCode thanks for the PR!


Are you interested in me writing a clang format config for bone-lisp?

I am happy to write a clang-format config for bone-lisp to enforce c code style across the project, can then hook it up to make as make format or potentially to run every build (if you want to be more aggressive).

If clang-format is able to leave all line-breaks as they are and just fix other things, then this would certainly be useful.


Should it fail sbt when it fails to parse .travis.yml?

Namely here.
https://github.com/dwijnand/sbt-travisci/blob/master/src/main/scala/sbttravisci/TravisCiPlugin.scala#L79-L84
The rationale for failing outright is that such a failure should fail the CI build.

That behaviour comes from #8, so we'd need to evaluate changing it with @djspiewak's use-case.  I'd be happy to have an opt-in "strict mode".

Yeah it's really for when you don't have a .travis.yml. I think failure to parse is probably something that could be errored on, but to what end? Also happy for an opt-in strict mode.


Add modal element for Tip button with Web3 transfer

add Bulma's modal element that show on clicking "Tip" (https://bulma.io/documentation/components/modal/#javascript-implementation-example)
this modal element has:

user and waiter confirmation (names, addresses)

amount of RUB to tip

"Tip" button

call transferFrom on click and save the transaction to database

on the "transferFrom" call here's a question: how to approve for the spender? or just use transfer?


EnvironmentDecryptApplicationInitializer.insert(...) infinite loop possibility

The ApplicationContext overload of insert (currently at line 124) has a while loop that looks like it could get stuck.
This probably can't happen in reality, but it's easy to fix, just move the
				parent = parent.getParent();

one line down, outside the if-instance-of block:
		while (parent != null) {
			if (parent.getEnvironment() instanceof ConfigurableEnvironment) {
				ConfigurableEnvironment mutable = (ConfigurableEnvironment) parent
						.getEnvironment();
				insert(mutable.getPropertySources(), propertySource);
			}
			parent = parent.getParent();
		}

Doubt it @larrywest unless I am missing here - parent.getParent() would get the parent application context which would ultimately be null when it hits the root context and the loop will terminate. The only way this can go wrong is if the parent of a context is itself, I am assuming that there is logic somewhere to prevent such a thing.

@bijukunjummen Well... that would be true with my suggested change (above), but in the existing code (below), I don't see how parent would ever change its value if parent.getEnvironment() were to return something not an instance of ConfigurableEnvironment:
		while (parent != null) {
			if (parent.getEnvironment() instanceof ConfigurableEnvironment) {
				ConfigurableEnvironment mutable = (ConfigurableEnvironment) parent
						.getEnvironment();
				insert(mutable.getPropertySources(), propertySource);
				parent = parent.getParent();
			}
		}

Ah ! yes, missed that.


all: do comparison substitution for extra coverage

Collect comparison arguments for extra coverage.
For that, we now need to start remote coverage collection for every forked program.
Substitute the arguments into all calls that have remote_cover set.

Do we need any special handling for the SYZ_EXECUTOR_USES_FORK_SERVER == 0 case?

Hmm, the backward incompatibility is worse than I thought.
Since we may lose a current->kcov reference (the fix will be posted soon), we're not just leaking remote kcov handles. If it were only that, we could address it by trying out a number of different handles for every procid.
But we're also unable to reuse the same *kcov object since

It's already not in KCOV_MODE_INIT, which breaks https://elixir.bootlin.com/linux/latest/source/kernel/kcov.c#L621
We cannot reset/free it: KCOV_DISABLE only works for current->kcov: https://elixir.bootlin.com/linux/latest/source/kernel/kcov.c#L612

And we cannot obtain a new /dev/kcov handle inside execute_one() -- when we're there, we're already in a sandboxed environment.

Can we create the remote kcov on a thread that never executes? I.e. not the main thread, but rather an additional special thread that exists just for that purpose?

Can we create the remote kcov on a thread that never executes? I.e. not the main thread, but rather an additional special thread that exists just for that purpose?

Interesting!
Though it won't be a thread that truly never executes since it will at least have to execute the KCOV_REMOTE_ENABLE ioctl. We can try to e.g. block the thread after that ioctl. There's still a chance to catch an irq in between, but a much smaller one.

Can we create the remote kcov on a thread that never executes? I.e. not the main thread, but rather an additional special thread that exists just for that purpose?

Interesting! Though it won't be a thread that truly never executes since it will at least have to execute the KCOV_REMOTE_ENABLE ioctl. We can try to e.g. block the thread after that ioctl. There's still a chance to catch an irq in between, but a much smaller one.

KCOV_REMOTE_ENABLE is done once per top level process, so it should reduce probability several orders of magnitude.
If can be mmaped by a different thread, then we could even create them in syz-fuzzer (I wouldn't bother now, we can do it in the rewritten executor runner).


Standard deployment broken

Type of issue

[x ] Bug
[ ] New feature
[ ] Enhancement

Description
C:\source\PCS2\pcs-cli>pcs -t remotemonitoring -s standard
? Enter a solution name: ...
? Select a subscription: ...
? Select a location: East US
? Enter prefix for .azurewebsites.net: ...
? Enter a user name for the virtual machine: ...
? Enter path to SSH key file path: C:\Users....ssh...
✓ Application registered: https://portal.azure.com/microsoft.onmicrosoft.com...
✓ Created resource group: https://portal.azure.com/...
✓ Provisioning State: Succeeded Resource Type: Microsoft.DocumentDb/databaseAccounts
✓ Provisioning State: Succeeded Resource Type: Microsoft.Devices/Iothubs/Iothubkeys
✓ Provisioning State: Succeeded Resource Type: Microsoft.Web/sites/sourcecontrols
✓ Provisioning State: Succeeded Resource Type: Microsoft.Storage/storageAccounts
✓ Provisioning State: Succeeded Resource Type: Microsoft.Web/sites
✓ Provisioning State: Succeeded Resource Type: Microsoft.Web/serverfarms
✓ Provisioning State: Succeeded Resource Type: Microsoft.ContainerService/containerServices
✓ Provisioning State: Succeeded Resource Type: Microsoft.Devices/Iothubs
✓ Provisioning State: Succeeded Resource Type: Microsoft.Network/publicIPAddresses
✓ Deployed successfully, time taken: 8 minutes & 10 seconds
✕ Error: ENOENT: no such file or directory, open 'C:\Users\timlav.kube\config-...-cluster'

@timlaverty were you running pcs from nodejs repo or compiled and linked from source code?

I was running from the CLI repo; e.g. npm install/start/link, then pcs login, then pcs -t remotemonitoring -s standard.

I tried to reproduce the issue but by Windows environment is probably not setup correctly, the deployment fails earlier than that, without error message (see other issue opened).
Given the error message, I'm thinking it might be an issue of missing folder or non-writable path, or maybe an old version of az/kubectl.  In any case, looks like we need to improve the "prerequisites check" before starting the deployment.

@ppathan did we find the root cause? I'd like to close the ticket with some sort of resolution notes

Sorry I was trying to comment and mistakenly closed the issue. But, yes i have root caused the issue and will send out PR shortly.


Allow default params

Allow default params to be globally set for all requests:
Blanket.wrap('http://domain.com', default_params: { access_token: ´12345678´ })

Thanks for contributing! Since none of the other global options is named default_*, would you consider changing default_params to simply params in order to keep consistency?

My pleasure! While coding it I thought that naming it params was causing a name collision. I'll change it it.

Coverage increased (+0.04%) to 99.12% when pulling c1996902b36465ab1d7ee6277e9ff5469254e24b on nicooga:feature/allow_default_parmas into 65be2e0bc46d7e55b5fd27fbdbc8a7f198297eae on inf0rmer:master.

Coverage increased (+0.03%) to 99.15% when pulling 972cfe7f7395fa0df54187cc3a907807fb1833de on nicooga:feature/allow_default_parmas into 6233233f465dd630b4e971b7d8d5d32ac6221bc9 on inf0rmer:master.

Coverage increased (+0.03%) to 99.15% when pulling 0ec96a3a1b04192015496d0fbdac86d54bc3f81d on nicooga:feature/allow_default_parmas into 6233233f465dd630b4e971b7d8d5d32ac6221bc9 on inf0rmer:master.

Coverage increased (+0.03%) to 99.15% when pulling 0ec96a3a1b04192015496d0fbdac86d54bc3f81d on nicooga:feature/allow_default_parmas into 6233233f465dd630b4e971b7d8d5d32ac6221bc9 on inf0rmer:master.


840 conflict resolution

Conflict resolution has been implemented:

If you start editing what someone else is editing, you get stopped with a prompt to either take control, or leave it alone
If control is taken from you, you get an alert and your page reloads
If the person who has control is you, but in another tab/window/whatever, this is called to your attention, although the same rules still apply for the same reasons (no loss of work, no weird competing save callback scenarios)
All of this applies to both pieces and pages
Apostrophe doesn't start trying to autosave an area until you engage with it for the first time, which eliminates major causes of excess page versions and conflicts that shouldn't even be conflicts

Right now there are a fair number of confirm() and alert() calls. We really need #916 for that to be nice. But our clients are in serious need of this feature right away so there's no reason to hold it...
Except for good testing & code review by someone who isn't me! This is significant.

Must reproduce and fix the bug where after taking control of a piece in the editor and saving it (?), I am unable to close the manage modal. And/or verify it is not our bug.


Cannot set Dependency Property of the FontWeight? (Nullable) type in XAML

Describe the bug
    public class MyControl : Control {
        public static readonly DependencyProperty NullableStructProperty = DependencyProperty.Register("NullableStruct", typeof(FontWeight?), typeof(MyControl), new PropertyMetadata(null));
        public FontWeight? NullableStruct { get { return (FontWeight?)GetValue(NullableStructProperty); } set { SetValue(NullableStructProperty, value); } }
    }

<local:MyControl x:Name="c3" NullableStruct="Bold"/>

Result: 'XAML parsing failed.' 'Failed to create a 'Windows.Foundation.IReference`1<Windows.UI.Text.FontWeight>' from the text 'Bold'.
Reproduced in any version, including 1.0.0-preview2
Steps to reproduce the bug
Run sample App1.zip
Expected behavior
No response
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 0.8: 0.8.0 (If you're seeing your issue in older previews of WinUI 3, please try this release)
Windows app type

[ ] UWP
[X] Win32

Device form factor
Desktop
Windows version
May 2021 Update (19043)
Additional context
No response

I suspect the xaml compiler needs the type of the property to be FontWeight not Nullable to be able to convert the string "Bold" to that type.  Have you tried setting the property explicitly with something like:
<local:MyControl x:ame="c3">
    <local:MyControl.NullableStruct>
        .... Your Value here, I'm not sure what the syntax is for the Nullable<FontWeight> type...
    </local:MyControlNullableStruct>
</local:MyControl>

@RealTommyKlein FYI


WIP: MNT: Update pinned versions.

Hi! This is the friendly conda-forge-admin automated user.
I've bumped some of the conda-forge pinned versions, and noticed that it impacts this feedstock.
If the changes look good, then please go ahead and merge this PR.
If you have any questions about the changes though, please feel free to ping the 'conda-forge/core' team (using the @ notation in a comment).
Thanks!

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Need to re-render the feedstock first.

Please re-render with conda-smithy version 1.3.2 before merging this or any other PR.

@vgauthier, this should not have been merged yet. Please re-render with conda-smithy version 1.3.2 in a new PR and bump the build number.

FYI the reason this is important is Travis CI changed the default image. So we may lose compatibility with 10.9 for this build. Re-rendering will change us back to the right image.
xref: https://github.com/conda-forge/conda-forge.github.io/issues/249

Raising as issue ( https://github.com/conda-forge/cairomm-feedstock/issues/4 ).


Added text parameter option to Facebook

Hi Michael, saw you post this addon on discord and had been planning to get something like this for an upcoming project. I found that I could add text to a facebook post as well using the 'quote' parameter in the url so thought i would do a pull request for it as I was going to make the adjustment for my use case.

Awesome! Thanks for the pull request.


Fix mirror clone

What problem does this PR solve?
Fix some minor bugs.

Codecov Report

Merging #271 into master will decrease coverage by 37.34%.
The diff coverage is 21.73%.

@@             Coverage Diff             @@
##           master     #271       +/-   ##
===========================================
- Coverage   61.22%   23.87%   -37.35%
===========================================
  Files          40       36        -4
  Lines        2298     2923      +625
===========================================
- Hits         1407      698      -709
- Misses        674     2132     +1458
+ Partials      217       93      -124

Flag
Coverage Δ

#coverage
?

Impacted Files
Coverage Δ

cmd/clean.go
18.91% <0.00%> (-37.84%)
:arrow_down:

cmd/install.go
66.66% <0.00%> (-7.53%)
:arrow_down:

cmd/list.go
17.18% <0.00%> (-61.39%)
:arrow_down:

cmd/run.go
0.00% <0.00%> (-62.78%)
:arrow_down:

cmd/status.go
12.50% <0.00%> (-68.46%)
:arrow_down:

cmd/uninstall.go
36.20% <0.00%> (-43.80%)
:arrow_down:

cmd/update.go
38.18% <0.00%> (-24.51%)
:arrow_down:

main.go
100.00% <ø> (ø)

pkg/repository/clone_mirror.go
0.00% <0.00%> (ø)

pkg/repository/crypto/keys.go
0.00% <0.00%> (ø)

... and 62 more

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 eeb5600...b1744e0. Read the comment docs.


Updating the troubleshooting docker errors document to align with our latest release

Hi @dbreshears, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (devinb). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@dbreshears : Thanks for your contribution to the Azure documentation! The author, @allclark, has been notified to review your proposed change.

#sign-off

Only the author of the article can use the sign off comment to add the ready-to-merge label.

ok I'm not assigned as of yet (but will be), can we get it approved and pushed out or do I need to wait to be assigned?

#sign-off

#sign-off


Fixes annotations on Ansible development tips

Description of the change:
Documentation Change
Fixes #3833
Motivation for the change:
Incorrect snippet
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[X] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[X] Add or update relevant sections of the docs website in website/content/en/docs

LGTM


Port legends

This should be fairly trivial -- porting over Vega 1 code with minimal changes, and adding it to scene/group.js in Vega 2 as per axes.

Done!

Congrats @arvind! :balloon:

Thanks :)


S3 endpoint doesn't allow containing the scheme

May I have a question that why fybrik doesn't allow S3 endpoint containing the scheme i.e. "http://" ?
https://github.com/fybrik/fybrik/pull/874/files
In a DEV/test environment, we use localstack to setup a local S3 service, if the endpoint doesn't contain the scheme "http", the data access modules ( arrow flight module or mover) have to request data by "https" , but this will fail due to the default SSL certificate mismatch.
Caused by: javax.net.ssl.SSLPeerUnverifiedException: Certificate for <localstack.fybrik-notebook-sample.svc.cluster.local> doesn't match any of the subject alternative names: [*.cloudfront.localhost.localstack.cloud, *.execute-api.localhost.localstack.cloud, *.localhost.localstack.cloud, *.s3.localhost.localstack.cloud, *.scm.localhost.localstack.cloud, localhost.localstack.cloud]

Below is my workaround, it works with the localstack environment.
diff --git a/manager/controllers/app/plotter_generator.go b/manager/controllers/app/plotter_generator.go
index 354f2c29..3e842d50 100644
--- a/manager/controllers/app/plotter_generator.go
+++ b/manager/controllers/app/plotter_generator.go
@@ -69,17 +68,17 @@ func (p *PlotterGenerator) GetCopyDestination(item DataInfo, destinationInterfac

        connection := taxonomy.Connection{
                Name: "s3",
                AdditionalProperties: serde.Properties{
                        Items: map[string]interface{}{
                                "s3": map[string]interface{}{
-                                       "endpoint":   url.Host,
+                                       "endpoint":   bucket.Endpoint,
                                        "bucket":     bucket.Name,
                                        "object_key": genObjectKeyName,
                                },

Historical reasons... Once data-movement-operator did not support schemas inside an endpoint name, thus we had to remove it.
As I understand, this is no longer the case. Please feel free to submit a PR.


Question about weed backup!

环境：搭建集群，副本001。
操作：使用weed backup备份数据。
现象：备份的文件，有些collection的文件大小与原始的两个互为副本文件大小都不一致，有什么特殊的处理导致他们大小不一致，数据是否有丢失的问题。

Environment: build a cluster, a copy of 001.
Action: Use weed backup to back up your data.
Phenomenon: the backup file, some of the collection file size and the original two copies of each other are not the same file size, what special treatment led to their size inconsistencies, whether the data is missing.

There are some problem with "weed backup". Please just setup "rsync" instead for now.

https://github.com/chrislusf/seaweedfs/issues/399  "weed backup" is fixed in https://github.com/chrislusf/seaweedfs/pull/902


Fix recipe remainders for meals

Resolves #13
I can infer that CookingResultSlot#onTakeItem is a slightly modified version of the method it overrides. By looking up CRAFTING recipes instead of MEAL recipes, a matching recipe cannot be found, resulting in no remainders. Additionally, the check for hasRecipeRemainder seems to be an unnecessary addition that only contributes to the issue.
I was unable to reproduce the duplication issue described in #12 (I had to substitute porkchops for pancakes) with these changes. If there are other ways to produce duplications, I'll need some help with testing those.
I don't believe the various overrides of Item#hasRecipeRemainder are effective. This is what the base method does:
   public boolean hasRecipeRemainder() {
      return this.recipeRemainder != null;
   }

The overrides effectively don't change anything because they match which items have the recipe remainder set in their settings and which ones do not. I have therefore removed the overrides to avoid any confusion this might cause.

Thank you very much! I was just desperate to get this to work somehow.


Error 9002 - "Connection reset." appears once per day.

Avoid duplicates

[x] Bug is not mentioned in the [FAQ]
[x] Bug is specific for Android
[x] Bug is not already reported in another issue

Technical details

Device name: Google Pixel 3a
Android version: 11
App version: 2.0.4

Describe the bug
When opening the app for the first time of the day, the error 9002 "Connection reset" occurs.
This error only occurs since the 2.0.4 update but is reproducible every day. It is only triggered when opening the app for the first time.
I tried triggering it by switching from WLAN to LTE during daytime, resetting our Router connection, turning the phone off and on, but did not have any success.
As far as I can tell, functionality of the app is not touched by this, as after confirming the error, contacts get checked like normally.
Steps to reproduce the issue

Have CWA App closed overnight.
Wake up in the morning.
Open CWA App.
Get error.

Expected behaviour
The App opens without any error.

Internal Tracking ID: EXPOSUREAPP-6714

@sfadschm Tanks for reporting. We created an Internal Tracking ID: EXPOSUREAPP-6714. The development team will have a look at this issue. Any further news regarding this issue will be published here. Best wishes, DS

Corona-Warn-App Open Source Team

Ditto, I'm experiencing this too
OnePlus 6T android 10
CoronaWarn app v 2.0.4

Addition:
Just got the same error a second time today.
Still cannot trigger it intentionally.

This issue is listed in the resolved section of the FAQs https://www.coronawarn.app/en/faq/#cause9002_timeout which refers also to issue https://github.com/corona-warn-app/cwa-app-android/issues/998.
I have seen it occurring sporadically on an emulated Google Pixel 3a device using Android 11. Emulators are generally slower than real physical devices, so this would be consistent with a timeout issue.

Not sure this is the same issue. From the GitHub issue list, it seems error 9002 can have multiple causes. This here os "Connection reset." Not sure its the same as "timeout".

@sfadschm

Not sure this is the same issue. From the GitHub issue list, it seems error 9002 can have multiple causes. This here is "Connection reset". Not sure its the same as "timeout".

Thanks for pointing this out.
You're right that error 9002 can cover many reasons:
https://github.com/corona-warn-app/cwa-app-android/blob/f65f66f7e84517cfb12f67b0b6b7ad05d86c8b24/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/exception/reporting/ErrorCodes.kt#L19
In the case of the timeout issue #998 the stack trace is showing
java.net.SocketTimeoutException: timeout
whereas your stack trace from this issue (#2953) shows
java.net.SocketException: Connection reset
There may be a relationship between the two, but we probably need the help of the developers to troubleshoot further. I did try again today to see if I could reproduce, but as it happens I'm not getting any error.

With me it also occurs sporadically during startup.
I had generated a QR code after the update to 2.0.4, logged into this event and logged out again. The next time I restarted, the above-mentioned problem occurred.
Stack trace:
java.net.SocketException: Connection reset
at java.net.SocketInputStream.read(SocketInputStream.java:209)
at java.net.SocketInputStream.read(SocketInputStream.java:139)
at org.conscrypt.ConscryptEngineSocket$SSLInputStream.readFromSocket(ConscryptEngineSocket.java:5)
at org.conscrypt.ConscryptEngineSocket$SSLInputStream.processDataFromSocket(ConscryptEngineSocket.java:21)
at org.conscrypt.ConscryptEngineSocket$SSLInputStream.access$100(ConscryptEngineSocket.java:1)
at org.conscrypt.ConscryptEngineSocket.doHandshake(ConscryptEngineSocket.java:7)
at org.conscrypt.ConscryptEngineSocket.startHandshake(ConscryptEngineSocket.java:10)
at okhttp3.internal.connection.RealConnection.establishProtocol(RealConnection.kt:30)
at okhttp3.internal.connection.RealConnection.connect(RealConnection.kt:27)
at okhttp3.internal.connection.ExchangeFinder.findHealthyConnection(ExchangeFinder.kt:113)
at okhttp3.internal.connection.ConnectInterceptor.intercept(ConnectInterceptor.kt:20)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:14)
at okhttp3.internal.cache.CacheInterceptor.intercept(CacheInterceptor.kt:222)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:14)
at okhttp3.internal.http.BridgeInterceptor.intercept(BridgeInterceptor.kt:36)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:14)
at okhttp3.internal.http.RetryAndFollowUpInterceptor.intercept(RetryAndFollowUpInterceptor.kt:35)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:14)
at de.rki.coronawarnapp.http.HttpErrorParser.intercept(HttpErrorParser.kt:1)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:14)
at de.rki.coronawarnapp.http.interceptor.RetryInterceptor.intercept(RetryInterceptor.kt:2)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:14)
at okhttp3.logging.HttpLoggingInterceptor.intercept(HttpLoggingInterceptor.kt:4)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:14)
at de.rki.coronawarnapp.http.interceptor.WebSecurityVerificationInterceptor.intercept(WebSecurityVerificationInterceptor.kt:1)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:14)
at okhttp3.internal.connection.RealCall.getResponseWithInterceptorChain$okhttp(RealCall.kt:25)
at okhttp3.internal.connection.RealCall$AsyncCall.run(RealCall.kt:12)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1162)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:636)
at java.lang.Thread.run(Thread.java:764)

Thanks for the stacktrace in text form :grin:
My current theory is that the error orginates from:
https://github.com/corona-warn-app/cwa-app-android/blob/e1ec946e09600826d16ea389aa8dd96f7c959129/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/diagnosiskeys/server/DiagnosisKeyServer.kt#L31-L38
It should not have a relation with CheckIns, I don't know yet why this suddenly occurs.

In any case, the time factor plays a role. If I wait a certain amount of time (about 3h I tried) before restarting the app after the last crash, the error comes back, otherwise it starts normally.

Did anyone have a registered test when the error showed?

Did anyone have a registered test when the error showed?

Now that you mention it: yes!
It was an old one and it expired yesterday so it was deleted. Today I did not get the error yet.

FYI PR https://github.com/corona-warn-app/cwa-app-android/pull/2973

Did anyone have a registered test when the error showed?

Yeh, I did have an old one too that just expired. Upon deleting it, I too did not get this error this morning.

@dsarkar #2973 is not a fix for the connection reset error. That likely can't be fixed on client side. Maybe hidden, but without any other error reporting, just hiding it is also not great.
#2973 fixes EXPOSUREAPP-6784, it will prevent the connection reset error from causing the "test invalid" card to display which could cause the user to remove actually valid test.
We should leave this ticket open for other users to find it until either server side the error is fixed, or a decision was made to hide the error popup for this case type of error.

Did anyone have a registered test when the error showed?

I also have an expired test that I have not yet deleted.

I also have an expired test that I have not yet deleted.

Do you still get the error?

No, the error does not seem to occur anymore.

I have been trying to reproduce this error on an Android 8 device with a dummy test registered, but so far, no error message. I tried for instance disabling WiFi for more than 4 hours (with no mobile data enabled either), then starting CWA before and after WiFi was reenabled.
Sidenote: CWA Android 2.0.5 has been released and I have installed it from the Google Play Store.


Enable pending acceptance tests

Description:
Re-enables tests for:

HTS Precompile methods: wipeTokenAccount, wipeTokenAccountNFT
HTS Token methods: allowance, transferFrom

Related issue(s):
Fixes #521
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report
Base: 76.38% // Head: 76.38% // No change to project coverage :thumbsup:

Coverage data is based on head (0404af6) compared to base (400a1e7).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #524   +/-   ##
=======================================
  Coverage   76.38%   76.38%
=======================================
  Files          12       12
  Lines         923      923
  Branches      144      144
=======================================
  Hits          705      705
  Misses        165      165
  Partials       53       53

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Moving changes from this PR to #464 to simplify merging


vite errors on cyclic imports

Describe the bug
I originally reported this in https://github.com/vitest-dev/vitest/issues/4143 but now I can reproduce in vite.
When importing certain modules with cyclical imports vitest will evaluate a file before it's imports are undefined, causing unexpected TypeError: _ is undefined errors. The module can be imported as native ESM in node and works in esbuild.
Reproduction
https://stackblitz.com/edit/vitejs-vite-bouzoh?file=server.js
Steps to reproduce
I found this bug when using https://github.com/amazon-ion/ion-js. Here's a simplified version that reproduces the issue
ion-js
├── dom
│   ├── Blob.js
│   └── index.js
├── Ion.js
└── IonTypes.js

// ion-js/Ion.js
export { IonTypes } from "./IonTypes.js";
import * as dom from "./dom/index.js";
export { dom };

// ion-js/IonTypes.js
export const IonTypes = {
    BLOB: "Blob",
};

// ion-js/dom/index.js
export { Blob } from "./Blob.js";

// ion-js/dom/Blob.js
import { IonTypes } from "../Ion.js";
export const Blob =  IonTypes.BLOB

The issue is related to import * as dom from "./dom/index.js"; in Ion.js. If that is replaced with export * as dom from './dom/index.js' or import { Blob } from './dom/Blob.js' the issue doesn't reproduce.
stateDiagram-v2
    [*] --> Ion.js
    Ion.js --> IonTypes.js: export { IonTypes }
    Ion.js --> dom/index.js: import * as dom
    dom/index.js --> dom/Blob.js: export { Blob }
    dom/Blob.js --> Ion.js: import { IonTypes }

System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.20.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.4.2 - /usr/local/bin/npm
    pnpm: 8.6.10 - /usr/local/bin/pnpm
  npmPackages:
    vite: ^4.4.9 => 4.4.9

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I also reported this in https://github.com/vitejs/vite/issues/14010

Is there any workaround available? (besides refactoring the codebase) 😞


Make STYLE_URLS_SOURCE be configurable

Having API_URL configurable is really helpful for those of us in a firewalled environment so we can point to our own GitHub Enterprise server, however you get a later error when trying to download styles, as this always attempts to reach https://github.com/joeyespo/grip to grab the CSS from a README page.
I hacked constants.py to point to a GitHub page that is reachable from our network and everything works great:
STYLE_URLS_SOURCE = "https://git.mydomain.com/user/repo

Would it be possible to expose this as a setting so we can change it in ~/.grip/settings.py rather than needing to edit the source?
Thanks again for a useful tool!

Can I work on this issue?


Feature/2392 Limit Z-score banding calculation to 2 decimal places

What's included?
Closes #2392
Who should test?
✅ Product owner
✅ Developers
✅ UTG
How to test?
Example exercise: https://jac-admin-develop--pr2430-feature-2392-limit-z-9rdeeby0.web.app/exercise/NmzLHCZMAsKjH3GwrCxb/tasks/all/qualifyingTest/finalised/

Go to "QT Merit List".
Check if a single band for each z-score rounded up to 2 decimal places displays in the table.

Risk - how likely is this to impact other areas?
🟢 No risk - this is a self-contained piece of work
Additional context
Include screen grabs, video demo, notes etc.
Related permissions
Have permissions been considered for this functionality?

No permission changes required
Permissions have been added / updated. Details:

PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

@HalcyonJAC Quick question, would this be retroactive on QT tasks that have already been completed or will it only work on new exercises going forward?

@HalcyonJAC Quick question, would this be retroactive on QT tasks that have already been completed or will it only work on new exercises going forward?

@nickaddy It will only work in the incompleted QT tasks as the pass mark has been set up. Does that make sense?


Extension does not add itself to the API of SD.next

The method that the extension uses to detect whether its working within an installation of the webui seems to not work with sdnext: https://github.com/thygate/stable-diffusion-webui-depthmap-script/blob/5250c8b7ea895a4b15575d17a278c88c38cb2132/src/backbone.py#L76
This causes it to run in standalone mode and--i assume--skip the part where it adds itself to the API.
The depth endpoint is missing from the docs and 404s when I try to ping it.
Otherwise, the extension works completely within the UI.

Relevant issue in SD.next https://github.com/vladmandic/automatic/issues/2801

Thanks for reporting! I won't have time to fix this :(
I will be happy to merge the fix if somebody would create one.

Support was just added in sd.next dev. Thanks!


full_like, zeros_like, ones_like

New top-level functions. Fixes #1102

Looks like one of your tests (TestAsCompatibleData.test_full_like) needs to use the requires_dask decorator so it skips if dask is not installed.

fixed

ping - is there anything outstanding before you can merge this?

Done and ready for review

This looks great -- thanks!


Error in nested DITA-OT project file is not properly reported.

Expected Behavior
The DITA-OT build should fail, when there is a problem with the DITA-OT project file. The error message should help me to determine the position of the error.
Actual Behavior
I get an error like this:
Error: Failed to read project file file:/home/runner/work/foo/foo/bar.xml:3:122:
text not allowed here; expected the element end-tag or element
"context", "deliverable", "include" or "publication"

First problem is, that the build does not fail at all on GitHub Actions. This should be fixed.
The reported line (3) and character (122) position is misleading, because the bar.xml file contains only <include> elements. Probably the message is triggered, when the includes were resolved. This makes it hard to determine the position of the error.
Possible Solution
It would be nice, if the DITA-OT build would fail immediately, if the project file is invalid. It would be nice, if the error message would echo the ID/name of the deliverable. This would help a lot to nail down the error.
Steps to Reproduce
Not available, proprietary code.
Copy of the error message, log file or stack trace
See above.
Environment
GitHub Actions runner

DITA-OT version: 4.x
Operating system and version: Linux
How did you run DITA-OT? GitHub Actions via DITA-OT project file
Transformation type: Various

@stefan-jung - I understand that you cannot share your actual testcase. However, we need something to reproduce the issue. Can you please create a small sanitized testcase that demonstrates the issue?


v3.0.1 release

Last release before I push the resource rewrite to master.

          1 Warning

:warning:
This Pull Request is probably missing tests.

  Generated by :no_entry_sign: Danger


Add guide for Docker, Traefik & Letsencrypt

As requested by @Idez in #1922. I hope to contribute more in the future regarding documenting Traefik. 👍

@mvdstam Thanks!
Could fix the build?

Errors from misspell:
docs/user-guide/getting-started-with-docker-and-lets-encrypt.md:86:12: "mimimum" is a misspelling of "minimum"
Please fix the above errors. You can test via "misspell" and commit the result.

@ldez Done!

@nmengin I've updated the PR with your requested changes. The remark you made regarding the restart policy seems unnecessary as this is a well-documented (and well-understood) feature within Docker and I see no reason to change that in the example I described in the document. Especially since the example describes the setup of a front-facing reverse proxy like Traefik, I wouldn't understand why you wouldn't want the restart policy on anything else than always.

@mvdstam It was just a suggestion. Of course, for a user guide, the type of restart policy is not really important. I should say might be instead of may be ;).

@mvdstam could you authorize us to modify your PR (rebase)?
https://help.github.com/articles/allowing-changes-to-a-pull-request-branch-created-from-a-fork/

@ldez Done!


[UE-15] Add set attribute method

The intention for this change is to give users the ability to add attributes after the wrapper has been initialized. This is helpful in insteances where required attributes cannot be acquired until long after an app is initialized. For instance, with canary testing a feature will only be available to select users. The user's id cannot be determined until after they have logged in.
To achieve this, a new public method has been added to the wrapper that takes in an attributes parameter. If the client already has attributes (ie they were added during init), the new attributes are added alongside the old attributes. Otherwise, the new attributes are assigned to the attributes key in the client's context.
[changelog]
added: public set attribute method
ps-id: 8E6ED4DD-BB8E-4C27-85E3-4D1F85DF0667

I manually integrated this with a couple small additions to the tests. Therefore I am closing this PR.


wrong order of next/prev buttons at touchbar

I reopen this, because it got closed multiple times, despite beeing unresolved.
issue is described here: https://github.com/microsoft/vscode/issues/35906
happens on a fresh installed macbook 16 inch late 2019 with mac os catalina 10.15.1 and vscode 1.40.1.
Reproduction:

install vscode on macbook with touchbar
start vscode
look at the touchbar
see the unintuitive arrangement of navigation buttons

https://github.com/microsoft/vscode/issues/35906#issuecomment-434259372 suggest that it might be related to different locales
and indeed, the order is correct when i change the language to en (from de). Looks like a code-smell where someone iterates over these buttons sorted by their labels...

Up for grab if someone has an easy fix.

@bpasero my fix was to change the locale to english 🤦♂ which is ok, its anyway often better to just switch your tools to english as a developer: if you have some error message, its easier to google it.
for example, it needed to debug something in the horrible internet explorer and it reported its already cryptic errors in german...

@macrozone you mean VSCode's locale or something else?

@bpasero yes, you can change vscodes language to english: https://stackoverflow.com/questions/36868322/how-to-change-ui-language-in-visual-studio-code


Commands not loaded on iOS

My scripts in the advpaste folder do not load on iOS and thus are not available to me as a command. The same scripts work on my MacBook.

Thanks for opening this issue. Unfortunately, I don't have any iOS devices to test with.
I can't offer any help without error logs provided by you.


[Issue 13756][Broker]Optimize topic policy with HierarchyTopicPolicies about subscribeRate

Master Issue: https://github.com/apache/pulsar/issues/13756
Motivation
See issue: https://github.com/apache/pulsar/issues/13756
Modifications

Add field subscribeRate  in org.apache.pulsar.common.policies.data.HierarchyTopicPolicies
Methods to update subscribeRate  in org.apache.pulsar.broker.service.AbstractTopic
Other Modifications are concerning on org.apache.pulsar.broker.service.persistent.SubscribeRateLimiter  where  subscribeRate are used

Verifying this change

[x] Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): ( no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation

[x] no-need-doc

/pulsarbot run-failure-checks

@Jason918 PTAL


what is {{ tpl . $ }} in template file?

Hi:
I want to know,what is the meaning of {{ tpl . $ }}.
what is $ used for?
Output of helm version:
Client: &version.Version{SemVer:"v2.15.2", GitCommit:"8dce272473e5f2a7bf58ce79bb5c3691db54c96b", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.15.2", GitCommit:"8dce272473e5f2a7bf58ce79bb5c3691db54c96b", GitTreeState:"clean"}

Output of kubectl version:
v1.14.8
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
AKS

Closing as resolved


Dim Disabled Items in Context-Menus

Pull request checklist:

[x] CHANGELOG.md was updated, if applicable

Description
The items in the context-menu, that shows up when right-clicking the input, all had the same text color. When right-clicking in any other input, the text had an "etched" text effect (#1991). This PR dims these disabled items and removes the text-effect. I have no clue why setting a border disables this text effect. There are some references to an etch-diabled-text, but that doesn't seem to (have) work(ed).

Theme/State
White
Light
Dark
Black

Before

After

Fixes #1991.

I imagine this should just be settable by the theme itself? now that themes are JSON-bound
remove draft & poke me when it's ready for review again and I can take a look


bot.private method not returning send method

There was no send method returned in the bot.private method.
Update:
There is a send method returned. However:

The body and meta parameters in the callback function are the wrong way round
The meta parameter returns the user who sent the message's whatsapp ID, not an object of data about them as well as the message as I believe it should.

Can you test if this is fixed with the new protocol system?

It is not, I'm not done with it yet (hence the "breaks everything")


Small fix in documentation

Original author: Nicolas Bock
Original issue: https://charm.cs.illinois.edu/redmine/issues/274

I have added missing curly braces in an example code. The patch is against 460b06dae77ccfe3752afb3768a5c45b9abe67b6.

Original date: 2013-11-11 07:07:02

Applied. Thanks.


fix oracle sql

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #4595 (9fda4ae) into master (56c2a05) will decrease coverage by 0.16%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4595      +/-   ##
============================================
- Coverage     69.33%   69.18%   -0.16%
+ Complexity     7824     7813      -11
============================================
  Files          1037     1037
  Lines         29816    29805      -11
  Branches       2724     2720       -4
============================================
- Hits          20673    20620      -53
- Misses         7518     7559      +41
- Partials       1625     1626       +1

see 17 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


MINIFI-104 - Making connection ids filesystem friendly, unique

Reviewing

Changes look like they would resolve the issue in question but would be interested to hear your thoughts on opting for hashing what is their currently.  Thanks!

@brosander all are fair points and definitely was looking at collisions as a risk.  my primary focus was looking to relegate the name to nothing more than a variable to get it in a similar space as how NiFi handles these things.  To that end, maybe we introduce an id that is required to be unique.  From our curent toolkit, we could just lift the uuid that NiFi creates.  In the case that someone is creating a flow from scratch/handwriting, they would be responsible for generating a unique one.  This should alleviate the woes we have run into, mapping more closely to NiFi's component "namespace".
This will require thoughts and handling of how to take existing flows that may not have an ID as generated by the previous toolkit.
We should additionally maybe start including some metadata about the YAML format such that we can make decisions in transform/loading that allow us to evolve.

@apiri thinking of adding version tag to yml, think we should stick with MiNiFi versioning for it?
e.g. use 0.1.0 for current master?

I think incremental is probably most straightforward and would likely just call this version 2 if that seems fair.

Hey @brosander, I started taking a look at the PR and there is a lot of great work here. When looking through the PR I see that the IDs are only for connections. I'm thinking it would make more sense to just blanket add them to everything instead of just saying "Oh those are just for connections". I believe it would make more sense to the user and would let us address MINIFI-106 too.
Thoughts @apiri?
Also a comment on the PR, don't forget to add documentation about config schema changes to the "System_Admin_Guide.md" in the docs module.

Sounds fair.  I wonder if it makes sense to treat the IDs as optional properties overall where we use kind of the current way to minimize pain and allow custom specification for advanced cases.  I realize the file isn't the end desire for configuration, but in the interim, making the format as friendly as possible would be helpful.

@apiri makes sense to me

Closing in favor of #33


Runtime error when calculate segmatic loss(HTC)

An error occurs in calculating the semantic loss of the HTC model's forward train.
The error is due to the difference in the prediction of semantic head and  gt_seg.
Is there a way to solve this?

What error? What's your config?

The error is shown below.

and, config file is "configs/htc/htc_dconv_c3-c5_mstrain_400_1400_x101_64x4d_fpn_20e.py"


If canRead returns false, nodes will timeout

Related #https://github.com/dao-xyz/peerbit/issues/126.
When canRead returns false, a response should nevertheless be returned to notify the user that they can not read.

Solved in https://github.com/dao-xyz/peerbit/commit/573688d51b9b8566e58b3c40d6b7854a515cb665


solang now requires data account to the first account

Signed-off-by: Sean Young<EMAIL_ADDRESS>

https://github.com/hyperledger/solang/pull/1039

Why are you disabling some tests?

Why are you disabling some tests?

Events are no longer working with this library.

I've added them to the github workflow (yarn lint checks both)

Events are no longer working with this library.

Should we fix this?

Should we fix this?

I thought about that too. I thought I would have a look once your work is merged.


ocsp: rotate certs of test data to fix flaky tests

Commit Message: ocsp: rotate certs of test data to fix flaky tests
Additional Description:
See #23632.
Risk Level: none. test only update.
Testing: n/a.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

cc @ggreenway cc @lizan
Almost all PRs was blocked by the flaky test and this PR should fixed it.

cc @alyssawilk Here fixed the flaky ci.

i wonder if we can give these certs a longer ttl so we dont need to update them so frequently

i wonder if we can give these certs a longer ttl so we dont need to update them so frequently

I agree. Now it's two years, I think. For test, I think a longer ttl is completely ok.

We can update these certs refresh script with a pr to achieve this target. Or even generate certs dynamically at runtime?

either way seems ok - not sure how long they take to generate - but probably they can just be generated dynamically

I'd prefer to have the certs generated by bazel and not checked into the repo; this would fix the issue that old versions of envoy will always fail tests.


Search integration

See how good WP+ACF+WPML is.
In principle I’ve seen this should work, but I fear it might be unpredictable. We may just have to try, which is the aim of the sprint.

https://wordpress.org/support/topic/only-partial-results-in-english/
https://wpml.org/forums/topic/translated-pages-do-not-appear-in-search-results/page/3/

A very initial exploration of WPML+WP for search.
Searching fusion

Spanish: https://dev-science-in-school.pantheonsite.io/?s=fusion&lang=es
English: https://dev-science-in-school.pantheonsite.io/?s=fusion&lang=en

We'd need to adapt the search template, but that should be fine.
For ACF content the search works but a snippet/excerpt isn't being shown.

https://dev-science-in-school.pantheonsite.io/?s=english
https://dev-science-in-school.pantheonsite.io/?s=intentar&lang=es

I've tried using the default WP theme and the behaviour is the same.
I also didn't see any helpful options in https://dev-science-in-school.pantheonsite.io/wp-admin/options-general.php?page=acfbs_admin_page
I did see that this plugin may be of some help https://wordpress.org/plugins/relevanssi/
Conclusions:

The ACF+WPML search does work
There are some cosmetic things to be sorted

A very initial exploration of WPML+WP for search.
Searching fusion

Spanish: https://dev-science-in-school.pantheonsite.io/?s=fusion&lang=es
English: https://dev-science-in-school.pantheonsite.io/?s=fusion&lang=en

We'd need to adapt the search template, but that should be fine.
For ACF content the search works but a snippet/excerpt isn't being shown.

https://dev-science-in-school.pantheonsite.io/?s=english
https://dev-science-in-school.pantheonsite.io/?s=intentar&lang=es

I've tried using the default WP theme and the behaviour is the same.
I also didn't see any helpful options in https://dev-science-in-school.pantheonsite.io/wp-admin/options-general.php?page=acfbs_admin_page
I did see that this plugin may be of some help https://wordpress.org/plugins/relevanssi/
Conclusions:

The ACF+WPML search does work
There are some cosmetic things to be sorted

Enabling the Relevanssi plugin gives the expected functionality with ACF snippets being rendered.
It also gives easy access to things like "boosting" and weights along with stopwords, highlights, etc

As always performance will be something to watch for, but I think this plugin is worth further investigation.

Enabling the Relevanssi plugin gives the expected functionality with ACF snippets being rendered.
It also gives easy access to things like "boosting" and weights along with stopwords, highlights, etc

As always performance will be something to watch for, but I think this plugin is worth further investigation.


Willing to pay someone to help me set this up - discord: aleferra#5464

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Additional context
Add any other context here.

me too email me<EMAIL_ADDRESS>


[Jetsurvey] Email TextField goes outside visible region

On clicking Email TextField in the initial screen, the TextField is scrolled above the device display region.
Probable reason - Issues with small display size devices.
https://user-images.githubusercontent.com/29737108/140621077-e51656ce-5429-46ef-9ca7-f221c5fab7d0.mov
Device details - Xiaomi Redmi 4, Android 7.1.2, API 25

Yes i noticed this to. I don't think this has anything to do with device.

Have been fixed by https://github.com/android/compose-samples/pull/779

Thanks for calling that out, Tsuharesu!


The doctor check crashed

flutter doctor complains it crashes when checking ItelliJ IDEA Community Edition.
Steps to Reproduce

simply run "flutter doctor" in terminal.
...
...

P.S. I don't care about android development, so I disabled it via flutter config --no-enable-andoird.
Logs

Logs

$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.10.1, on macOS 12.2.1 21D62 darwin-x64, locale en-CN)
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
[✓] Chrome - develop for the web
[☠] IntelliJ IDEA Community Edition (the doctor check crashed)
    ✗ Due to an error, the doctor check did not complete. If the error message below is not helpful, please let us know about
      this issue at https://github.com/flutter/flutter/issues.
    ✗ FormatException: Unexpected extension byte (at offset 5)
[✓] VS Code (version 1.64.2)
[✓] Connected device (2 available)
[✓] HTTP Host Availability

! Doctor found issues in 1 category.

$ flutter doctor -v
[✓] Flutter (Channel stable, 2.10.1, on macOS 12.2.1 21D62 darwin-x64, locale en-CN)
    • Flutter version 2.10.1 at /Users/stone/devel/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db747aa133 (10 days ago), 2022-02-09 13:57:35 -0600
    • Engine revision ab46186b24
    • Dart version 2.16.1
    • DevTools version 2.9.2
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[☠] IntelliJ IDEA Community Edition (the doctor check crashed)
    ✗ Due to an error, the doctor check did not complete. If the error message below is not helpful, please let us know about
      this issue at https://github.com/flutter/flutter/issues.
    ✗ FormatException: Unexpected extension byte (at offset 5)
    • #0      _Utf8Decoder.convertSingle (dart:convert-patch/convert_patch.dart:1789:7)
      #1      Utf8Decoder.convert (dart:convert/utf.dart:351:42)
      #2      InputStream.readString (package:archive/src/util/input_stream.dart:207:30)
      #3      new ZipDirectory.read (package:archive/src/zip/zip_directory.dart:40:30)
      #4      ZipDecoder.decodeBuffer (package:archive/src/zip_decoder.dart:19:30)
      #5      ZipDecoder.decodeBytes (package:archive/src/zip_decoder.dart:14:12)
      #6      IntelliJPlugins._findPluginXml (package:flutter_tools/src/intellij/intellij.dart:130:44)
      #7      IntelliJPlugins._readPackageVersion (package:flutter_tools/src/intellij/intellij.dart:141:40)
      #8      IntelliJPlugins.validatePackage (package:flutter_tools/src/intellij/intellij.dart:63:35)
      #9      IntelliJValidator.validate (package:flutter_tools/src/intellij/intellij_validator.dart:103:15)
      #10     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:111:32)
      #11     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:109:18)
      #12     _rootRun (dart:async/zone.dart:1426:13)
      #13     _CustomZone.run (dart:async/zone.dart:1328:19)
      #14     _runZoned (dart:async/zone.dart:1861:10)
      #15     runZonedGuarded (dart:async/zone.dart:1849:12)
      #16     runZoned (dart:async/zone.dart:1780:12)
      #17     asyncGuard (package:flutter_tools/src/base/async_guard.dart:109:3)
      #18     Doctor.startValidatorTasks (package:flutter_tools/src/doctor.dart:205:9)
      #19     Doctor.diagnose (package:flutter_tools/src/doctor.dart:309:47)
      #20     DoctorCommand.runCommand (package:flutter_tools/src/commands/doctor.dart:50:48)
      #21     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1320:12)
      <asynchronous suspension>
      #22     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1161:27)
      <asynchronous suspension>
      #23     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #24     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
      <asynchronous suspension>
      #25     FlutterCommandRunner.runCommand.<anonymous closure>
      (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
      <asynchronous suspension>
      #26     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #27     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
      <asynchronous suspension>
      #28     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
      <asynchronous suspension>
      #29     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #30     main (package:flutter_tools/executable.dart:94:3)
      <asynchronous suspension>

[✓] VS Code (version 1.64.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 98.0.4758.102

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Oh, I missed the IDEA version:
IntelliJ IDEA 2021.3.2 (Community Edition)
Build #IC-213.6777.52, built on January 28, 2022
Runtime version: 11.0.13+7-b1751.25 x86_64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 12.2.1
GC: G1 Young Generation, G1 Old Generation
Memory: 750M
Cores: 12
Non-Bundled Plugins:
    org.jetbrains.kotlin (213-1.6.10-release-961-IJ6777.52)
    Dart (213.5744.122)
    io.flutter (64.1.4)

Kotlin: 213-1.6.10-release-961-IJ6777.52

I have same issue
My MacBook: M1  Monterey verson.12.1

and I run
flutter doctor -v

Same issue here ...
[☠] IntelliJ IDEA Ultimate Edition (the doctor check crashed)
✗ Due to an error, the doctor check did not complete. If the error message below is not helpful, please let us know about this issue at
https://github.com/flutter/flutter/issues.
✗ FormatException: Unexpected extension byte (at offset 5)

@arloan and everyone in this thread facing this issue, please note below:
This was a known issue that has since been fixed. The initial report was in https://github.com/flutter/flutter/issues/94060.
The fix is available on the master channel but not yet rolled out to stable. Check this issue comment for your reference: https://github.com/flutter/flutter/issues/94060#issuecomment-1018801980
I'm going to close this since it has already been fixed.


Camera feeds send data for half a second then become unstable.

Not sure if this is a problem with my cameras, configuration, or network, but has anyone had a similar problem?
Launching two cameras by specifying their names. I don't have a config file but have set format, width and height in my launch file.
Seems to work with a very strong internet connection but I want to optimize this.

Thank you very much for your help. ros2 topic bw is making things make a lot more sense, and i'm averaging about 1.5 MB/s per stream. I may come back and detail what worked for me, but this is not an issue with camera_ros so I think its fine to close for now.
Again, thanks for your advice.


2Spades Ban Appeal

Hey I was perm banned for Stealing and griefing today when I contacted a mod. I tried appealing my temp ban because I was sure that i was falsely accused. There werent any records of me breaking anything, But it showed that I had taken a few items (which I have no recollection of doing, especially since i didnt want to get banned in the first place). I have never once in my life been banned from a server and never have with intent broken rules. If you could, could I be unbanned. Ive just wanted to play pixelmon on the server because this was my first chance to actually be able to experience.  Id really appreciate if I could join the server again.

We will be reviewing this, you'll get some news soon.

Alright thanks

Um this hasnt been resolved
Ive been waiting for a response

I apologize. You've now been unbanned, and this should have been done way sooner. Let me know if you need anything else, my discord is Rasgnarok#6969.


Wrong error message when trying to send all your tokens

Problem
If I try to send all my tokens to some other account, it will silently fail and in the console it shows
Error: {}
    at Object.u [as parseRpcError] (rpc_errors.js:25)
    at i.sendJsonRpc (json-rpc-provider.js:60)
    at async f.signAndSendTransaction (account.js:75)

Expected Behavior
Report the real error to users (LackBalance).
Steps to reproduce

Go to wallet.betanet.near.org and register an account.
Try sending all the tokens to some other existing account.

@vgrichina @kcole16 First of all, why sending all fail?

This actually should be fixed by this: #481 since we'll prevent user from sending their entire balance.
Basically, this error is due to user trying to remove their state stake (minimum balance) without deleting their account, which is a requirement to do so.
Also, I can't find the locked balance PR anymore. Did we remove it?

Also, I can't find the locked balance PR anymore. Did we remove it?

It's here: #534

Right, realized that after posting my message!

Basically, this error is due to user trying to remove their state stake (minimum balance) without deleting their account, which is a requirement to do so.

But it's already handled with #534 changes. Right?

@kcole16 @vgrichina Additionally, locally and on testnet we receive Timeout error so I'm not sure how can we handle it?

Closed by #534


Provide IP update service.

References https://github.com/metal-stack-cloud/api-server/issues/217.

@iljarotar @Pilzington With this the console must be modified


Create version that works with PHPUnit 7

Oops. I am blind >_>
https://github.com/whatthejeff/nyancat-phpunit-resultprinter/pull/13


Version required warning label change

Summary of the Pull Request
Changed version required warning label, rebuilded bundle.js

PR Checklist

[x] Applies to #2987 #2967
[x] CLA signed. If not, go over here and sign the CLA

Validation Steps Performed
Manually tested

Let's merge it.


Crash on Navigator.push when current route has nested Navigator containing pages with Heros

Minimal example:
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) => MaterialApp(home: Root());
}

class Root extends StatelessWidget {
  @override
  Widget build(BuildContext context) => Navigator(
        onGenerateRoute: (settings) => MaterialPageRoute(builder: (context) => Page1()),
      );
}

class Page1 extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: GestureDetector(
          onTap: () => Navigator.of(context).push(MaterialPageRoute(builder: (context) => Page2())),
          child: Hero(
            tag: "superman",
            child: Container(width: 40, height: 40, color: Colors.red),
          ),
        ),
      ),
    );
  }
}

class Page2 extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Hero(
            tag: "superman",
            child: GestureDetector(
                onTap: () => Navigator.of(context, rootNavigator: true)
                    .push(MaterialPageRoute(fullscreenDialog: true, builder: (context) => Modal())),
                child: Container(width: 60, height: 60, color: Colors.red))),
      ),
    );
  }
}

class Modal extends StatelessWidget {
  @override
  Widget build(BuildContext context) => Scaffold(backgroundColor: Colors.purple);
}

Tapping the red square twice results in:
flutter: ══╡ EXCEPTION CAUGHT BY SCHEDULER LIBRARY ╞═════════════════════════════════════════════════════════
flutter: The following assertion was thrown during a scheduler callback:
flutter: There are multiple heroes that share the same tag within a subtree.
flutter: Within each subtree for which heroes are to be animated (typically a PageRoute subtree), each Hero
flutter: must have a unique non-null tag.
flutter: In this case, multiple heroes had the following tag: superman
flutter: Here is the subtree for one of the offending heroes:
flutter: # Hero(tag: superman, state: _HeroState#0b567)
flutter: # └KeyedSubtree-[GlobalKey#5c93c]
flutter: #  └Container(bg: BoxDecoration(color: MaterialColor(primary value: Color(0xfff44336))), constraints: BoxConstraints(w=40.0, h=40.0))
flutter: #   └ConstrainedBox(BoxConstraints(w=40.0, h=40.0), renderObject: RenderConstrainedBox#48feb relayoutBoundary=up4)
flutter: #    └DecoratedBox(bg: BoxDecoration(color: MaterialColor(primary value: Color(0xfff44336))), dependencies: [_LocalizationsScope-[GlobalKey#91878], Directionality, MediaQuery], renderObject: RenderDecoratedBox#7672f)
flutter:
flutter: When the exception was thrown, this was the stack:
flutter: #0      Hero._allHeroesFor.visitor.<anonymous closure> (package:flutter/src/widgets/heroes.dart:210:15)
flutter: #1      Hero._allHeroesFor.visitor (package:flutter/src/widgets/heroes.dart:220:12)

In our case Root is a widget having several tabs, each with their own Navigator, while the root Navigator is used to (among other things) show modals route on (which should cover the tabbar).
This seems to be the cause of #25362 and #25218, but the issue is more generic.
[✓] Flutter (Channel master, v1.2.2-pre.21, on Mac OS X 10.14.3 18D109, locale en-NL)
    • Flutter version 1.2.2-pre.21 at /Users/spkersten/Development/flutter
    • Framework revision 035e0765cc (3 hours ago), 2019-02-16 01:48:28 -0500
    • Engine revision f37b09a11b
    • Dart version 2.1.2 (build 2.1.2-dev.0.0 c92d5ca288)

Possible solutions:

Associate each Hero with a specific HeroController.
a. HeroController gets an optional list of Hero tags it should animate. That list would have to be injected through MaterialApp. Or;
b. Hero and HeroController share some common "controller tag". In this case the MaterialApp could use a default controller tag that is used by default by Heros as well. HeroController only animates Heros with a controller tag equal this its controller tag. (Instead of a default controller tag, the absence of a controller tag could be used.)
When searching for Heros (Hero._allHeroesFor), always use the last found Hero for a tag, assuming that earlier found Heros are for routes further down the stack.

For 1a, one of the top-level widgets (whoever creates the HeroController) gets knowledge of widgets further down the tree (Heros). That sounds like bad design.
For 2, we lose the useful diagnostic warning about multiple Heros with the same tag. And the logic of finding the only visible Hero with a certain tag seems fragile/too implicit.
Option 1b sounds quit good though. It is entirely backwards compatible and it feel like a natural way to group Heros. A downside I see is that won't support using one Hero to fly on transition of more than one Navigator. I don't think that is a common use case though (certainly not one of ours) and can easily be solved by use two Heros (one for each Navigator). It also introduces a way of making mistakes: using mismatching controller tags. That is of a similar nature as the current Hero tags and is only relevant when actually using the controller tag.

The simplest solution here seems to be to just have Navigators not walk into nested Navigators when looking for heroes. I think that's a reasonable behaviour, because heroes are intended to work at a route level, and what would it even mean to transition a hero from a nested route?

@Hixie I'm not sure that solution would work for our case (or maybe only on the very short term). We have something like this:

Navigator:

Onboarding with Hero A
Post-onboarding, tab controller: (should replace previous page but not possible yet #28041)

Navigator:

First daily UI page with Hero A

Navigator:

Settings page 1 with Hero B
Settings page 2 with Hero B

The issue appears when showing a modal route on the root navigator from "Settings page 2".
Having a hero transition from an onboarding page to the first page of a daily UI doesn't strike me as unreasonable.

@spkersten I am not sure I understand the use cause you pointed out in your last comment. You have the root navigator, which shows your onboarding page with Hero A. From there, you want to push a new page with a tab bar onto the root navigator and animate that Hero A into a place in the first tab. Each tab here has there own nested navigator, so the hero ends up in nested navigator. Is this what you want?

@goderbauer Yes, that's what I meant.
To be fair, the first tab currently doesn't have a nested Navigator yet in our case. So what @Hixie proposed would work in the short term. But our app will eventually get that Navigator.

Understood. I am currently prototyping a fix for the crash that would still allow for the nested navigator case and is also transparent to the user (e.g. it doesn't require additional tags supplied by the user). Stay tuned.


[FEATURE] PnP commands to publish a SPO site template to the tenant

Is your feature request related to a problem? Please describe.
Problem is currently there is no way to publish a custom SPO modern template to the tenant.
I am looking for a way to use the .pnp package from the below command and use that to publish a custom SPO site template to the Tenant that appears under the "From your Organization" in the UI for end users to be able to choose an Organization template and apply. This would include lists, libraries, navigation, site logo as well as client-side custom pages.
Get-PnPSiteTemplate -out template.pnp -IncludeAllClientSidePages -PersistBrandingFiles
Describe the solution you'd like
Solution could be something like below two commands -
Import-PnPTenantTemplate -Path ./template.pnp
&
Publish-PnPTenantTemplate -Identity "Template Name" -Scope Organization
Describe alternatives you've considered
Current solution is for Admins to use this - Invoke-PnPSiteTemplate .\template.pnp.
End users cannot do this as they don't know PnP PS.
Publishing to the tenant allows end users to pick from any of the published organization templates in SPO.

@rachoksh - Are Site scripts & Site designs not what you're after?

@rachoksh - what you are looking is site templates and scripts.
You can read more about it here:
https://learn.microsoft.com/en-us/sharepoint/dev/declarative-customization/get-started-create-site-design
You can call PnP Provisioning from the site scripts. The provisioning needs to be run on a separate hosted environment or server. You can consider  Az function or runbooks or something else.


blue outline on slides

It looks like when you set dragEnabled={false} there is a blue outline when you toggle to the next page. I assume this is an accessibility feature but is there a way to get rid of this?

Hi @moontaekim, thanks for using Pure React Carousel!
This issue looks like a support question. As outlined in the question issue type, these types of issues will be closed because this is intended to be an issue tracker not a support forum. There are better places on the Internet to find support for questions like this.
If you'd like to request a feature or report a bug, feel free to open another issue and use the provided templates. Thanks again for using our carousel, and good luck!

Hey @moontaekim, did you ever find a solution to this?
I know I can usually switch the blue border of by using something like:
.my-class:focus {         border: none !important;         outline: none !important;     }
But I can't seem to get rid of this blue border.

Hi there,
I was not able to figure it out. I decided to use React-Slick instead.

Ah, thanks for the reply @moontaekim.
For others that might be facing the same problem, with the help of a colleague we were able to remove it, but it's not the cleanest solution:
The Slide component as has a .focusRing class which sets the outline. But trying to overwrite it directly or adding a new className to the Slide component  and overwriting it in that class also doesn't work. This however did:
 .focusRing___1airF.carousel__slide-focus-ring {         outline: none !important;     }
The class can be seen in the browser when a user clicks on it.

To me this sounds like a bug or a question on a design decision.

@HanliTheron in my case your variant to fix this outline don't work, but I add this style to other class:
.carousel__slider {  outline: none !important; }
It works fine for me))

Any idea how to hot fix it with styled components?

const StyledSlide = styled(Slide)`
  display: flex;
  width: 110px;
  min-width: 110px;
  min-height: 72px;
  .focusRing___1airF.carousel__slide-focus-ring {
    outline: none !important;
  }
`;

const StyledBackButton = styled(ButtonBack)`
  position: absolute;
  left: 0;
  top: 50%;
  transform: translateY(-50%);
  display: flex;
  width: 24px;
  justify-content: center;
  align-items: center;
  height: 100%;
  border: 0px;
  background: rgba(0, 0, 0, 0.5);
  color: #fff;
  outline: none !important;
`;

u can override styles for next/back button and for slide you can overrride using the answer above @HanliTheron

Shouldn't there be a toggle to turn this off?

@HarishTeens the outline is needed for accessibility compliance.  In general, you see that same outline all over the web on form elements when you tab through, unless the author has overridden the outline in their CSS.  Defeating the outline will make your carousel not compliant with current accessibility guidelines.  A major reason for creating the carousel was to adhere to accessibility guidelines.  Therefore, I doubt we'll add an option to turn this off anytime soon, unless guidelines change.

Thats great @mrbinky3000 . But sadly none of the above potential fixes work me :(

Oh my god I am sorry. There was an error with my css file. This fix .focusRing___1airF.carousel__slide-focus-ring { outline: none !important; } worked perfectly

Oh my god I am sorry. There was an error with my css file. This fix .focusRing___1airF.carousel__slide-focus-ring { outline: none !important; } worked perfectly

Simply referencing .carousel__slide-focus-ring will suffice


Fix update attribute

Proposed changes
This PR fixes update attribute.
Types of changes
What types of changes does your code introduce to the project: Put an x in the boxes that apply

[X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update only documentation, not any source code.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of
them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before
merging your code.

[X] I have read the CONTRIBUTING doc
[X] I have signed the CLA
[X] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you
did and what alternatives you considered, etc...

Coverage remained the same at 100.0% when pulling a62a13668e9577752830f81fc738f0b729ee63f0 on fisuda:feature/attr_command into 7f8837c1f8bf69ec3d4893ed01964cdf7a7c3808 on lets-fiware:main.

Coverage remained the same at 100.0% when pulling a62a13668e9577752830f81fc738f0b729ee63f0 on fisuda:feature/attr_command into 7f8837c1f8bf69ec3d4893ed01964cdf7a7c3808 on lets-fiware:main.


Added support for asynchronous usage via ApifyClientAsync

The Python SDK will use async methods by default, and for it to be effective, we need to support async usage in the Python Client as well.
This PR adds ApifyClientAsync, a sibling to ApifyClient with all methods being async. The usage and structure is the same as in ApifyClient, but unfortunately, a lot of the code had to be mostly duplicated, because we had to add an await here and there, and we had to adjust the typings a bit. At least the docs don't have to be duplicated, I made a helper decorator for that.
The downside is that now, if we add features to the client, we will have to add it to both the sync and the async version. Hopefully in the future we will remove the sync client and have only the async one.
You don't need to review every changed line, it's enough to check the changes in the main logic, not in the actual resource clients.

BTW, have you looked at https://pypi.org/project/asgiref/#function-wrappers ? I wonder if we could just have async implementation and use "async to sync" for the sync client?


Lightbox full screen option cutting off content at the top

Hi,
I'm using the full screen option on your bs5-lightbox module to display some HTML content but it's cutting off at the top. Example here (click on 'view plant list' button half way down the page): http://fermionx.britwebtest.co.uk/electronic-contract-manufacturing/pcba/
The green border is the modal-body class provided by Bootstrap 5 and is the right height. The red border is the lightbox-carousel class from the bs5-lightbox module and is a child of modal-body. This is causing the sizing problem.
On further investigation, I can see that's it's positioned absolutely with a top 50% and left 50% and using CSS translate to move it up and left 50% back to the edge of the browser. This is done with the bootstrap utility classes translate-middle, top-50 and start-50. Removing these classes actually resolves the problem as the positioning reverts to the default top 0 and left 0.
I don't really want to modify the module directly as my changes will be overwritten if/when it gets updated so could you please apply this fix to the module or suggest an alternative solution or explanation as to why this problem is occurring? I look forward to your reply, thank you.

Not really the solution for your problem, but just wanted to let you know that you can safely "modify" the module with patch-package (https://www.npmjs.com/package/patch-package). You can just modify the things you need, run the patch-package command for this package and it will create a patch file in your own repo.
Then you can run patch-package in a postinstall hook and it will always try to apply the patch.
This should unblock you for now.
Alternatively you can fork the repo, make the change you described and then create a pull request.

@garryknapper can you please share how you solved the problem? I have a vertical image and is both cut at the top and scrollable 😞

@valii2334 I modified the module with patch package as described by @timvandijck. Look for the element with a class of lightbox-carousel and remove these classes from that element: translate-middle, top-50, start-50. That should resolve the sizing problem and remove the scrolling as well as fixing the top being cut off.


Reduce readme

Now that we have wiki pages, all we need is links to the them

At some point we should consider locking down the wikis to only contributors


Component Customization

I  customize the component and how can i Export as Jar file.
Ex:

Hello,
You can build JFoenix using gradlew :jfoenix:build which will generate your cusomized jfoenix.jar.
Regards,

Hi JFoenix, Im big fan of you, and u really make awesome job with javaFX. My wishes for your future JFoenix updates. So can u please just give me a clear instructions or a simple tutorial for that please.

Hello,
this link will help to get you started in developing custom controls in JavaFX:
http://www.guigarage.com/2012/11/custom-ui-controls-with-javafx-part-1/
Regards,

you need to use gradle and run the following command:
gradlew :jfoenix:build

Bro Thank you very much. Can i know when your new update available?

Bro i use "gradlew :jfoenix:build" to build it, Then where can i find the jar file ?

I build a JAR file with your  help. Now i need to customize the JFXTabpane. I just want to remove the right and left control button. And i just need to keep drag only. Could you pls help me with that.


Specifying content type in request to application/x-www-form-urlencoded fails

What happened?
Specifying content type in request to application/x-www-form-urlencoded and setting the body to param1=val1&param2=val2 gives
Warning  CannotCreateExternalResource  4m7s (x28 over 24m)  managed/request.http.crossplane.io  something went wrong: unexpected token "&"

How can we reproduce it?

Deploying a request resource with following spec
apiVersion: http.crossplane.io/v1alpha1
kind: Request
metadata:
  name: example
spec:
  deletionPolicy: Delete
  forProvider:
    mappings:
      - headers:
          Content-Type:
            - application/x-www-form-urlencoded
        method: POST
        body: param1=val1&param2=val2
        url: >-
          (.payload.baseUrl + "/" +
          "some/path/")
    payload:
      baseUrl: 'https://some-url'
  managementPolicy: FullControl
  providerConfigRef:
    name: http-conf

What environment did it happen in?
Crossplane version: 1.14.0
Distribution: Openshift

Thanks for flagging this! After checking out the issue, it seems like the problem lies in how the body is set up in the request mappings.
In the provided Request resource, the body field within the mappings expects a jq filter. To fix this, just make sure to put the string in parentheses, like this:
apiVersion: http.crossplane.io/v1alpha1
kind: Request
metadata:
  name: example
spec:
  deletionPolicy: Delete
  forProvider:
    mappings:
      - headers:
          Content-Type:
            - application/x-www-form-urlencoded
        method: POST
        body: ("param1=val1&param2=val2")  # Enclose the string within parentheses
        url: >-
          (.payload.baseUrl + "/" +
          "some/path/")
    payload:
      baseUrl: 'https://some-url'
  managementPolicy: FullControl
  providerConfigRef:
    name: http-conf

That should sort out the unexpected token issue.
Oh, and I noticed you didn't include a GET mapping. Without it, the provider might struggle to validate that the resource is up to date. If you're looking to perform a one-time request, consider using the DisposableRequest resource instead. Here's a quick example:
apiVersion: http.crossplane.io/v1alpha1
kind: DesposibleRequest
metadata:
  name: example
spec:
  deletionPolicy: Orphan
  forProvider:
    url:  https://some-url/some/path
    method: POST
    body: param1=val1&param2=val2
    headers:
      Content-Type:
        - application/x-www-form-urlencoded
  providerConfigRef:
    name: http-conf

No need to worry about jq here; just specify the body without parentheses and you're good to go!
Hope this helps! Let me know if you have any further questions or concerns.

Hello, thanks ill check it out, i m working on doing a missing configuration in keycloak (feat missing in upstream tf provider), my workflow is

Generate a access token from keycloak api
Use access token to perform the missing configuration against keycloak

Oh, and I noticed you didn't include a GET mapping. Without it, the provider might struggle to validate that the resource is up to date.

If i understand it correctly, i would have to include get request as well making the overall flow to be

Generate a access token from keycloak api
Get configuration from keycloak
Use access token to perform the missing configuration against keycloak

Thanks for sharing your workflow! It sounds like you’re on the right track. Considering your scenario, it seems you’ll need to perform three requests:
1.	Use a DisposableRequest to retrieve the access token from the Keycloak API.
2.	Use another DisposableRequest to retrieve the configuration data from Keycloak.
3.	For your final action, whether it’s creating, updating, or deleting resources, you can choose between a DisposableRequest and a Request:
•	If you’re performing a one-time action without managing any resources, like sending a single request to Keycloak, use a DisposableRequest.
•	If you’re managing resources and need to ensure their state, such as creating or updating resources based on the retrieved configuration data, use a Request.

Just to clarify, DisposableRequest is typically used for one-time interactions, while Request is more suitable for managing resources through HTTP requests.
Let me know if you have any questions or need further clarification on implementing these steps.

ok, thanks for the comprehensive reply, it helps. Ill go with a combination of desposible request + request for performing this configuration.

I have a request that im making from cli against keycloak api using
curl -d 'client_id=admin-cli&username=admin&password=a1b2c3&grant_type=password' https://keycloak-instance/realms/master/protocol/openid-connect/token

creating this resource as replacement for above command but it fails
apiVersion: http.crossplane.io/v1alpha1
kind: DesposibleRequest
metadata:
  name: example-disposable-request
spec:
  deletionPolicy: Orphan
  forProvider:
    waitTimeout: 5m
    rollbackRetriesLimit: 5
    url:  https://keycloak-instance/realms/master/protocol/openid-connect/token
    method: POST
    body: |
            grant_type=password&client_id=admin-cli&password=a1b2c3&username=admin
    headers:
      Content-Type:
        - application/x-www-form-urlencoded
      Accept:
        - application/json
  providerConfigRef:
    name: http-conf

it might be something with the api but your insight would be helpful if any.

Could you please share the status of the DisposableRequest resource? It contains the response from the server, which will help us troubleshoot the issue.

Please find the error below :
  response:
    body: >-
      {"error":"unsupported_grant_type","error_description":"Unsupported
      grant_type"}

It seems the request was sent successfully, indicating a potential issue with the request payload.
Before proceeding, could you please provide the entire status of the DisposableRequest resource? This will help us diagnose the problem more accurately.

Please find the status of DesposableRequest resource below,
status:
  conditions:
    - lastTransitionTime: '2024-03-19T14:06:23Z'
      reason: Available
      status: 'True'
      type: Ready
    - lastTransitionTime: '2024-03-19T14:07:23Z'
      reason: ReconcileSuccess
      status: 'True'
      type: Synced
  failed: 5
  requestDetails:
    body: |
      grant_type=password&client_id=admin-cli&password=admin&username=admin
    headers:
      Accept:
        - application/json
      Content-Type:
        - application/x-www-form-urlencoded
    method: POST
    url: >-
      https://keycloak-instance/realms/master/protocol/openid-connect/token
  response:
    body: '{"error":"invalid_grant","error_description":"Invalid user credentials"}'
    headers:
      Referrer-Policy:
        - no-referrer
      X-Xss-Protection:
        - 1; mode=block
      Strict-Transport-Security:
        - max-age=31536000; includeSubDomains
      X-Frame-Options:
        - SAMEORIGIN
      Content-Type:
        - application/json
      Set-Cookie:
        - >-
          63a25785f19a75e5b651f66a19c0b7aa=412610dce75fb0cc0c737424680b7a9f;
          path=/; HttpOnly; Secure; SameSite=None
      Cache-Control:
        - no-store
      X-Content-Type-Options:
        - nosniff
      Content-Length:
        - '72'
      Pragma:
        - no-cache
    statusCode: 401
  synced: true

status:
  conditions:
    - lastTransitionTime: '2024-03-19T14:52:06Z'
      reason: Available
      status: 'True'
      type: Ready
    - lastTransitionTime: '2024-03-19T14:52:36Z'
      reason: ReconcileSuccess
      status: 'True'
      type: Synced
  failed: 5
  requestDetails:
    body: |
      client_id=admin-cli&username=admin&password=admin&grant_type=password
    headers:
      Accept:
        - application/json
      Content-Type:
        - application/x-www-form-urlencoded
    method: POST
    url: >-
      https://keycloak-instance/realms/master/protocol/openid-connect/token
  response:
    body: >-
      {"error":"unsupported_grant_type","error_description":"Unsupported
      grant_type"}
    headers:
      Referrer-Policy:
        - no-referrer
      X-Xss-Protection:
        - 1; mode=block
      Strict-Transport-Security:
        - max-age=31536000; includeSubDomains
      X-Frame-Options:
        - SAMEORIGIN
      Content-Type:
        - application/json
      Set-Cookie:
        - >-
          63a25785f19a75e5b651f66a19c0b7aa=412610dce75fb0cc0c737424680b7a9f;
          path=/; HttpOnly; Secure; SameSite=None
      Cache-Control:
        - no-store
      X-Content-Type-Options:
        - nosniff
      Content-Length:
        - '79'
      Pragma:
        - no-cache
    statusCode: 400
  synced: true

Thanks for providing the details.
Looking at the requestDetails, it appears that the request sent from the provider closely resembles the curl command you mentioned.

  requestDetails:

    body: |

      client_id=admin-cli&username=admin&password=admin&grant_type=password

    headers:

      Accept:

        - application/json

      Content-Type:

        - application/x-www-form-urlencoded

    method: POST

    url: >-

      https://keycloak-instance/realms/master/protocol/openid-connect/token

The only differences I noticed are the Accept header and the password in the request body. These variances could be causing the issue. However, since the request was successfully sent, the problem likely doesn’t lie with the provider. If you have any other insights or need further assistance, feel free to let me know!

hey @arielsepton, thanks alot for your support
I tried sending the same request with kind: Request and its working, I'm guessing that kind: DisposableRequest somehow messes the request body before sending it.

I’m glad to hear that it worked for you with the Request resource!
However, it’s unlikely that the request body is being altered differently between the two resources, especially since the same HTTP client class is used to perform the request in both cases.
That said, there might be other factors at play here. One possibility could be the lack of support for jq filters in DisposableRequest.
If you need further explanations or assistance, don’t hesitate to reach out!

hmmm, might be possible
I did had another question, Whats a better way to provide credentials to this resource ? currently im including it in body, but i would want it to be read from a secret as variables which are then specified in the body
The examples in repo use injectedidentity that i dont really understand.

hmmm, might be possible
I did had another question, Whats a better way to provide credentials to this resource ? currently im including creds in body, but i would want it to be read from a secret as variables which are then specified in the body
The examples in this repo use injectedidentity that i dont really understand.

Currently, you can achieve this by leveraging provider-kubernetes. You can refer to this example for guidance: link.
While the feature of using secrets data directly is still in development, utilizing provider-kubernetes can fulfill this requirement in the interim.


Updated LIT + Gemma Colab for LIT v1.2.

Description of the change
Updates to the LIT + Gemma Colab to use the new functionality introduced in LIT v1.2.
Motivation
These changes address user feedback collected from Gemma DevRel and reported to the LIT team.
Type of change
Choose one: Documentation
Checklist

[x] I have performed a self-review of my code.
[x] I have added detailed comments to my code where applicable.
[x] I have verified that my change does not break existing code.
[x] My PR is based on the latest changes of the main branch (if unsure, please run git pull --rebase upstream main).
[x] I am familiar with the Google Style Guide for the language I have coded in.
[x] I have read through the Contributing Guide and signed the Contributor License Agreement.

@gustheman


Editor is extremely slow

If you use the editor for a few minutes and a few exercise, it gets slow unusable until yo refresh it.

No me resulta inusable pero sí me pasa que se vuelve un poco lento.

Fixed in https://github.com/mumuki/mumuki-bibliotheca/commit/8febd5290cdda3f5f164120848bfbfa5642bd6b6