[Bug]: Share_Plus IOS not showing the text i try to share

Platform
IOS 17
Plugin
share_plus
Version
9.0.0
Flutter SDK
3.19.3
Steps to reproduce
await Share.shareXFiles([XFile(tempPath)], subject: subject, text: body);
and u will see before u even choose the app (Plain text and 1 document)

Code Sample
final url = Uri.parse(imgToDownload);
    final response = await http.get(url);
    final tempDir = await getTemporaryDirectory();
    final shareImagesDir = Directory('${tempDir.path}/shareImages');
    if (!await shareImagesDir.exists()) {
      await shareImagesDir.create(recursive: true);
    }
    final tempPath = '${shareImagesDir.path}/imageFromUrl.jpg';

    List<int> bytes = response.bodyBytes;
    await File(tempPath).writeAsBytes(bytes);
    await Share.shareXFiles([XFile(${shareImagesDir.path}/imageFromUrl.jpg)], subject: subject, text: body); //NOT WORKING

    Share.share('check out my website https://example.com'); //WORKING WELL I SEE MY TEXT

Logs
### Logs

na

Flutter Doctor
[✓] Flutter (Channel stable, 3.19.3, on macOS 14.4.1 23E224 darwin-arm64, locale sv-SE)
    • Flutter version 3.19.3 on channel stable at /Users/USER/FSDK/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ba39319843 (8 weeks ago), 2024-03-07 15:22:21 -0600
    • Engine revision 2e4ba9c6fb
    • Dart version 3.3.1
    • DevTools version 2.31.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/USER/Library/Android/sdk/
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/USER/Library/Android/sdk/
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.15.2

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

[✓] Android Studio (version 2023.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)

[✓] VS Code (version 1.88.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.88.0

[✓] Connected device (3 available)
    • iPhone 15 Pro Max (mobile) • XXXXXX-BE42-4932-BCD2-XXXXXX • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-4
      (simulator)
    • macOS (desktop)            • macos                                • darwin-arm64   • macOS 14.4.1 23E224 darwin-arm64
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome 124.0.6367.118

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Checklist before submitting a bug

[X] I searched issues in this repository and couldn't find such bug/problem
[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I'm using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

@miquelbeltran I have spent time reading the README but there is no mention about the thing i face.
I DONT share to an app I just click on share button and don't choose an app, so the IOS built in share dialog what i'm talking about.

the apps listed in the share sheet are decided by the OS, not the plugin

the apps listed in the share sheet are decided by the OS, not the plugin

I know but i mean why i don't see the actual text instead of "plain text and 1 document"
Thanks

I believe it is because when sharing an image and a text, for UIActivityViewController is like sharing data items, and will display that (text and 1 document).
You can try to provide only the subject, or only the text, to see if that makes a difference.
If you can modify the iOS code, you can try to modify the implementation of the plugin when it comes to providing data to UIActivityViewController, e.g. implementing the metadata provider as shown here:  https://stackoverflow.com/a/58234878 The plugin doesn't support providing the metadata at the moment.


Stop sendin isTyping event when no actual typing is happening

If you press "alt" it handles it as a typing event, so when you alt+tab out of the browser, it's as if you were typing. I believe a typing event should happen when an actual character is typed.
Originally opened in #104.

That and/or a way to send STOP_TYPING/NOT_TYPING events which are useful when coupling this with a SocketIO chat.


K3s installer only checks install dir for kubectl

Describe the bug
The k3s installer installs a duplicate kubectl binary if the existing binary is not in the default BIN_DIR directory (/usr/local/bin).
To Reproduce
Steps to reproduce the behavior:

Install kubectl in your path in the /usr/bin/ directory
Run the installer script: https://raw.githubusercontent.com/rancher/k3s/master/install.sh

Expected behavior
The k3s installer does not install a symlink in /usr/local/bin/ entitled kubectl, and instead checks all directories in the user's path first.
Additional context
I use a number of different contexts with kubectl (remote staging and production clusters), so it was confusing to install k3s, and then no longer be able to access these remote clusters. I expected kubectl to use my ~/.kube/config file, but it instead used a different config file that wasn't visible to me.
An alternative (and arguably better) solution would be to install kubectl if it doesn't exist on the user's machine, and then update their local `~/.kube/config file to reference this cluster, similar to what https://cloud.google.com/sdk/gcloud/reference/container/clusters/get-credentials does.

Closing; appears to be fixed in #657


Expose build_system_rpc_future and TransactionPoolAdapator in sc-service

This change exposes necessary pieces to make replicating build_network possible as we'd like to disable the built-in block sync service so that we use another syncing approach while keeping the rest function of SyncService.

I don't see how these change help with custom NetworkBlock implementation but if they do that's great. I'm not the right person to review these changes though


Update new board URL

Depends on
None.
Description
Need to sync with web team to create a new landing page for the board/target URL embedded in DAPLink and submit a patch upstream with the new URL.
Todo

[ ] Check that web task is completed
[x] Update URL in our fork and test
[x] Send PR to upstreams DAPLink
[x] Follow up PR, respond/implement review comments and collaborate until merge

Completion Criteria

[ ] Website landing page (even if it is a redirect)
[x] New URL in DAPLink master for micro:bit builds

https://github.com/microbit-foundation/microbit-i18n/issues/852

https://github.com/microbit-foundation/microbit-i18n/issues/852

Waiting for the URL to be updated before closing this one.

Landing page redirect done in https://github.com/microbit-foundation/microbit-i18n/pull/960.


Disable class sharing for agent layer

The Java 8 Corretto runtime for Lambda explicitly enables class data sharing. We need to rewrite the JVM command line to append -Xshare:off to disable this behavior. Java 11 does not exhibit this problem.
https://stackoverflow.com/questions/65084786/override-jvm-options-in-aws-lambda

Hi @anuraaga,
Why CDS (class data sharing) needs to be disabled in Java 8?

@tylerbenson @rapphil I don't think we need to do something on this? WDYT? Should we close this issue?


113 Improve reporting of test failures for CQL tests

Closing in favor of https://github.com/datastax/adelphi/pull/159/files


Allow subscribing to Multi using callbacks

Add methods to subscribe to a multi using callbacks.

Fixed in #1


Shrink the memory request down from 4G to 500M

4G as a default request will make it hard to schedule in small envs

https://github.com/rancher/rancher/pull/11762


Added vanilla injection

In order to have a base reference to compare all frameworks, I have added simple object creation with native constructor.

Thank you very much! Can you please catch up the master changes and submit your pull request again? Much appreciated!

I think there is a problem with vanilla container: you made it serves as like A is a Singleton. But it is not the case actually, as per JSR330, only bean class or factory method with annotation @javax.inject.Singleton shall be treated as singleton. Otherwise, it shall construct the new instance as per request. This is also true for all other containers that participates the test.
I've added identity test case in the unit test to specifically make sure the bean is not the same object when invoke get method twice.

I made some changes to be compliant with JSR330.
Corrected the conflicts for this PR too.

Thanks for the contribution! I've merged and updated the README.md file with vanilla benchmark data


add meta function to clear known lru_cache instances

Codecov Report

Merging #34 (f988b09) into master (fd49922) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff            @@
##            master       #34   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        11    +1
  Lines          682       689    +7
=========================================
+ Hits           682       689    +7

Impacted Files
Coverage Δ

courlan/meta.py
100.00% <100.00%> (ø)

courlan/urlstore.py
100.00% <100.00%> (ø)


OCM-6092 | fix: add "create_admin_user" in cluster datasource

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story (OCM-xxxx):
Fixes #OCM-6092
Change type

[ ] New feature
[x] Bug fix
[ ] Build
[ ] CI
[ ] Documentation
[ ] Performance
[ ] Refactor
[ ] Style
[ ] Unit tests
[ ] Subsystem tests

Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.

/retest

/retest

/retest

/approve


Update version: CologneCodeCompany.XYplorer version 26.40.0400

Pull request has been created with komac v2.6.0 :rocket:

Resolves #186075

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Scrollbar search map

Has next bugs:

black if there are no matches - should not be shown at all
transparent if no matches - should not be transparent

#1752


Upgrading to Clerk Core 2

Part of: https://github.com/colonial-heritage/colonial-collections/pull/587
In April 2024, Clerk updated their SDKs to include Core 2. This new core includes:

💅 Redesigned and improved component UIs.
🔪 The 401 & "flash of white page" that was visible sometimes when syncing auth state was eliminated.
🧹 A large number of smaller housekeeping changes, bugfixes, and improvements.
📊 Telemetry to allow us to better determine feature usage and priority (easy opt-out if you don't like this)

Implementation

I followed the upgrade guild
There where a also lot changes in the translations files, I updated them with two commits, the first was a full replace with there translations. The second was an update with our changes. So if you want to check what I changed see commit: https://github.com/colonial-heritage/colonial-collections/commit/f1e4c4c4bab30c980bb637a52101674fa3f6b73c

We decided not to update Clerk for now.


[bug] Frontend reloads on wake after sleep in Windows 11

Describe the bug
When I put my computer to sleep and wake it up again all state has been lost on the frontend of my Tauri application. The page seems to be reloaded. The state is till available for the "backend" of my application.
Reproduction
On windows 11, start your Tauri app, then put the computer to sleep, then wake it up again. State on frontend is lost.
Expected behavior
No response
Platform and versions
[✔] Environment
    - OS: Windows 10.0.22621 X64
    ✔ WebView2: 119.0.2151.72
    ✔ MSVC:
        - Visual Studio Build Tools 2019
        - Visual Studio Professional 2022
    ✔ rustc: 1.74.0 (79e9716c9 2023-11-13)
    ✔ cargo: 1.74.0 (ecb9851af 2023-10-18)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 20.8.1
    - npm: 10.2.0

Stack trace
No response
Additional context
No response

Same behavior on our build: https://github.com/GregorGT/RuNo/
(Adding the comment to stay updated)

Is this a WebView2 or a Tauri related issue?
This feels like it should be a much more common issue for people. Can't be just me and @GregorGT.

@ottosson Sadly it is a chromium bug.
Basically I've tried at 3 browsers (brave, edge, firefox) and tauri
Brave, edge and tauri (windows) does reload the page
Firefox doesn't
That's my findings, lemme open source the code I've used

Here's the repo https://github.com/ahqsoftwares/tauri-webview-reload-test

Conversation: https://discord.com/channels/616186924390023171/731495028677148753/1246008474039619596
It apparently does not happen in production, I've confirmed that as well

If it is a Chromium bug shouldn't Electron be affected? I haven't experienced it there.

The bug is not a damper for production. I stumbled on that as well @ahqsoftwares.
I am going to unsubscribe.... as this is not really a bug. It only happens in dev.

Yup, same for me. A bit annoying but not as bad as I first feared.

@ottosson soooo, I guess this issue is more/less resolved? Cuz it doesn't happen in prod

@ottosson soooo, I guess this issue is more/less resolved? Cuz it doesn't happen in prod

Ok I'll drop this for now 😉
Thanks for helping shed some light on the situation @ahqsoftwares


Loop in atom updater

Each time I start atom (for platformio), I get a window that pops up saying:

PlatformIO IDE has been successfully installed!
However, some of its components will only become available after Atom window reload. You can click "Reload now" button below to perform reload immediately, or click "Reload later" and perform reload yourself with "View > Reload" command whenever you're ready.

with a "Reload now" and "Reload later" button. If I reload now, it restarts the IDE and then pops up again. If I press "Reload later" it goes away until I restart atom.
Versions: IDE 1.0.4 | CLI 2.8.4
Running on Debian stretch/sid

Please move issue to appropriate repository https://github.com/platformio/platformio-atom-ide

Whoops, sorry about that. Moved here: https://github.com/platformio/platformio-atom-ide/issues/63


bring TNDD #6, 7, 10, 12, 13, 20, 23,  over to TNND #32 to #38

requested: 2024-08-16

6
7
10
12
13
20
23

completed:

Rules copied over
6 > 32 added $presibnode1 to TNND brk
7 > 33  pre-inline already available
10 > 34
12 > 35  post-inline already available
13 > 36  pre-inline already available
20 > 37
23 > 38

SC:
I added examples of #32 in TNND XXD 32. Looks good.
I added examples of #33 & 34 in TNND XXD 33. Looks good.
I added examples of #36 in TNND XXD 35. Looks good. I attempted to make #35 work, but I can't remember the conditions, and I can't search the TNDD tag-errors view for a #7. Could you put an example flagged by #35 in TNND XXD 35?
I added examples of #37 in TNND XXD 37. Looks good.
I added examples of #38 in TNND XXD 38. Looks good.
I added examples of #38 in TNND XXD 38. Looks good.

I added tests for 35 and 36 in \c 35
Can I renumber 32-34 and 39 for \brk and also 35and 36 for \imp and also 37 and 38 as just char generic.?


The location of Garden Cottage, Shipston Road, CV35 0TR is wrong. I added Tysoe Walled Kitchen Garden to OSM in May 2014. 'Garden' could be used to place the location in the correct place in this case rather than being driven by the post code centroid.

Moved here from https://github.com/OpenAddressesUK/theodolite/issues/47 .

Closed as duplicate of https://github.com/OpenAddressesUK/forum/issues/10 .


TokenRefresher

How to pass a TokenRefresher?
Example: https://github.com/ili101/PartnerCustomerCommunity/commit/fb5d0460378ff0658bb1d51c95be6eb60f19c4fa.
Error when the class try to refresh the token:
Message : There is no Runspace available to run scripts in this thread. You can provide one in the DefaultRunspace property of the  System.Management.Automation.Runspaces.Runspace type. The script block you attempted to invoke was:  Write-Host 'In'.

Exception             :
    Type           : System.Management.Automation.MethodInvocationException
    ErrorRecord    :
        Exception             :
            Type    : System.Management.Automation.ParentContainsErrorRecordException
            Message : Exception calling "Get" with "0" argument(s): "Refreshing the credentials has failed."
            HResult : -2146233087
        CategoryInfo          : NotSpecified: (:) [], ParentContainsErrorRecordException
        FullyQualifiedErrorId : PartnerException
        InvocationInfo        :
            ScriptLineNumber : 115
            OffsetInLine     : 12
            HistoryId        : -1
            ScriptName       : C:\Users\illym\Documents\Git\PartnerCustomerCommunity\PartnerCustomerCommunity.psm1
            Line             : Return $PartnerOperations.Profiles.OrganizationProfile.$Get()

            PositionMessage  : At C:\Users\illym\Documents\Git\PartnerCustomerCommunity\PartnerCustomerCommunity.psm1:115 char:12
                               +     Return $PartnerOperations.Profiles.OrganizationProfile.$Get()
                               +            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
            PSScriptRoot     : C:\Users\illym\Documents\Git\PartnerCustomerCommunity
            PSCommandPath    : C:\Users\illym\Documents\Git\PartnerCustomerCommunity\PartnerCustomerCommunity.psm1
            CommandOrigin    : Internal
        ScriptStackTrace      : at Get-PartnerOrganizationProfile, C:\Users\illym\Documents\Git\PartnerCustomerCommunity\PartnerCustomerCommunity.psm1: line 115
                                at <ScriptBlock>, <No file>: line 1
    TargetSite     :
        Name          : ConvertToMethodInvocationException
        DeclaringType : System.Management.Automation.ExceptionHandlingOps, System.Management.Automation, Version=<IP_ADDRESS>0, Culture=neutral, PublicKeyToken=31bf3856ad364e35
        MemberType    : Method
        Module        : System.Management.Automation.dll
    Message        : Exception calling "Get" with "0" argument(s): "Refreshing the credentials has failed."
    Data           : System.Collections.ListDictionaryInternal
    InnerException :
        Type           : Microsoft.Store.PartnerCenter.Exceptions.PartnerException
        ErrorCategory  : Unauthorized
        Context        : Request Id: 652fcd4a-f82f-436c-af6f-53ad9abe8e85, Correlation Id: 69afef9b-2491-42fd-bdd9-cb96739c7d6c, Locale: en-US
        TargetSite     :
            Name          : SynchronousExecute
            DeclaringType : Microsoft.Store.PartnerCenter.PartnerService
            MemberType    : Method
            Module        : Microsoft.Store.PartnerCenter.dll
        Message        : Refreshing the credentials has failed.
        InnerException :
            Type        : System.Management.Automation.PSInvalidOperationException
            ErrorRecord :
                Exception             :
                    Type    : System.Management.Automation.ParentContainsErrorRecordException
                    Message : There is no Runspace available to run scripts in this thread. You can provide one in the DefaultRunspace property of the
System.Management.Automation.Runspaces.Runspace type. The script block you attempted to invoke was:  Write-Host 'In'
                    HResult : -2146233087
                CategoryInfo          : InvalidOperation: (:) [], ParentContainsErrorRecordException
                FullyQualifiedErrorId : ScriptBlockDelegateInvokedFromWrongThread
            TargetSite  :
                Name          : GetContextFromTLS
                DeclaringType : scriptblock
                MemberType    : Method
    OffsetInLine     : 12
    HistoryId        : -1
    ScriptName       : C:\Users\illym\Documents\Git\PartnerCustomerCommunity\PartnerCustomerCommunity.psm1
    Line             : Return $PartnerOperations.Profiles.OrganizationProfile.$Get()

    PositionMessage  : At C:\Users\illym\Documents\Git\PartnerCustomerCommunity\PartnerCustomerCommunity.psm1:115 char:12
                       +     Return $PartnerOperations.Profiles.OrganizationProfile.$Get()
                       +            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    PSScriptRoot     : C:\Users\illym\Documents\Git\PartnerCustomerCommunity
    PSCommandPath    : C:\Users\illym\Documents\Git\PartnerCustomerCommunity\PartnerCustomerCommunity.psm1
    CommandOrigin    : Internal
ScriptStackTrace      : at Get-PartnerOrganizationProfile, C:\Users\illym\Documents\Git\PartnerCustomerCommunity\PartnerCustomerCommunity.psm1: line 115
                        at <ScriptBlock>, <No file>: line 1

What is your fix at the moment? I've been looking at
( the .NET samples)[https://github.com/microsoft/Partner-Center-DotNet-Samples/blob/master/sdk/SdkSamples/Configuration/UserAuthenticationSection.cs]
and I can see the new-module requires a Refreshtoken without generating one

Im now trying to get this to work
https://docs.microsoft.com/en-us/partner-center/develop/enable-secure-app-model#get-authorization-code
followed hopefully by
https://docs.microsoft.com/en-us/partner-center/develop/enable-secure-app-model#get-refresh-token

So let's list the stages from the doc and see what we have

Create a web app
Get an authorization code
Get a refresh token
Get an access token
Make a Partner Center API call

Stages 1-3
Workaround: I correctly use this
Solution: Rewrite it without the deprecated modules, didn't look into it yest.
So if it's sending the token to https://localhost:44395/ in some stages we need a web server on this port, you can probably use Pode to retrieve it.
Stages 4
Solved: New-PartnerAccessToken is in this module
Stages 4 "Refresh"
Workaround: I don't know how to refresh it with the Dlls implementation, I didn't acutely need it as all my scrips run under 1 hour, and if I'm working on the terminal for long tine I just run Connect-PartnerCenter again.
Solution: This is the problem I created this Issue for, if you and I don't know how to solve it maybe I will post in the Powershell Repo/Discord for help?

Thank you for the very clear summation, I think I have some ideas on problem 4, but I need to fix number 2 first.
I'll look into both options
The Powershell Discord should be an excellent start yes

I looked at the code from the PartnerCustomer model, they don't actually use the replyUrlsWithType, they hardcode localhost and find a free port. I copped the code from there.
@koenhalfwerk dose everything work for you? If so do you want to clean, pack it up and PR?
Also see if you have idea on how to solve problem 4. thank you!
$Port = 8399;
while (++$port -lt 9000) {
    try {
        Start-PodeServer {
            Add-PodeEndpoint -Address localhost -Port $Port -Protocol Http
            Add-PodeRoute -Method POST -Path '/' -ScriptBlock {
                Write-PodeJsonResponse -Value $WebEvent.Data
            }
            # Close-PodeServer
        }
        break
    }
    catch {
        if ($_.Exception.Message -like '*Only one usage of each socket address*') {
            Write-Warning "Port $Port is taken with exception '$($_.Exception.Message)'; trying to connect to the next port."
        }
        else {
            throw $_
        }
    }
}

$Uri = 'https://login.microsoftonline.com/common/oauth2/authorize'
$Parameters = @{
    client_id     = $ApplicationCredential.UserName
    response_mode = 'form_post'
    response_type = 'code id_token'
    scope         = 'openid profile'
    nonce         = 1
    redirect_uri  = "http://localhost:$port/"
}

Neat idea with the ports, and does explain why my dual port with get and post  was breaking.
Does it actually return the code or does it still require a copy/pasta?
I'll combine what I can and make a pr

Not sure how to do that but we can do something like saving the token encrypted to Temp folder, then Close-PodeServer:
Start-PodeServer {
    Add-PodeEndpoint -Address localhost -Port $Port -Protocol Http
    Add-PodeRoute -Method POST -Path '/' -ScriptBlock {
        Write-PodeJsonResponse -Value $WebEvent.Data
        $WebEvent.Data.code | ConvertTo-SecureString -AsPlainText | ConvertFrom-SecureString | Set-Content -Path (Join-Path $env:TEMP 'Token.txt')
        Close-PodeServer
    }
}
$token = Get-Content -Path (Join-Path $env:TEMP 'Token.txt') | ConvertTo-SecureString | ConvertFrom-SecureString -AsPlainText
Remove-Item (Join-Path $env:TEMP 'Token.txt')

@Badgerati can we return token string from $WebEvent.Data.code to the parent PS session without going to disc?

@koenhalfwerk dose everything work for you? If so do you want to clean, pack it up and PR? Also see if you have idea on how to solve problem 4. thank you!

quick regard to step 4, the refreshtoken inside new-partneraccesstoken's   $ReturnCred.refresh_token does not work for you? because it does for me.
I really need  to start the weekend, but ill come back to this on Monday, as well as extending new-partneraccesstoken with -Resource https://management.azure.com/

Hey @ili101,
Yes, it's possible via Out-PodeVariable :)
Start-PodeServer {
    Add-PodeEndpoint -Address localhost -Port $Port -Protocol Http

    Add-PodeRoute -Method POST -Path '/' -ScriptBlock {
        Write-PodeJsonResponse -Value $WebEvent.Data
        Out-PodeVariable -Name Token -Value $WebEvent.Data.code
        Close-PodeServer
    }
}

# $Token is available here for use

AuthenticationFlow -DeviceCode done/merged.
AuthenticationFlow -OIDC with Pode to be done.

Dome using Pode in a Job https://github.com/ili101/PartnerCustomerCommunity/commit/9c47a1994a1ef84969cc9587d14363c788284a87#diff-78dfc795c2105c7b5e52de25a71230e2dcd0c0c232d84d2969c1c0b29ef58795R194-R266
[0.0.2-alpha] - 2022-09-08
Added

Connect-PartnerCenter -RefreshTokenScript
New-PartnerWebApp -AuthenticationFlowAllowed
New-PartnerRefreshToken -AuthenticationFlow OIDC With Pode


Feature request: Search messages by content

Would be good to be able to search messages by content to determine if an issue is related to a failed jms message

Started

@eric-vlaanderen do you want to have a look at this pull request?

Remaining work is to plumb the search capability into the front end

Resolved by #94


Added TerritoriesService and TerritoriesVariable to make territories data available in front-end templates even when not using a field.

Added TerritoriesService and TerritoriesVariable to make territories data available in front-end templates even when not using a field.
This introduces duplication of the getTerritories() method, but I think the Fieldtype could be made to use the service to retrieve that data. Eventually I would hope to also replicate the getAvailableTerritories() functionality into the service so that ad-hoc inputs could be created directly in templates (independent of any custom field) by providing some ad-hoc settings parameters.

👍  @johndwells that would be cool if you could merge this one ;)


[#97] Update DeveloperGuide with FindCommand details

Fixes #97.
Update DeveloperGuide to contain implementation details of FindCommand.

CodeCov is not activating.


Removed unnecessary cast that crashes for 3.1 schema

Fixes #4657
Extra: fixed yaml checks - yaml by default always uses \n separator

@micryc @frantuma  could you please take a look at this PR or tell me who to ping?


[NFC] Add PrettyStackTraceConformance and use it

Also allow the printFooDescription functions to suppress the trailing newline.

@swift-ci Please smoke test.

@swift-ci Please smoke test.


doc: fixed port is not recommended

@Argonus what we talked about. Not deprecated but strongly advised not to use it ;)


drivers: dmic: Intel PDM digital mic driver

This PR adds an implementation of device driver for Intel PDM digital microphone controller.

Codecov Report

Merging #9810 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9810   +/-   ##
=======================================
  Coverage   52.23%   52.23%
=======================================
  Files         212      212
  Lines       25954    25954
  Branches     5594     5594
=======================================
  Hits        13558    13558
  Misses      10160    10160
  Partials     2236     2236

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 984ce02...f48b07c. Read the comment docs.


Refer ServiceAccountTokenVolumeProjection documentation instead of providing incorrect instructions.

Since Kubernetes v1.20 this feature is enabled by default.
fixes https://github.com/kubernetes/website/issues/30317

/assign @savitharaghunathan

/lgtm


doc/reference/storage/settings: fix misleading examples

Code snippets show misleading example of read_callback usage
in h_set handlers.
This patch aligns snippets to implemented behavior.
fixes #15451
Signed-off-by: Andrzej Puzdrowski<EMAIL_ADDRESS>

cc @MarekPieta @kapi-no

@dbkinder could you review this one, so we can get it in for 2.0.0 release?


Connect to NOW

I create a simple Micro-Bot following the ReadMe,
I tried to deploy it using Now but I don't get it :
My Bot is deployed, I used  now -e BOT_TOKEN='YOUR BOT TOKEN'
When I go to https://api.telegram.org/bot/getWebhookinfo I see this :

My bot in just not responding unless I type BOT_TOKEN='TOKEN' npm start how can I make it running without having to do this ?
Thanks for your help

Perhaps you can try adding scripts to your package.json
"scripts": {
    "now-start": "micro-bot -t {BOT_TOKEN}",
    "start": "now -e BOT_TOKEN=@secret-api-key --public"
  }

Check out this on how to setup your secret now key.


Re-introduce the stored_event_model config

Makes it easier to upgrade
Makes it easier if you just want to use a different model
Will do nothing if you use a different repository

I've also moved the StoredEvent class out of the models folder to avoid confusion, as this is just a DTO and not an actual Laravel Model

Thanks!


Issue with Adding Manually Through Integrations Page

When attempting to add a device manually via the integrations page it tells me there was a 500 error and this stack trace appears in the logs.
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/aiohttp/web_protocol.py", line 435, in _handle_request
    resp = await request_handler(request)
  File "/usr/local/lib/python3.10/site-packages/aiohttp/web_app.py", line 504, in _handle
    resp = await handler(request)
  File "/usr/local/lib/python3.10/site-packages/aiohttp/web_middlewares.py", line 117, in impl
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 60, in security_filter_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 222, in forwarded_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 28, in request_context_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 81, in ban_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 236, in auth_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 136, in handle
    result = await result
  File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 148, in post
    return await super().post(request)
  File "/usr/src/homeassistant/homeassistant/components/http/data_validator.py", line 73, in wrapper
    result = await method(view, request, data, *args, **kwargs)
  File "/usr/src/homeassistant/homeassistant/helpers/data_entry_flow.py", line 71, in post
    result = await self._flow_mgr.async_init(
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 226, in async_init
    flow, result = await task
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 253, in _async_init
    result = await self._async_handle_step(flow, flow.init_step, data, init_done)
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 368, in _async_handle_step
    result: FlowResult = await getattr(flow, method)(user_input)
  File "/config/custom_components/rd200_ble/config_flow.py", line 146, in async_step_user
    if discovery_info.advertisement.local_name is not None or not discovery_info.advertisement.local_name.startswith("FR:RU"):
AttributeError: 'NoneType' object has no attribute 'startswith'

Darn, had some bad logic. Try the latest version of config_flow.py

Looks like it's no longer throwing the 500 but this time it's showing
Please wait, starting configuration wizard for RD200 BLE

and then appears to time out while attempting to configure the device
Config flow could not be loaded:
504 Gateway Time-out
 The server didn't respond in time.

When I bypass the web proxy in front of my HA instance I see this instead of gateway timeout.
Failed to connect

and the logs show
Error connecting to and getting data from 24:D7:EB:5E:B6:8A: 24:D7:EB:5E:B6:8A - 24:D7:EB:5E:B6:8A: Failed to connect:

The official application is able to connect though I don't have it connected while I'm attempting to connect with this integration to make sure they don't collide.

I found the connection to be touchy. Move it a little closer to the HA or ESPHome gateway. Also, make sure nothing else is connected. If you have a bluetooth plugged into a USB port, but it on a port extender.
I had my ESPHome connected as a BLEClient and it would not connect. Took a while to figure that out.
If you turn on debugging, you'll get some more logs.
logger:
  default: error
  logs:
    custom_components.rd200_ble: debug

Because the ESPHome component didn't support v2 of the device I only had it connecting for a few hours when I first received the RD200 and the only ESP32 running the Bluetooth Proxy is ~20 feet away through a floor. I've only got the official app on a single phone and I force closed the application to make sure it wasn't connected. I added the custom_components logger level at debug and retried running through the setup with the RD200 sitting less than a foot from the Bluetooth adapter. Here are the relevant logs
2022-12-26 10:13:21.367 DEBUG (MainThread) [custom_components.rd200_ble.config_flow] Found My Device
2022-12-26 10:13:21.367 DEBUG (MainThread) [custom_components.rd200_ble.config_flow] RD2000 Discovery address: 24:D7:EB:5E:B6:8A
2022-12-26 10:13:21.368 DEBUG (MainThread) [custom_components.rd200_ble.config_flow] RD2000 Man Data: {}
2022-12-26 10:13:21.368 DEBUG (MainThread) [custom_components.rd200_ble.config_flow] RD2000 advertisement: AdvertisementData(local_name='FR:RU22201030184', service_uuids=['0000abf0-0000-1000-8000-00805f9b34fb'], rssi=-46)
2022-12-26 10:13:21.370 DEBUG (MainThread) [custom_components.rd200_ble.config_flow] RD2000 device: 24:D7:EB:5E:B6:8A: FR:RU22201030184
2022-12-26 10:13:21.370 DEBUG (MainThread) [custom_components.rd200_ble.config_flow] RD2000 service data: {}
2022-12-26 10:13:21.370 DEBUG (MainThread) [custom_components.rd200_ble.config_flow] RD2000 service uuids: ['0000abf0-0000-1000-8000-00805f9b34fb']
2022-12-26 10:13:21.370 DEBUG (MainThread) [custom_components.rd200_ble.config_flow] RD2000 rssi: -46
2022-12-26 10:13:21.370 DEBUG (MainThread) [custom_components.rd200_ble.config_flow] RD2000 advertisement: FR:RU22201030184
2022-12-26 10:14:42.397 ERROR (MainThread) [custom_components.rd200_ble.config_flow] Error connecting to and getting data from 24:D7:EB:5E:B6:8A: 24:D7:EB:5E:B6:8A - 24:D7:EB:5E:B6:8A: Failed to connect:

The logging output shows a different service UUID than what's in your parser but looking in BLE-Monitor on my Android I definitely see both the read and write characteristic UUIDs under a service with the UUID 00001523-0000-1000-8000-00805f9b34fb so it looks like my RD200 is advertising the correct UUIDs, it's just that HomeAssistant that can't find them for some reason...

This is weird. It found your device, but will not connect. I think the UUIDs are fine, doubt you have a special model.
Are you sure you do not the MAC address still in the esphome ble_client: line? I did, and it kept me from connecting. Do you have your proxy set as active(not sure if matters)?
bluetooth_proxy:
  active: true

My whole ble_client section was commented out after I realized that it wasn't going to work with my version. I've flashed that device several times since then thanks to the bluetooth_proxy updates. I am also able to connect with my phone if I use the app so I really don't think anything else is connected. Here are the relevant sections from my ESPHome though just in case.
# ble_client:
#   - mac_address: 24:D7:EB:5E:B6:8A
#     id: radon_eye_ble_id

bluetooth_proxy:
  active: true
esp32_ble_tracker:
  scan_parameters:
    interval: 320ms
    window: 320ms

Maybe remove the interval and window? I think that makes it scan all the time, so little time to write.
Mine looks like:
esp32_ble_tracker:
  scan_parameters:
    active: true

bluetooth_proxy:
  active: true

I reflashed with these settings
bluetooth_proxy:
  active: true
esp32_ble_tracker:
  scan_parameters:
    active: true
    # interval: 320ms
    # window: 320ms

and got the same results in HomeAssistant but the ESPHome logs were still open in another browser and I saw
[11:14:12][I][bluetooth_proxy:254]: [0] [0B:17:DD:89:A0:64] Connecting v3 without cache
[11:14:12][D][esp32_ble_tracker:216]: Pausing scan to make connection...
[11:14:12][I][esp32_ble_client:064]: [0] [0B:17:DD:89:A0:64] 0x01 Attempting BLE connection
[11:14:14][D][sensor:127]: 'BLE Receiver Free Heap': Sending state 80492.00000 B with 1 decimals of accuracy
[11:14:22][I][esp32_ble_client:078]: [0] [0B:17:DD:89:A0:64] Disconnecting.
[11:14:26][D][sensor:127]: 'BLE Receiver Min Free Heap': Sending state 31164.00000 B with 1 decimals of accuracy
[11:14:34][D][sensor:127]: 'BLE Receiver Free Heap': Sending state 80492.00000 B with 1 decimals of accuracy
[11:14:42][W][esp32_ble_client:131]: [0] [] Connection failed, status=133

I unplugged my only ESPHome with a bluetooth_proxy enabled so that only the USB dongle that was ~6" away was available and tried again but with the same dialogs and logs.

Not sure what those logs are (I am not a BT expert, but that is not your RD200 MAC).
Maybe power cycle the RD200? Not sure what else can be done. I just removed my integration and restarted. It found it when I manually added it. I do switch between app, python script and integration.
Since you fail the "connect" step, it does not even get to where it reads the characteristic.
You are the only one I have heard from, so maybe there is something cached in my setup that allows it to connect.
Perhaps try manually adding it a few times in a row and maybe it will connect? Sorry it is giving you trouble.

Not sure what those logs are (I am not a BT expert, but that is not your RD200 MAC).

It's from the ESPHome logging output (the stuff that comes up right after you flash a device again through the web GUI.
I ended up rebooting the server my HomeAssistant is running on and after it came back up, HomeAssistant automatically detected the RD200 and offered to let me configure it. I hit that and everything worked perfectly... Not sure what was going on but it seems like an issue that was fixed by: "When in doubt, reboot it out!" Thanks for all the help and troubleshooting.


Crash with SNPE "mdn" after multi-rounds:  NaN/Inf present in posterior eval.

Hi,
I experience this Nan/Inf crash (sorry but it was after a long rounds run)
Round[5]: density_estimator training
Using SNPE-C with atomic loss
 Training neural network. Epochs trained: 17Traceback (most recent call last):
  File "DESY1_sbi_SNPE_multi.py", line 375, in <module>
    do_multi_pass(num_simu = 100_000, num_rounds = 10, tag="mdn_default",
  File "DESY1_sbi_SNPE_multi.py", line 321, in do_multi_pass
    density_estimator = inference.append_simulations(
  File "/anaconda3/envs/sbi/lib/python3.8/site-packages/sbi/inference/snpe/snpe_c.py", line 179, in train
    return super().train(**kwargs)
  File "/anaconda3/envs/sbi/lib/python3.8/site-packages/sbi/inference/snpe/snpe_base.py", line 327, in train
    train_losses = self._loss(
  File "/anaconda3/envs/sbi/lib/python3.8/site-packages/sbi/inference/snpe/snpe_base.py", line 548, in _loss
    log_prob = self._log_prob_proposal_posterior(theta, x, masks, proposal)
  File "/anaconda3/envs/sbi/lib/python3.8/site-packages/sbi/inference/snpe/snpe_c.py", line 280, in _log_prob_proposal_posterior
    return self._log_prob_proposal_posterior_atomic(theta, x, masks)
  File "/anaconda3/envs/sbi/lib/python3.8/site-packages/sbi/inference/snpe/snpe_c.py", line 331, in _log_prob_proposal_posterior_atomic
    utils.assert_all_finite(log_prob_posterior, "posterior eval")
  File "/anaconda3/envs/sbi/lib/python3.8/site-packages/sbi/utils/torchutils.py", line 372, in assert_all_finite
    assert torch.isfinite(quantity).all(), msg
AssertionError: NaN/Inf present in posterior eval.

It's a pity as the multi rounds contours were slowly converging to the likelihood NUTS sampling which serves as the "true" contours.
Any idea to fix the pb?

Hi, thanks for reporting. This seems related to this. What is your version of pyknos?

pyknos                  0.15.1

closing this due to inactivity. Please feel free to re-open if this is still relevant.


Check HTTP Status Codes Before Decoding Response Body

Make sure Discord sends you the A OK before you try to read a book with words you don't know :smile:
Decode("Unexpected absent key", "id")

I think I tagged this right

Closed by 7e254c5c6098bb1a47bac26c9895098a46cdc53f.


[WSL-336] Implementing Stdin redirection

Implements the following methods with the same API as exec/exec.go (and often the same implementation):
var Cmd.Stdin io.Reader
func (c *Cmd) StdinPipe() (io.WriteCloser, error)
type closeOnce struct
func (c *closeOnce) Close() error

Status: Stdout and stderr work, but stdin doesn't yet.

I run all my tests before commiting, don't worry! 😄


Berstein-Vazirani: change "a" for "s" in 1.3 explanation

Is your suggestion related to a problem? Please describe.
In section 1.3 of Berstein-Vazirani there is an inconsistency when explaining the input to the oracle $f_s$, as in the second to last equation it shows as if we want to obtain $a$, even though this was referred to as the initial state, and the string we want to find out is rather $s$.
This in my opinion makes it a big hard to follow. In the examples that follow, again the string is referred as $s$ and not $a$.
Describe the solution you'd like
In the last two equations of section 1.3, the $a$'s should be changed for $s$'s.
Describe alternatives you've considered
NA

I'd would gladly work on this issue.

Thanks for making this issue, sorry I missed this. The textbook is being superseded so this will not be fixed.


Use Docz on Existing Create-React-App on Netlify

Question
How can I use Docz along an already existing project build with create-react-app and hosted on Netlify?
Description
I have a project hosted on Netlify. I'd like to use Docz as documentation for the project. Ideally the URL's would look like this:
Main Site -> https://www.projectname.com/
Docs Page -> https://www.projectname.com/docs/
I'm aware of the base within a doczrc.js file at the root of an application, as seen on the documentation. I've also read through some other issues that mention a similar thing. However, I'm still unable to get this to work.
I also know that Netlify has a way to use a _redirects file to redirect what is served. https://www.netlify.com/docs/redirects/ .
Current Configurations
doczrc.js
export default {
  ordering: 'ascending',
  dest: '/src/docs/',
  base: '/docs/',
  htmlContext: {
    favicon: 'https://raw.githubusercontent.com/alexlee-dev/portfolio/master/public/favicon/favicon.ico'
  }
}

@alexlee-dev The easiest way is to just publish the Docz site as a new Netlify site, and customise the build on that Netlify site with the Docz build command and output directory. So two Netlify sites feeding off the same repo - both are deployed whenever there's a change.
In terms of custom urls you could have:
https://www.projectname.com
https://docs.projectname.com

Which is fine I think?

@alexlee-dev i created a plugin to resolve the problem https://github.com/nicholasess/docz-plugin-netlify

@jedrichards Yeah, that's what I ended up doing, and it works alright.
@nicholasess I'll check it out. Thanks!

Related to this issue...
As a default files in public are currently copied to.docz/dist/public directory on docz build. Is there not a way of specifying that they're sent to .docz/dist instead?
Use case being, we might have these Netlify _redirects/_headers files or other compiled assets  that need to end up in the built dest root. Seems like a fairly common requirement?
docz-plugin-netlify only really works under the assumption that all a Netlify project would need is SPA redirects to index.html.


Add query suggestion operation

I've added query suggestion to this package. The way this new type of query is posted is similar to the search query. I've added an operation enum that is used to distinguish between the different operations (search / query suggestion). I've also updated the dependencies and created a Flutter project around the example file to make sure that the example can be used directly. All dependencies that were used in the example file and not in the package itself have been moved from the pubspec.yaml package to the example pubspec.yaml.
All changes are backwards compatible.

Sorry for the delay, I was away from dev the last months... Now back to work :)
Great work @Senne021, thank you for being part of this project!


nixos/stalwart-mail: migrate logging config

Description of changes
The logging configuration keys have changed,
which caused this config to be ignored.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 315710 run on x86_64-linux 1

1 test built:

nixosTests.stalwart-mail

Thank you!


Template Not Found: page__login

I am trying to move from requirejs to webpack. I have loaded all the dust templates and js required. I could also able to see the compiled dust templates in app.js but dust.cache doesn't have any template to load.
Please find my webpack config. Please do let me know how to load the particular templates on demand.
const path = require('path');
const webpack = require('webpack');

const source_path = path.join(__dirname, 'src/');
const vendor_path = source_path + 'vendor/';
const vendor_lib_path = vendor_path + 'lib/';
const core_path = source_path + 'core/';
const CopyPlugin = require('copy-webpack-plugin');

module.exports = {
    mode: 'development',
    output: {
        filename: 'app.js',
        path: path.resolve(__dirname, 'dist'),
    },
    resolve: {
        modules: [
            path.join(__dirname, 'node_modules'),
            source_path
        ],
        extensions: ['.js', '.dust'],
        alias: {
            'dust.core': 'dustjs-linkedin',
            'dust.compiler': '',
            'c3': vendor_lib_path + 'c3/c3',
            'd3': vendor_lib_path + 'd3/d3',
            'core': core_path,
            'module': source_path + 'module/',
            'configuration': source_path + 'configuration/'
    },
    plugins: [
        new webpack.ProvidePlugin({
           $: "jquery",
           jQuery: "jquery",
           _: "underscore"
       }),
       new CopyPlugin([
        { from: 'htmls' },
        { from: 'favicon.ico' },
      ]),
    ],
    module: {
        rules: [
            {
                test: /\.dust$/,
                use: [
                    {
                        loader: "dust-loader-complete"
                    }
                ]
            }
        ]
    },
};

Can you please provide a snippet of code showing how you are trying to render the template? Also, in the compiled app.js you should find some code like function(dust){dust.register("template_name",body_0) - what does yours have for the template name?

Hi @jrrbru,
I couldn't able to see any code like dust.register. I can see only dust.render. I have just added a small snippet of my code to demo this issue below with screenshot. It would be awesome, if you give me some pointers. Thanks!
Main.js
dust.render("views/hello", {"version": dust.version}, function(err, out) {
  if(err) console.error(err);
  document.getElementsByTagName('body')[0].innerHTML = out;
});

views/hello.dust

Hello World!

{>"views/version"/}

Modified webpack.config.js

module.exports = {
    mode: 'development',
    entry: './lib/main.js',
    output: {
        filename: 'main.js',
        path: path.resolve(__dirname, 'dist'),
    },
    resolve: {
        modules: [
            path.join(__dirname, 'node_modules'),
            source_path + '/views1'
        ],
        extensions: ['.js', '.dust'],
        alias: {
            dustjs: 'dustjs-linkedin',
            'dust.core': 'dustjs-linkedin',
            'c3': vendor_lib_path + 'c3/c3',
            'd3': vendor_lib_path + 'd3/d3',
            'core': core_path
        }
    },
    module: {
        rules: [
            {
                test: /\.dust$/,
                use: [
                    {
                        loader: "dust-loader-complete"
                    }
                ]
            }
        ]
    },
};

I've fixed this issue, looks like I have forgot to add define.config.amd = true in the setup file. My bad. I am closing this now. Thanks for your time @jrrbru


Updates in Helper\Navigation\Breadcrumbs.php and Menu.php

Added possibility to set parameters(variables) for partial to populate in the view.
In the current version there is no possibility to transmit variables into partial view , you can only set partial and module like this:
$this->navigation('navigation')
    ->menu()
    ->setPartial('partial/yourPartial.phtml','yourModule');

so with this update you can simply set those variables like this:
$this->navigation('navigation')
    ->menu()
    ->setPartial('partial/yourPartial.phtml','yourModule')
    ->setPartialParams(array('variableName' =>$variable));

That's all.

:-1:
This is not needed, because:

you can use menu options in partial view script (same for Breadcrumbs)
or the view helper "Placeholder"

@froschdesign From reading the description, I think the reporter wants to be able to pass arbitrary parameters to the partial, not just menu options.
@Zyqsempai Please add unit tests, and also run ./vendor/bin/php-cs-fixer fix within your checkout before your next push.

@weierophinney
The description was not clear enough. Therefore, the second option: view helper "Placeholder".

@Zyqsempai

->setPartial('partial/yourPartial.phtml','yourModule')

[There is no second parameter!](https://github.com/zendframework/zend-view/blob/master/src/Helper/Navigation/Menu.php#L674)

Added possibility to set parameters(variables) for partial to populate in the view.

Can you provide a use case?

Hi , here is use case from my current project,
we have few view's and we need to show on them one navigation-partial but with some differences , for that we need to transmit simple flag(show or not) into this partial with this update you can simple do like this:
$this->navigation('navigation')
->menu()
->setPartial('partial/yourPartial.phtml','yourModule')
->setPartialParams(array('flag' => false));

Added unit test for setter and getter for PartialParams , also fixed all issues with php-cs-fixer.

for that we need to transmit simple flag(show or not)

Show or not? You can use the custom page properties:
$page1 = new Zend\Navigation\Page\Mvc();
$page1->header = true;

$page2 = new Zend\Navigation\Page\Mvc();
$page2->footer = true;

I'm not entirely against this PR, but I think there are too many options in the navigation view helpers.

->setPartial('partial/yourPartial.phtml','yourModule')

Again: [there is no second parameter!](https://github.com/zendframework/zend-view/blob/master/src/Helper/Navigation/Menu.php#L674)

I like the general idea (passing variables to the partial). However, I'm not convinced by the implementation. Typically, I suspect I'd want to pass parameters to the partial on a case-by-case basis, at the time of invocation — in other words, I wouldn't want stateful parameters. As such, I'm wondering if this should warrant an alternate rendering method instead of additional helper properties — something like this:
echo $this->navigation()->menu()->renderPartialWithParams(['flag' => $flag]);

The signature would be:
public function renderPartialWithParams(array $params = [], $container = null, $partial = null)

I've provided a "sketch" if the idea here: https://github.com/weierophinney/zend-view/commit/80912976f00a90ea1d7f84984edd74e424e07d13
@froschdesign — what do you think of this approach?

@weierophinney
Looks good to me. No extra options in the helper!

@Zyqsempai Do you want to take it from here? or would you like me to do a competing PR?

@weierophinney, What do you mean, "to take it from here"?
Regarding your idea i will be glad to help(if i can) to implement this functionality.

Personally I think the method name indicates how it should be implemented: renderPartialWithParams. Render a partial with parameters, thus adding the parameters to the method renderPartial.
The suggested implementation makes me think about methods like renderMenuWithContainer or renderMenuWithContainerAndOptions. This is (fortunately) not implemented that way and instead there is a single method renderMenu with optional parameters $container and $options. It makes more sense to me to use the same approach with renderPartial.

@Martin-P The problem is that renderPartial() has the following signature already:
public function renderPartial($container = null, $partial = null)

These are both null by default, and the method does some work to auto-determine them if they are not present. Considering the standard use case is to pass no arguments, adding the parameters as an option to the end makes the method unwieldy:
echo $this->navigation->menu()->renderPartial(null, null, ['flag' => $flag]);

This is why I suggested adding a new method, putting the argument at the start.
@Zyqsempai — by "take it from here," I was inviting you to re-factor your patch to follow the direction I was providing. :) I haven't written tests, as I just wanted to demonstrate how I'd approach it, so you'd need to:

Merge my commit, or copy/paste it into your branch.
Make similar changes for the Breadcrumbs helper.
Write tests for each that validate that the parameters passed are being passed to the partial via the view model.

You can then push to this same pull request. I'd recommend starting from a fresh branch, and then force-pushing to the original branch on your github fork.

@weierophinney , Thank you for your invitation, i will start it tomorrow.

@weierophinney I understand your point, but isn't it more intuitive to keep the method more like the regular partial view helper? At this moment when I want to use renderPartial with the default container I already have to use null as the first parameter.
Looking at renderPartialWithParams I think most people will use it like this:
echo $this->navigation->menu()->renderPartialWithParams(
    ['flag' => $flag],
    null,
    'myPartial.phtml'
);

So perhaps it is better to ommit the $container completely here. We always have setContainer for this and don't really need it as a parameter(which only causes trouble). That way the method can be simplified:
public function renderPartialWithParams($partial = null, array $params = [])

And looking at this... I can't ignore the possibility to introduce a BC break instead of creating new methods and change the signature for renderPartial into the same signature as the partial view helper:
public function renderPartial($partial = null, array $params = [])

I am aware it is a BC break, but I think this is a better approach instead of creating fixes because of something that can possibly be called a design flaw?

@Zyqsempai I'm trying to keep the order of the arguments internally consistent in the helpers. Since the withPartial() method already has container and partial in that order, it makes sense to keep them in that order throughout. We can revisit the parameter order at a later date.

@weierophinney , So, what the plan? I will implement you idea, as discussed earlier?
 How i can get the ZF2 development version? I mean with all test and modules , doing it through composer i got latest ZF2 version but without tests, in other way by cloning modules one by one i don't have enough dependencies to start those tests. Also php-cs-fixer, i have installed manually. Those facts made me think that maybe i got "production" version or something like that. Thank you in advance.

Please close this PR. It was merged with #8.

ping @weierophinney


NIFI-9238: Mapped JMSConnectionFactoryProvider properties to Qpid JMS…

… client setters
https://issues.apache.org/jira/browse/NIFI-9238
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Thanks @turcsanyip - is it worth adding a paragraph for this Qpid integration in the additionalDetails?
https://nifi.apache.org/docs/nifi-docs/components/org.apache.nifi/nifi-jms-processors-nar/1.14.0/org.apache.nifi.jms.cf.JMSConnectionFactoryProvider/additionalDetails.html

@pvillard31 It is worth definitely. I extended the documentation with Qpid JMS.

Merged, thanks @turcsanyip


undocumented environment vulnerability: 'python'

L48:
if os.path.basename(python_cmd).startswith('mbed'):
    python_cmd = 'python'

The validity of doing this should be documented, because as it stands, it appears wrong: 'python' will invoke whatever executable has that name on the current system path. It may not be the same interpreter that is executing the current file. It may not work with other aliases of python, or in environments where python is not on the system path. This could cause a number of issues e.g. when using a python interpreter that is directly invoked from an absolute path, or when using virtual environments.

This is now merged and released in Mbed CLI 1.4.0


DF Module Name: Summarize issue...

Module
DF Quality of Life v1.8.0
Describe the issue
Disabling "Enable roll table menu option to quick roll" option removes all other options in the config menu for DF QOL. Also seems to disable other features of the DF QOL module like ability to color folder text and actor locking. Have not looked/ tested for other features' functionality.
To Reproduce
Steps to reproduce the behavior:
FVTT : v10
Build: 288
System: DnD5e 2.0.3

Go to configure settings and move to DF Quality of Life section.
Unselect the "Enable roll table menu option to quick roll" option.
save changes and reload page.
Go to configure settings and move to DF Quality of Life section and only setting available is the
"Enable roll table menu option to quick roll" option.
check other features of module to see unexpected behaviors.

Expected behavior
Disable quick rolls for roll table menus only.

The issue is an error prevents the rest of the "init" hook from running.
DF QoL has two issues the one mentioned above and the game setting for "vehicle capacity".


[SPARK-48487][INFRA] Update License & Notice according to the dependency changes

What changes were proposed in this pull request?

This PR updates License & Notice files according to the dependency changes
Why are the changes needed?

to meet apache release policy
Does this PR introduce any user-facing change?

no
How was this patch tested?

manually check
Was this patch authored or co-authored using generative AI tooling?
no

cc @cloud-fan as the release manager of 4.0.0-preview1

should we put it in the release process guideline so that it's a step of the release?

Yes. We also need to do this in routine dependency update PRs，which can sufficiently reduce the RM's workload

Merged to master


Ensure node_modules are in place before running vitest

Short description
This pull request addresses the issue where the Vitest tools fail if the node_modules directory is missing.
Proposed changes

If node_modules is not found, npm install is executed silently (with output redirected to /dev/null).

Pull Request Review Guidelines

I'll squash the branch before merging it, fyi :)


chore(package): update ember-cli-chai to version 0.4.1

Closes #34

CI is 🍎 for some probably unrelated reason


Update nginx.prod.conf

Issue #: https://github.com/bcgov/met-public/issues/
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #2312 (5b1177c) into main (74ce32b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2312   +/-   ##
=======================================
  Coverage   68.55%   68.55%
=======================================
  Files         472      472
  Lines       15590    15590
  Branches     1193     1193
=======================================
  Hits        10688    10688
  Misses       4693     4693
  Partials      209      209

Flag
Coverage Δ

metweb
61.48% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Group members can delete each others' annotations

It seems that individual members of a group that is part of the same project can delete each other's annotations. If a user clicks on the 'x' symbol on an annotation document (for example, by error due to a lagging internet connection), that could potentially delete all 50 annotations in that document created by another user!
I think the ideal solution would be to be able to turn on and off the "God mode". There's probably an easy way to do this: only allowing the "owners" of a group to be able to delete other people's annotations, the rest being able to view/edit their annotations only.

For the most part, this is resolved by #415.  It

Adds a confirmation dialog when deleting annotations
Removes the "delete annotation" button from regular (non-admin) users

It does not implement toggling god mode.  If that is still desired, feel free to add it to a new issue.


Adding CosmosDB Dapr User Agents

Description
User agents were not implemented when migrating to the Track 2 SDKs for CosmosDB. This sets the Dapr version user agent.

/ok-to-test


Prevent dynamic compilation of a module if precompiled by VS.

To see if we need to compile we check compilation IO and the previous context stored in obj/**/dotnet-compile-csc.rsp which contains compilation options and all compilation IO paths previously used.
Before, when building from VS, binaries was outputed to the artifacts folder while dynamic compilation references them from regular projects bin folders. So, dynamic compilations was done to re-generate these binaries, and compilation IO paths (and then the compilation context) was not the same.
Now it's ok, but for ambient projects, which are not dynamically compiled, the choice was to always reference their binaries from the runtime directory. So, to generate the same compilation context as VS, and because it's now possible, for ambient projects we also 1st try to use regular project bin folders.
This way, in a dev context, the dotnet-compile-csc.rsp file generated by VS or dynamic compilation is now the same. So now, if a module is pre-built with VS, it is not dynamically re-compiled at runtime.
Best

Saweet - should have a perf benefit too right?

Right, but only during dev when you pre-build a module with VS before launching the app. Also if you pre-build a core project which is not part of Orchard.Web but referenced by some modules.


Production and development performance inconsistency

I made a todo list app like this:

This is TodoList render:

When in development I clicked a checkbox to finish a todo item, the TodoItem component re-rendered and the TodoList component re-rendered too, but in production only the TodoItem re-render the TodoList didn't re-render. How did I found this, I set a componentDidUpdate hook in TodoList and I found in production the hook didn't be triggered .
So I am confused why is that?

Are you using prop types?
Op wo 24 okt. 2018 04:43 schreef kwzm<EMAIL_ADDRESS>
I made a todo list app like this:
[image: preview]
https://user-images.githubusercontent.com/16114101/47402197-f4531b00-d776-11e8-93b4-ed68fb4b495f.PNG
This is TodoList render:
[image: todolist-render]
https://user-images.githubusercontent.com/16114101/47402418-126d4b00-d778-11e8-8e02-f56d20d7910f.PNG
When in development I clicked a checkbox to finish a todo item, the
TodoItem component re-rendered and the TodoList component re-rendered too,
but in production only the TodoItem re-render the TodoList didn't
re-render. How did I found this, I set a componentDidUpdate hook in
TodoList and I found in production the hook didn't be triggered .
So I am confused why is that?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/issues/585, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhLMle2bR7L7uu-8dPIBQ8bgI9sVBks5un9O0gaJpZM4X3BKj
.

@mweststrate Yes,I did.

See https://github.com/mobxjs/mobx-react/issues/56. If it is unclear,
please share more of your component definitions or create a sandbox
Op wo 24 okt. 2018 om 09:59 schreef kwzm<EMAIL_ADDRESS>
@mweststrate https://github.com/mweststrate Yes,I did.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/issues/585#issuecomment-432552924,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhBbDuWMTCA2q-zmBAzd9Su4Cdiavks5uoB3NgaJpZM4X3BKj
.

Mobx subscribes for observables accessed in render.
However some observables can be accessed by react during prop type validation.
Mobx can't differentiate whether the observable was accessed by you directly or by react during validation.
const Parent = observer(({ array }) => {
  // the array content is not accessed, therefore the component doesn't react to array modifications
  return <Child array={array}>
})

// With prop type validation
Parent.propTypes = {
  array: PropTypes.arrayOf(PropTypes.string),
}

const Parent = observer(({ array }) => {
  // the array is accessed during validation performed by react, therefore the component will react to array modifications
  array.forEach(item => {
    if (typeof item !== 'string') {
      console.warn('Expecting array of strings');
    }
  })

  return <Child array={array}>
})


Integral Available Potential Energy calculation

to use as so
fields = all_fieldtimeseries("weno_thirtytwo" ...)
APE = Diagnostics.integral_available_potential_energy(fields[:b])

@LucyBrock4


chore: update vue test utils to latest stable

What is accomplished by your PR?
Update Vue test utils to stable version (was beta).
They changed to async mode so had to update some tests (https://github.com/vuejs/vue-test-utils/releases/tag/v1.0.0)
Is there something controversial in your PR?
Don't think so.
Checklist
New Feature / Bug Fix

[x] Run unit tests to ensure all existing tests are still passing
[ ] Add new passing unit tests to cover the code introduced by your PR
[ ] Change documentation for the code introduced by your PR
[ ] Add Story / Design System Page for a new component introduced by your PR

Closing as this is merged as part of #525


Update README.md

yarn install is depreciated and needs to be updated to yarn add.

Thanks for the heads-up. 👍


Many mods not working

Minecraft version
1.20.1
Describe the bug
Apparently some mods aren´t working with the connector, i got a new computer, and i´m installing the mods again, but i don´t understand why it is not working properly
Steps to reproduce

I installed the mods DynamicSurroundings, friendsandfoes, entity texture features, entity model features
Then i installed the Synitra Connector
The Launcher told me to put some extra mods that the othe mods need so i install them: gml, ash api, puzzle lib, midnight lib and mod menu
When i loaded it the game crashed

Crash Report:

Logs
No response
Additional context
No response

if you go on the discord we can help you fix this it's a easy solution

if you go on the discord we can help you fix this it's a easy solution

Thank you i´m going


removed explicit role for efk in cluster.yml

Patch for #1000 wherein "efk" related tasks were executing inadvertently
Tested with efk_enable: false

ci check this

ci showing incorrect status, but builds were successful and efk was suppressed as it should be,
TASK [kubernetes-apps/efk/elasticsearch : ElasticSearch | Write ES deployment] *
task path: /builds/kargo-ci/kubernetes-incubator__kargo/roles/kubernetes-apps/efk/elasticsearch/tasks/main.yml
skipping: [node1] => {"changed": false, "skip_reason": "Conditional check failed", "skipped": true}


gn: add more info on unpack failure

as just the return code is everything but helpful to debug any issues when unpacking.
Use check_output for this purpose and error out with more details.
Also remove the pointless return statement

LGTM.  Just need you to sign the commit


Fix caching editable settings forever when no request

There's a bug where if code is being executed in a setup that doesn't use requests (like in shell or a Celery worker) then editable settings will be cached once and never change.
The fix is pretty simple, but we might want to drop NULL_REQUEST entirely since I believe with the change it doesn't provide any value. It was originally meant to be a value which could be inserted into the WeakKeyDictionary but that behavior is actually the cause of the bug. If there's no request we don't want to cache anything, or we'll accidentally cache it forever. @AlexHill, what are your thoughts?

Thanks David


Export disconnect function on Pusher client

Hi there!
We have started using your library in production and found some use cases when we need to disconnect from Pusher on component unmount. Especially when we reach a limit of opened connections :)
Here is an article about connections counting
https://support.pusher.com/hc/en-us/articles/202092346-How-are-connections-counted-

@SamMolokanov thank you for taking the time to make a pull request, I have just published 0.3.9 to npm.

@TheRusskiy Thank you!


Is this project abandoned? (new fork available)

There hasn't been an update in a while, and the coderplus.com domain appears to be abandoned.
Can I assume this project is abandoned as well?
If so, I'd encourage everyone to move over to my fork at https://github.com/kohlschutter/copy-rename-maven-plugin

Posted here, but would make more sense to fix in your repository, which unfortunately doesn't have issue tracking enabled: https://github.com/coderplus/copy-rename-maven-plugin/issues/23

@Googulator I've just enabled issue tracking on my fork. Feel free to create an issue there. Pull requests welcome!


Moderation: missing new categories

Describe the bug
Typing does not include the latest category additions as listed in the docs such as:
    | "self-harm/intent"
    | "self-harm/instructions"
    | "harassment"
    | "harassment/threatening"

See https://github.com/openai/openai-node/blob/3ec43ee790a2eb6a0ccdd5f25faa23251b0f9b8e/src/resources/moderations.ts#L41-L76
To Reproduce
import oai from "openai"

const someFlag: keyof oai.Moderation.Categories = "harassment" // type error

Code snippets
No response
OS
macOS
Node version
Node v18.16.0
Library version
openai 4.0.0

Thanks for the report! We'll have this updated shortly.

This has been updated in v4.0.1!


Uploaded image preview placeholder

@parasharrajat @JmillsExpensify
Details

Showing image preview until image uploaded.
Fixed Issues

$ https://github.com/Expensify/App/issues/7463
Tests | QA Steps

Open a chat window
Click (+ button / add attachment / choose from gallery)
Select an image
Click Send button in preview modal

[X] Verify that no errors appear in the JS console

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots

Web

Chrome
https://user-images.githubusercontent.com/1966034/152587003-b24c3926-cdb7-48b5-a61d-c48fd2156fb1.mov
Safari
https://user-images.githubusercontent.com/1966034/152587030-1776db53-05d2-476e-abff-bb166cce103f.mov
Mobile Web

https://user-images.githubusercontent.com/1966034/152587895-e1b94cfc-fbd9-4a8e-8596-55984cf06427.mov
Desktop

https://user-images.githubusercontent.com/1966034/152587069-d1742004-79a6-4775-8f9c-61fd0d1525e9.mov
iOS

https://user-images.githubusercontent.com/1966034/152587121-d8bd1300-ef62-4856-bbca-449f24d56443.mov
Android

https://user-images.githubusercontent.com/1966034/152587159-79a23d4d-4a07-48cb-bed8-31e86667e121.mov

I have read the CLA Document and I hereby sign the CLA

cc: @shawnborton Could you please review the design of the image preview?

Is there a way we can make this more smooth? Could we detect the image dimensions upon upload and at least make it so that the preview area doesn't jump around as much?
Also, I'm curious about ActivityIndicator we're using here. Do we have other examples of where we use it in the app and it has a solid background behind it? I would think we would just show the loading spinner on top of the content with no box around it. Let me know if that makes sense.

@shawnborton

Is there a way we can make this more smooth? Could we detect the image dimensions upon upload and at least make it so that the preview area doesn't jump around as much?

Yes It can, but it is a different issue than this.

Also, I'm curious about ActivityIndicator we're using here. Do we have other examples of where we use it in the app and it has a solid background behind it? I would think we would just show the loading spinner on top of the content with no box around it. Let me know if that makes sense.

The activity indicator is default by operating system. I added a faded background because it's hard to see with photos which have light colors. (https://github.com/Expensify/App/issues/7463#issuecomment-1027975719)
If you provide me a design, I can set it to it.

Why would you consider that a different issue than this?

Ok. yeah. Thanks for your thoughts @okansahin.

Is there a way we can make this more smooth? Could we detect the image dimensions upon upload and at least make it so that the preview area doesn't jump around as much?

I agree that this is not part of this PR and requires a bigger implementation change. But this is a good suggestion we can talk about this more on slack.

Also, I'm curious about ActivityIndicator we're using here. Do we have other examples of where we use it in the app and it has a solid background behind it? I would think we would just show the loading spinner on top of the content with no box around it. Let me know if that makes sense.

Let's try to work on this. @okansahin Do you have any suggestions?

Also, I'm curious about ActivityIndicator we're using here. Do we have other examples of where we use it in the app and it has a solid background behind it? I would think we would just show the loading spinner on top of the content with no box around it. Let me know if that makes sense.

Let's try to work on this. @okansahin Do you have any suggestions?

I thought this might be good for the ActivityIndicator to appear with every image clear.
If you think the same with me that faded background is required I might fill the faded background over image. But as @shawnborton said this usage is not exists with the other ActivityIndicators. It cause degradation of the design integrity.
We can use another spinner that you will choose, maybe with Lottie.
Or I can completely remove the faded background.
I think @shawnborton should decide this.
Thank you

Okay that's a fair comment that we can follow up about dimensions in another PR, but what I found surprising with this animation is that we go from this shape:

Then we get a new shape and a blank preview (which we should definitely avoid):

Then we land on a final shape like this:

I think we should try to avoid the shape changing as much as we possibly can. Do you have any ideas for that?
For the ActivityIndicator, could we just try it without the background fill behind it?

Okay that's a fair comment that we can follow up about dimensions in another PR, but what I found surprising with this animation is that we go from this shape:
Then we get a new shape and a blank preview (which we should definitely avoid):
Then we land on a final shape like this:
I think we should try to avoid the shape changing as much as we possibly can. Do you have any ideas for that?

This catches my attention too. Even if we apply the solutions I wrote above, this will continue to happen. It requires intervention to the general structure in chat window. If you open a topic in Slack I will be glad to research and participate in the solution.
Even you don't upload a picture, this issue also happens when you open the app and click a chat room. (Here maybe retrieving images from the cache will fix the problem.)

For the ActivityIndicator, could we just try it without the background fill behind it?

I will do it in a short time and send the screenshots in here.

Simulator iOS

Desktop Chrome

The spinner looks a bit too transparent on iOS, but looks fine in Chrome IMO.

So, what do you want me to change. Is it okay like this?

This catches my attention too. Even if we apply the solutions I wrote above, this will continue to happen. It requires intervention to the general structure in chat window. If you open a topic in Slack I will be glad to research and participate in the solution.

Even you don't upload a picture, this issue also happens when you open the app and click a chat room. (Here maybe retrieving images from the cache will fix the problem.)

Hmm but I think where I am a bit confused is that in my post above, why does the second screenshot happen? Why can't we just go from 1 to 3 and this way the size only changes once?

Because it's about the way of loading messages. This "way" not relational the code we have changed in this PR.

Maybe for iOS, we can use our brand dark color for the base spinner color?

Because it's about the way of loading messages. This "way" is not relational the code we have changed in this PR.

That's fair, I think we should create a separate follow up issue to address that then.

Maybe for iOS, we can use our brand dark color for the base spinner color?

It's looks like this with dark color which is defined in project. (#0b1b34)

That definitely looks hard to see. Could we see it using one of our light gray colors?

That definitely looks hard to see. Could we see it using one of our light gray colors?

It's #FAFAFA which is applied.

I think that looks better. What do you think?

Yes, it looks better.
I push code to repo if you agree with it.

Yup, I agree.

@parasharrajat @Julesssss
waiting for you review. thank you.

The iOS E2E tests failed with a timeout, just restarted them. Code looks pretty good to me other than that.
It would be good to make the follow-up improvements that yourself and Shawn discussed.

The iOS E2E tests failed with a timeout, just restarted them. Code looks pretty good to me other than that.

I saw that dotenv module not found in logs. Do I should merge main branch to this?

It would be good to make the follow-up improvements that yourself and Shawn discussed.

If you talking about for image jumping, I can create new bug request with that issue if you accept.

Do I should merge main branch to this?

I didn't look into the errors, but if you think it will help then please do this.

I didn't look into the errors, but if you think it will help then please do this.

I didn't merged, all checks have passed now.
thank you @Julesssss

Testing shortly...

I think that's probably fine. In a separate follow up issue, it would be nice to standardize on the same style spinner everywhere instead of having differences between platforms.

@okansahin, Great job getting your first Expensify/App pull request over the finish line! :tada:
I know there's a lot of information in our contributing guidelines, so here are some points to take note of :memo::

Now that your first PR has been merged, you can be hired for another issue. Once you've completed a few issues, you may be eligible to work on more than one job at a time.
Once your PR is deployed to our staging servers, it will undergo quality assurance (QA) testing. If we find that it doesn't work as expected or causes a regression, you'll be responsible for fixing it. Typically, we would revert this PR and give you another chance to create a similar PR without causing a regression.
Once your PR is deployed to production, we start a 7-day timer :alarm_clock:. After it has been on production for 7 days without causing any regressions, then we pay out the Upwork job. :moneybag:

So it might take a while before you're paid for your work, but we typically post multiple new jobs every day, so there's plenty of opportunity. I hope you've had a positive experience contributing to this repo! :blush:

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @Julesssss in version: 1.1.38-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @Julesssss in version: 1.1.38-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Feature Request: perf the specific process/thread

Is your feature request related to a problem? Please describe.
Currently bpftrace can't perf a specific process/thread. For example, bpftrace -e 'hardware:cycles:1e6 /pid == 23333/ {} looks like a way to do, but in fact it calls perf_event_open(..., pid=-1, ..., cpu=X) with the bpf prog to filter the process by if ((bpf_get_current_pid_tgid() >> 32) == target_pid). This ends up with measuring all processes on the specific CPU X.
According to the man 2 perf_event_open, properly measuring the specific process/thread requires an non-zero pid argument passed to perf_event_open(), something like perf_event_open(..., pid=23333, ..., cpu=X). I would appreciate it if bpftrace can support this.
Describe the solution you'd like
I was wondering if expanding the bpftrace probe is a reasonable solution, like bpftrace -e 'hardware:cycles:1e6:pid:23333 {}.
Describe alternative solutions or features you've considered
Alternatively, we could add some identifiers, so the new command may look like bpftrace -e 'hardware:cycles:c1e6,p23333 {}, where c1e6 stands for count=1e6, p23333 means pid=23333. Or more explicitly, hardware:cycles:count=1e6,pid=23333.

Seems like a reasonable thing to support. Although I think rather than adding to probe definition (where brevity and readability are very important), it's a better fit for probe attributes: https://github.com/bpftrace/bpftrace/issues/3360
Unfortunately probe attributes do not exist in code yet, so it's not quite a simple feature add. But it sounds like probe attributes have enough consensus to move forward

Is there a reason why bpftrace doesn't just pass the pid by default for perf events if the probe specifies a filter (and possibly other probes if they support this too) ?

Probably b/c -p and -c originally were for USDT use case:
    -p PID         enable USDT probes on PID
    -c 'CMD'       run CMD and enable USDT probes on resulting process

But probably makes sense to expand that scope as far as applicable, like here


Add Travis-CI and CodeCov.io integrations.

This PR adds the following:

Travis-CI tests on every pull request/master branch
E.g. https://travis-ci.org/nlutsenko/parse-server/builds/106656352
Travis-CI tests run in a matrix with Node 4.1 and 4.2
It's using the latest stable version of MongoDB 3.0 (3.0.8), but we can extend and or change that extremely easy - by simply adding another item into env array.
Added Codecov.io integration via npm module - this will run only if tests succeeded.
E.g. https://codecov.io/github/nlutsenko/parse-server?branch=nlutsenko.travis
Added mongodb-runner npm module, which starts a mongo on pretest/posttest, so we can ensure mongo is always there locally whenever tests are run.

Few benchmarks:

Average test run is ~1.5min on Travis-CI
https://travis-ci.org/nlutsenko/parse-server/jobs/106656353
Average full integration is ~3min on Travis-CI
https://travis-ci.org/nlutsenko/parse-server/builds/106656352

The only flow change here is that to run tests - we enforce running mongo via mongodb-runner, which I still feel is absolutely fine and the best thing we could do, since it's somewhat a best practice.
Please keep in mind - that this runner is never used in a non-dev environment, but is only installed and used when developing per devDependencies section of package.json.
This superseeds and closes #30.

This is currently waiting on tests to stabilize before merging, and I'll enable Travis-CI integration as soon as we are ready to merge this.


bugfix: JDBC resultSet, statement and connection closing order

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

fixes: #6224
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report
Attention: 24 lines in your changes are missing coverage. Please review.

Comparison is base (2363715) 49.97% compared to head (654c570) 49.85%.
Report is 6 commits behind head on 2.x.

:exclamation: Current head 654c570 differs from pull request most recent head f1845fa. Consider uploading reports for the commit f1845fa to get more accurate results

Additional details and impacted files

@@             Coverage Diff              @@
##                2.x    #6248      +/-   ##
============================================
- Coverage     49.97%   49.85%   -0.13%
- Complexity     4928     4933       +5
============================================
  Files           915      917       +2
  Lines         31927    32010      +83
  Branches       3855     3862       +7
============================================
+ Hits          15954    15957       +3
- Misses        14406    14494      +88
+ Partials       1567     1559       -8

Files
Coverage Δ

.../rm/datasource/exec/BaseTransactionalExecutor.java
74.00% <100.00%> (+0.26%)
:arrow_up:

.../seata/rm/datasource/exec/MultiUpdateExecutor.java
80.26% <100.00%> (+0.26%)
:arrow_up:

...va/io/seata/rm/datasource/exec/UpdateExecutor.java
84.09% <100.00%> (+0.36%)
:arrow_up:

...datasource/exec/mysql/MySQLUpdateJoinExecutor.java
78.48% <100.00%> (+0.13%)
:arrow_up:

...ec/mysql/MySQLInsertOnDuplicateUpdateExecutor.java
32.16% <0.00%> (-0.17%)
:arrow_down:

...er/console/impl/db/BranchSessionDBServiceImpl.java
0.00% <0.00%> (ø)

...erver/console/impl/db/GlobalLockDBServiceImpl.java
0.00% <0.00%> (ø)

...er/console/impl/db/GlobalSessionDBServiceImpl.java
0.00% <0.00%> (ø)

... and 23 files with indirect coverage changes


bug:

Did you check docs and existing issues?

[X] I have read all the persistence.nvim docs
[X] I have searched the existing issues of persistence.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.9.4
Operating system/version
macOS Sonoma 14.11
Describe the bug
Getting the following error when trying to persist a session:
E5108: Error executing lua ....repro/plugins/persistence.nvim/lua/persistence/init.lua:67: bad argument #1 to 'concat' (table expected, got nil)
stack traceback:
        [C]: in function 'concat'
        ....repro/plugins/persistence.nvim/lua/persistence/init.lua:67: in function 'save'
        [string ":lua"]:1: in main chunk

A similar error occurs when attempting to load a session.It appears that it is not finding the table builtin.
Steps To Reproduce

Use minimal init.lua as given below.
Hit <Leader>qs or <Leader>ql

Expected Behavior
Session is saved or loaded without error.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/persistence.nvim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

vim.keymap.set("n", "<leader>qs", "<cmd>wa<CR><cmd>lua require('persistence').save()<CR>")
vim.keymap.set("n", "<leader>ql", "<cmd>wa<CR><cmd>lua require('persistence').load()<CR>")

This is not an issue with persistence.nvim. In your config, you didn't use the default opts. Please add config = true as follows
{
  "folke/persistence.nvim",
  config = true,
}

Thank you, that was it.


How does APISIX(DP) deployed outside the K8S cluster access the routes created through apisix-ingress(CP)?

Issue description
因为通过apisix-ingress创建的route中的backends都是K8S中的ep或clusterip，传递到APISIX后，因为APISIX是部署在K8S之外的话会无法访问该后端地址
Environment

your apisix-ingress-controller version (output of apisix-ingress-controller version --long);
your Kubernetes cluster version (output of kubectl version);
if you run apisix-ingress-controller in Bare-metal environment, also show your OS version (uname -a).

If your network is an underlay network, this method will be more convenient.
Otherwise, you need to maintain network connectivity by yourself.

Yes, this belongs to the k8s network configuration. If you need to deploy DP outside, of course, you need to make sure that the outside DP can access the services (pod IP) inside the k8s cluster.
There are many solutions to open up the network, such as calico + BGP， but it is not the management scope of the Ingress controller.


Build fails on OpenBSD 7.3

Faced when trying to build static-web-server:
The following warnings were emitted during compilation:

warning: src/low_level/extract.c:26:7: error: use of undeclared identifier 'SI_MESGQ'
warning:     { SI_MESGQ, -1, 5 },
warning:       ^
warning: src/low_level/extract.c:36:37: error: invalid application of 'sizeof' to an incomplete type 'struct Const []'
warning:     const size_t const_len = sizeof consts / sizeof *consts;
warning:                                     ^~~~~~
warning: 2 errors generated.

error: failed to run custom build command for `signal-hook v0.3.15`

Caused by:
  process didn't exit successfully: `/home/andinus/dev/other/static-web-server-2.19.0/target/debug/build/signal-hook-7482100cb62ccacc/build-script-build` (exit status: 1)
  --- stdout
  TARGET = Some("x86_64-unknown-openbsd")
  OPT_LEVEL = Some("0")
  HOST = Some("x86_64-unknown-openbsd")
  cargo:rerun-if-env-changed=CC_x86_64-unknown-openbsd
  CC_x86_64-unknown-openbsd = None
  cargo:rerun-if-env-changed=CC_x86_64_unknown_openbsd
  CC_x86_64_unknown_openbsd = None
  cargo:rerun-if-env-changed=HOST_CC
  HOST_CC = None
  cargo:rerun-if-env-changed=CC
  CC = None
  cargo:rerun-if-env-changed=CFLAGS_x86_64-unknown-openbsd
  CFLAGS_x86_64-unknown-openbsd = None
  cargo:rerun-if-env-changed=CFLAGS_x86_64_unknown_openbsd
  CFLAGS_x86_64_unknown_openbsd = None
  cargo:rerun-if-env-changed=HOST_CFLAGS
  HOST_CFLAGS = None
  cargo:rerun-if-env-changed=CFLAGS
  CFLAGS = None
  cargo:rerun-if-env-changed=CRATE_CC_NO_DEFAULTS
  CRATE_CC_NO_DEFAULTS = None
  DEBUG = Some("true")
  CARGO_CFG_TARGET_FEATURE = Some("fxsr,sse,sse2")
  running: "cc" "-O0" "-ffunction-sections" "-fdata-sections" "-fPIC" "-gdwarf-2" "-fno-omit-frame-pointer" "-m64" "-Wall" "-Wextra" "-o" "/home/andinus/dev/other/static-web-server-2.19.0/target/debug/build/signal-hook-932164e25ff6283b/out/src/low_level/extract.o" "-c" "src/low_level/extract.c"
  cargo:warning=src/low_level/extract.c:26:7: error: use of undeclared identifier 'SI_MESGQ'
  cargo:warning=    { SI_MESGQ, -1, 5 },
  cargo:warning=      ^
  cargo:warning=src/low_level/extract.c:36:37: error: invalid application of 'sizeof' to an incomplete type 'struct Const []'
  cargo:warning=    const size_t const_len = sizeof consts / sizeof *consts;
  cargo:warning=                                    ^~~~~~
  cargo:warning=2 errors generated.
  exit status: 1

  --- stderr

  error occurred: Command "cc" "-O0" "-ffunction-sections" "-fdata-sections" "-fPIC" "-gdwarf-2" "-fno-omit-frame-pointer" "-m64" "-Wall" "-Wextra" "-o" "/home/andinus/dev/other/static-web-server-2.19.0/target/debug/build/signal-hook-932164e25ff6283b/out/src/low_level/extract.o" "-c" "src/low_level/extract.c" with args "cc" did not execute successfully (status code exit status: 1).

Hello.
According to Posix, that SI_MESGQ should be available. Can you somehow find out if OpenBSD just misses that thing (which is related to functions like mq_notify), or if it's available in some other header? I don't have that OS around and I would like to decide on how to fix that problem based on the info.

It seems like OpenBSD is missing that: https://github.com/boostorg/test/pull/231

OK, if this is a good enough fix for Boost, it should be good enough for this crate.
Can you test #147 before I merge it?

I ran cargo test ; cargo build on openbsd-compile-fix branch, it works. Thanks!

Released to crates.io as 0.3.16. Thank you.


python_package_installer

Pull Request Template
script name - AlPythonPackageInstaller
Brief about script
A python script based GUI interface to install python packages.
Issue no.(must) - #715
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me.
[x] One Change in one Pull Request
[x] My file is in proper folder (Name of folder should be in lowercase with no space in between) (E.g. meet_schedular)
[x] I am following clean code and Documentation and my code is well linted with flake8.
[x] I have added README.md and requirements.txt (Include version numbers too e.g. pandas==0.0.1) with my script
[ ] Just including required dependencies in requirements.txt (Don't include Python version too)

If issue was not assigned to you Please don't make a PR. It will marked as invalid.

Idk while flake8linter is failing. I have checked the code on pep8 online. There was no error in the code.


drivers: serial: nrfx_uart: Allow TX only mode

Extended nrf_uart driver to support TX only.
When RX pin is not provided then RX is not started at all. This
allows to achieve low power with logging/console enabled.
Similar to #22242.
Signed-off-by: Krzysztof Chruscinski<EMAIL_ADDRESS>

@ioannisg @carlescufi could you merge?


Add install instructions for developing locally

We had a link to the third-party docs for working with ui-example, within Dashboard, but we were missing instructions on developing locally in the ui-example repository itself.

The instruction to run npm install to install vite and other dependencies is missing from https://dashboard.flowfuse.com/contributing/widgets/third-party.html too.