AJAX: Send only datagrid snippets

In my project, I have $this->redrawControl() in startup in BasePresenter. As I want to redraw all snippets by default (and change this in render methods if needed).
However this breaks datagrid filtering, because in response I also get parent snippet, which redraws whole content (with grid).
I propose adding $this->getPresenter()->redrawControl(NULL, FALSE) here, which solves whole problem.
I'm not sure if this is BC break, but it could be avoided by adding onlyRedrawGrid option or something.
Also, onReload event would do the job (where I would call redrawControl(NULL, FALSE) myself).
I'm migrating from o5/grido, where on grid manipulation (filtering, paginating, ..) only grid snippets are returned no matter what I choose to redraw.

I think adding this line $this->getPresenter()->redrawControl(NULL, FALSE) wouldn't be right. It is a hack, not a solution. When you are redrawing all snippets in presenter, there of course have to be all snippets invalidated.
onReload  event is perfect solution. I will add that to the next version. Should i pass some arguments to the onReload  event? For example a list of snippets to reload, or something else?

As I look closely on the code, it looks like I'll have problem on more places (in signals you are redrawing controls aside of reload method - redrawItem, handleGetItemDetail, handleGetChildren).
So at this time, onReload event wouldn't solve this problem and would be wtf, as it is not really called on every grid interaction.
I'm not sure whether you can provide me any "grid changed" event or not.
I'll have a closer look if I can detect this without changes in datagrid, as you (and your code) is right that it's my problem.
PS: thanks for your fast reply! :)

Well, firing onReload event would make sense in each of methods you mentioned. At least if it won't be called onReload, but onRedraw.
The code is a little bit inconsistent, you're right. I will fix that later.

That'd be great!
I can't image what any would need in that event as parameter, probably list of snippets would be enough.
Thank you.

A added onRedraw event. I will be availible in next version.


add filtering packages by target

When a certain workspace contains target specific package selection then all packages and their dependencies are pulled into the resulting bitbake recipe.
This PR addresses this issue by adding an optional argument to generate the list of depend packages for a specific target.

fixed the optional argument for specifying the target


Integrate react cosmos

Hi again @Siddharth11,
I'm trying to make React Cosmos better by integrating it in OSS projects. This one was fairly easy so I'm submitting this PR in case you find it brings value to iTunes-Remote.
It's a fast & focused way to work on the client without starting (and waiting for) the backend. Here's what you can do with it:

Load any component from this repo into various states (fixtures already provided)
Open fixture editor (see gif) and see how Redux store maps to the loaded component
Track socket.emit calls inside console

Steps to test it on your machine:
git clone -b integrate-react-cosmos git@github.com:skidding/iTunes-Remote.git
cd iTunes-Remote
npm i
npm run cosmos
# open http://localhost:8989/

I would be grateful for any feedback. Thanks!

Hi,
I appreciate your work with React cosmos, it looks great! But to be honest, as of now, all this stuff is beyond my comprehension to review. I want this project to be simple. Can we keep this in a separate forked repo itself?


[RESTEASY-2366] Introduce RESTEasy integration with WildFly Elytron

@Skyllarr , thanks for the PR. Can you please ensure that the concerns I had on #2185 are addressed here as well? Thanks

Nevermind, I overlooked this.


archive .ipa

I want to know how to archive .ipa. I have downloaded the source codes of webrtc_ios and complied with Xcode.Thank you very much!

I guess this is not a question about apprtc itself right? I guess this is something related to the Apple process itself to distribution. Here is a link to help:
https://developer.apple.com/library/content/documentation/IDEs/Conceptual/AppDistributionGuide/SubmittingYourApp/SubmittingYourApp.html


Add-SPONavigation failes for URL with same address as Site URL

This week something strange happened.
When I add a node on a site and the node url starts with the same as the site url, the command fail.
Example:
On a site with the url: https://mytenant.sharepoint.com/sites/sales,
I run: Add-SPONavigationNode -Url 'https://mytenant.sharepoint.com/sites/sales' -Location TopNavigationBar -Title 'Sales'
Nothing happens and there are no error messages.
If I do this on the rootsite, I get some errors, and no site realted to the root site is added.
All urls starting with https://mytenant.sharepoint.com, will fail.
The error message for the site /sites/sales is:
Add-SPONavigationNode : Cannot open "/sites/sales": no such file or folder.
CategoryInfo          : WriteError: (:) [Add-SPONavigationNode], ServerException
+ FullyQualifiedErrorId : EXCEPTION,OfficeDevPnP.PowerShell.Commands.AddNavigationNode

Sorry for the delay in replying to this one: I just tried it on a site myself, but I encounter no issues. What kind of sitedefinition/template is your site based upon?

It is a Team Site. STS#0
I also have some other issues with this root tenant site.
I will try the same code on another o365 tenant root site and let you know how it goes

I ran into this issue as well.  Any update on this?

I think I can reproduce it: enable the Publishing Feature on Site Collection. The CmdLet still works if the specified url is NOT in the current site collection.

I have enabled publishing features on the site i am working with as well.

With the current release of the cmdlets I cannot reproduce this issue with both the publishing feature and not active. I will close this issue


feat: support send component message in chat

Types

[x] 🎉 New Features

Background or solution
The sumi.chat.sendMessage method support passing in a component type message
Changelog

/next

/next

LGTM


TypeError: Cannot read property '_headerSent' of undefined

I'm scraping a lot of urls in search of their RSS-feeds. Some errors just quit the whole script when encountering some domains, which is of course not desired behaviour. I tracked down one of the errors.
On line 967 of node/index.js is this:
if (method !== 'HEAD' && !req._headerSent) {
If I try to get "https://gtex.nl", the nodejs script exits with "TypeError: Cannot read property '_headerSent' of undefined". I know the url isn't working in browsers as well, but I think superagent has to throw an error, not stop the script entirely. So I wrote this above the current line 967:
if (!req) {     console.log('Failed on url:', this.url)     return this.callback(new Error('Req object doesn\'t exist'));   }
Now it doesn't exit the script and just throws an error I can catch. I don't know if this is a desired solution, but can this be fixed?
Edit: To avoid confusion, I'm not the owner of gtex.nl, so I can't change whatever they did wrong.

If you can provide a reproducible test case let me know.

Just wanted to chime in that I'm also seeing this randomly - if I can come up with a reproducible case I will post back. FWIW it is happening especially when superagent-cache-plugin is used.

Thank you @contra for letting me know.  Would love to see a reproducible test case with express/superagent or something.

The culprit seems to be this on line 526 of src/node/index.js:
delete this.req;

Which in turn happens when redirection occurs.

And this may be related to a redirection occurring, then having been aborted.  I'm not 100%. Hopefully either of you could find the issue, I'm super limited on time.

This was initially added here https://github.com/visionmedia/superagent/blame/v4.1.0/lib/node/index.js#L505 from https://github.com/visionmedia/superagent/commit/0ecc2e4f0c223375a45cb0ed8a0ad2f09be2fafb

Deleting that line causes the tests to break for redirection.  I think that with the information I provided above, you could write a test that reproduces this @contra ?

@niftylettuce Yep thanks for tracking that down - I will try to debug further and hopefully submit a PR. In the meantime can you open this ticket and assign it to me so it shows up on my work list?
Weird thing in our situation is there should be no redirects, they are requests fetching off a github CDN URL that should be constant.


Add mention about cookie replay to limitations.md

As cookie replay could be a security threat (client replays the same cookie multiple times and discards new cookies set by the application), I suggest we should pay attention to it somehow in the limitations.md.
Maybe add new paragraph "Cookie replay" and mention something like (just a quick suggestion):

It should be pointed out that PSR7Session does not address replay of old cookies. Legitimate client can send previous (old) cookies within the cookie lifetime period.

to make the cookie replay issue stand out better.
I was thinking the Cookie replay paragraph could come right before:

The idea around PSR7Session is that a session is not supposed to be an actual storage for transient client information, but rather be used for the concerns of authentication, authorization and eventually for validation concerns such as CSRF-token validation.
If you want to store frequently-updated or concurrently-updated information inside a session, then PSR7Session is likely not fitting your use-case.

Thus making those two paragraphs be under Cookie replay title and paragraph.
Thoughts?

@timoh6 it shall be done, thanks!

Handled in #39


HammerJS issue

Because you don't have swipe support or touch in general, I added HammerJS to Angular as it has built in support for that. Everything works fine until I try to close sidebar by swiping again - it just doesn't trigger the event again because aside is somehow blocking it. It is triggered when I try to swipe on backdrop tho.
  @HostListener('body:swipeleft', ['$event.pointerType']) left(pointer: string) {
    if (pointer === 'touch' && this.mobile) {
      // dispatch to store
    }
  }

Although, it works on my Iphone SE, it doesn't work on android device I tested on. Same as for Chrome Dev tools touch thingy. Any ideas?

A quick Google search showed issues with hammerjs swipe support in some Android browsers, so I suspect it's something to do with that. Nothing comes to mind for the sidebar's implementation that would cause this.

I was testing on Android device but on Chrome browser. The issue was on Chrome Dev tools mobile mode also.


Add Expand Capability to pfNotificationDrawer

@dtaylor113 @dlabrecq @dgutride Please review

Looks good (other than the one param documentation note above)

Fixed typo in documentation ( @dgutride issue )

Added a bit more demo capabilities (mark read on click, implement Mark All Read, Clear All)

Looks great @jeff-phillips-18 ! LGTM 👍

Added a touch of css cleanup to prepare classes to be in base patternfly.

LGTM.  I'd like to suggest an MiQ PR practice of including a screenshot of the area of the GUI/widget which is being updated.

Looks good to me.


does Microsoft.OperationsManagement support ARM TAGs ?

Hello there,
Getting a BadRequest “Endpoint Not Found” while trying to add an ARM TAG (through a PATCH) into a Microsoft.OperationsManagement resource:
"/subscriptions//resourceGroups/<ResourceGroup/providers/Microsoft.OperationsManagement/solutions/Security(DefaultWorkspace-GUID)"
I wasn't sure if that was because of the parenthesis " ( " and " ) " in the resouce name - of if because there's no PATCH keyword in https://github.com/Azure/azure-rest-api-specs/blob/master/specification/batch/resource-manager/Microsoft.Batch/stable/2017-09-01/BatchManagement.json

Hi @axelgMS sorry its very old issue, do you still met such issue?

haven't tried recently - but forwarding you an email with more context


[Feature Request]: Configurable Events to Listen for Cache Flushing

Summary
Make cache easier to bust
API Changes
Configuration changes
Intended Use Case
All in the summary

No longer needed.  Use IMemoryCache service


IME at wrong position

In order to type Chinese or Japanese text an Input Method Editor (IME) is required. It looks like SDL already does most of the heavy lifting and provides said editor. But as lagrange does not provide SDL with information where to open the IME, the position is incorrect, see the attached screenshot:

It looks like SDL_SetTextInputRect can be used to set the correct position.

I added the calls to SDL_SetTextInputRect (abe38195588660abc717f13454d8dd28e425fd9e). Let's see if that helps.

That does help! It is not quite perfect yet, for example here the position is still off:
 and the IME should be placed under the cursor not the input field:

And one more thing: Typically you can either let the IME handle all text editing (rendering a little text field with a cursor in a separate window) or do it yourself and the IME only displays possible candidates to select (SDL's default), in which case SDL_TextEditingEvents have to be handled.
It looks like this can be used to let the IME handle text editing, which is most likely easier than handling new events:
SDL_SetHint(SDL_HINT_IME_INTERNAL_EDITING, "1");

See comment: https://github.com/skyjake/lagrange/issues/417#issuecomment-985511677


Laravel Recipe Improvements

Q
A

Issue Type
Feature Request

I see a few issues have been opened for Laravel. To keep it central, imho it needs the following features:

[x] Storage/writable folder improvements (#572 #622 + below)
[x] Public storage folder symlink to shared storage (see quote below)
[ ] Clear cache after deployments (eg. cached views/html parts; php artisan cache:clear
[ ] Clear old cache and cache new config; both for speed and security (php artisan config:cache) (#527)
[x] Update shared folders before vendors. This because of php artisan optimize being dependant on environment, so needs shared .env file.
[ ] Add optional helpers tasks for php artisan migrate and php artisan db:seed, using the --force flag for non-interactive mode (but should perhaps confirm on the deployer side. And perhaps php artisan migrate:status for info. (#541 #622)

About Laravel folders, should we update the recipe to follow the 'guidelines' set by Taylor Otwell http://taylorotwell.com/spark-storage/

When using deployment systems like Envoyer, an entirely fresh copy of the repository is installed with each deploy and the storage directory is attached to a permanent storage directory using a symbolic link. If you store your user uploads in public, they will all go missing on your subsequent deployment. The solution is to store the uploads in your persistent storage directory and then create a symbolic link from your public directory to your storage directory during your deployment process.
Before yesterday, there was no conventional place to store these user uploads other than “anywhere you want within the storage directory”. After reviewing a few libraries in other languages, it became apparent that there would be great benefit to designating a standard location within the storage directory for files intended to be publicly accessible. So, I created the storage/app/public directory in the Laravel repository so that it will be present on all future installs of Laravel 5.2.

So the default would be this

Shared: .env + entire storage folder
Writable:  storage + bootstrap/cache (I would recommend sharing the entire storage folder)
Create symlink public/storage to storage/app/public (run("ln -s {{deploy_path}}/storage/app/public {{deploy_path}}/public/storage "); ?)

@barryvdh Can you send link about

Create symlink public/storage to storage/app/public (run("ln -s {{deploy_path}}/storage/app/public {{deploy_path}}/public/storage "); ?)

in Laravel document. I read http://taylorotwell.com/spark-storage/ but i support it only it is included Laravel document.

Okay, I've added a PR here: https://github.com/laravel/docs/pull/2265

See https://laravel.com/docs/5.2/filesystem#configuration

The public disk is meant for files that are going to be publicly accessible. By default, the public disk uses the local driver and stores these files in storage/app/public. To make them accessible from the web, you should create a symbolic link from public/storage to storage/app/public. This convention will keep your publicly accessible files in one directory that can be easily shared across deployments when using zero down-time deployment systems like Envoyer.

[x] Storage/writable folder improvements (resolved in #622 )
[x] Public storage folder symlink to shared storage.(resolved in #625 )
[x] Clear cache after deployments (eg. cached views/html parts; php artisan cache:clear (resolved in #625 )
[x] Clear old cache and cache new config; both for speed and security (php artisan config:cache) (resolved in #625 )
[x] Update shared folders before vendors. This because of php artisan optimize being dependant on environment, so needs shared .env file. (resolved in #622 )
[x] Add optional helpers tasks for php artisan migrate and php artisan db:seed, using the --force flag for non-interactive mode (but should perhaps confirm on the deployer side. And perhaps php artisan migrate:status for info. (resolved in #625 )

Hey Barry,
I think this line is wrong?
Create symlink public/storage to storage/app/public (run("ln -s {{deploy_path}}/storage/app/public {{deploy_path}}/public/storage "); ?)
It should be the following?
Create symlink public/storage to storage/app/public (run("ln -s {{deploy_path}}/storage/app/public {{release_path}}/public/storage "); ?)

Hmm no shouldn't matter, as storage is a symlink anyways.

The following line is what I had to use:
run("ln -s {{deploy_path}}/shared/storage/app/public {{deploy_path}}/current/public/storage");

Oh probably mixed release path and deploy path indeed. But was just an example, haven't actually used this in production yet.

And most is already implemented here: https://github.com/deployphp/deployer/pull/625

Merged.


draggableCancel not working? - [Question]

Hi all,
I'm trying to add a simple div in absolute position with class "dragCancel".
My data-grid contains the flag " draggableCancel: '.dragCancel' ".
Now if I try to drag the element it doesn't care at all where I grab it, the "undraggable" area is not doing its work... How to solve it?
HTML
 <ReactGridLayout className="layout" cols={1} rowHeight={35} width={500}>

        It should drag from here..

...but it should not from here...

</ReactGridLayout>

CSS
  .dragCancel {
    position: absolute;
    bottom: 0;
    left: 0;
    width: 100px;
    height: 30px;
    background-color: green;
  }

My bad, I fixed it passing the classname as a prop in the ReactGridLayout rather then in the data-grid.
<ReactGridLayout className="layout" draggableCancel='.dragCancel' cols={1} rowHeight={35} width={500}>

        It should drag from here..

...but it should not from here...

</ReactGridLayout>


Using the SDK for a target instead?

Is it possible to use the SDK if just wanting to compile a package?
The SDK make menuconfig does not have the:
"Advanced configuration options (for developers)" => "Enable package source tree override" feature.

Yes, But I haven't done that in a while and do not know the details right now.


[Collections] Don't allow package.versions to be empty

Motivation:
Currently packages in a collection can have versions array be empty. Can't remember if this was intentional but a package that has no versions is not very useful.
Modification:
Update validation to fail when package.versions is empty.

@swift-ci please smoke test


TFRS - DevOps and Development Work for January Release

Describe the Task:
The task involves orchestrating and executing all necessary development and DevOps procedures for the upcoming January release of the TFRS (Transportation Fuels Reporting System). This includes finalizing and integrating new features, addressing outstanding issues, conducting comprehensive testing, and ensuring that the system is fully prepared for deployment.
Purpose:
The primary objective of this task is to ensure a smooth and timely rollout of the January update for TFRS. This update is intended to enhance system capabilities, improve the overall user experience, and rectify any significant issues from the previous release. The successful completion and deployment of this update are critical in upholding our commitment to delivering continuous improvement and ensuring user satisfaction.
Acceptance Criteria:

[ ] Implementation and integration of all scheduled features for the January release are completed successfully.
[ ] Comprehensive end-to-end testing of new features and critical functionalities to guarantee system stability and optimal performance.
[ ] Identification and resolution of all significant and critical bugs found during the testing phase.
[ ] Documentation is updated to accurately reflect the changes and new additions in this release.
[ ] Successful deployment of the update in the staging environment, followed by a meticulous review and clearance for production deployment.
[ ] All monitoring and logging systems are updated, tested, and confirmed operational to effectively track the performance post-update.

Additional Context:

Close collaboration with the Product Owner is imperative to ensure a thorough and effective testing cycle.
A contingency plan for rollback should be in place and ready to be executed in case any critical issues arise post-deployment to the production environment.

Nothing for PO to test

Nothing for PO to test


remove -status in AddVersionedApiExplorer not work

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
i use  [ApiVersion("1.0-Public")] attribute
for split definitions in Swagger after that for routing just added
.AddVersionedApiExplorer(opt =>
                {
                    opt.GroupNameFormat = "'v'VVV";
                    opt.SubstitutionFormat = "V";
                    opt.SubstituteApiVersionInUrl = true;
                });

to set the major version
[Route("v{version:apiVersion}/[controller]/[action]")]
this sample works fine
var apiVersion = new ApiVersion( 1, 0 , "Public");
Console.WriteLine( apiVersion.ToString( "V" ) );

but in asp.net core APIs formatting of the version doesn't work
Expected Behavior
No response
Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version
5.0.0
Anything else?
No response

Can you quantify what you mean by it doesn't work? What happens? What do you expect to happen?
As shown, this is what will happen:
var apiVersion = new ApiVersion( 1, 0 , "Public");
Console.WriteLine( apiVersion.ToString( "V" ) );

Output:
1

The full formatting options are documented in the wiki under the version format topic.
The status is fundamentally part of the API version when you use it. It is possible to format parts of an API version for display purposes, but you cannot abuse to formatting create a 🐶 and 🐴 show where the client asks for some API version - say 1.0 and it somehow magically maps to another API version such as 1.0-Public. If your API version is 1.0-Public, then the request must include the full API version (ex: ?api-version=1.0-Public). Partial and implicit matches are not allowed - by design.
You can hide the status when formatting the API version used for grouping purposes. For example, you can show 1.0, when it really maps to 1.0-Public. The only convenience is that the name is shorter. I'm not sure why you'd want to do that. It would be confusing to clients or anyone reading the documentation. They still have to use 1.0-Public in Try It Now! as well as any other request.

my purpose is to divide definitions in swagger like this

but I do not want any change in my routine I already read
https://github.com/dotnet/aspnet-api-versioning/issues/516
is that logical?

Based on what you are showing, it would appear your goal is to provide an additional level of organization or grouping to a set of APIs. An API version status is not meant for that.
There are multiple ways that you can provide additional grouping, but at the end of the day, the Swagger UI only affords for a single dropdown without some significant customization to the UI. I can't say exactly how, nor can think of someone that has put forth that much effort, but it is technically possible.
ASP.NET Core and the API Explorer explorer do provide a way to define a group, but that typically conflicts with using the API version itself as the group once you add versioning. The desire to have both has come up enough that now there is a built in feature to support it.
Let's assume you have the following controller.
[ApiVersion(1.0)]
[ApiController]
[ApiExplorerSettings(GroupName="Admin")]
[Route("admin/[controller]")]
public class SecretsController : ControllerBase
{
    [HttpGet]
    public IActionResult Get() => Ok();
}

This defines the group Admin which has the API version 1.0. Next you'll need to make the following changes to the configuration setup:
services.AddApiVersioning()
        .AddApiExplorer(options =>
        {
            options.GroupNameFormat = "V";
            options.FormatGroupName = (group, version) => $"API v{version}-{group}";
            options.SubstitutionFormat = "V";
            options.SubstituteApiVersionInUrl = true;
        });

In order for the FormatGroupName callback to be invoked, it must be defined (obviously) and a group name must be specified via IApiDescriptionGroupNameProvider.GroupName, which can be done through ApiExplorerSettingsAttribute. The callback receives the group name and the ApiVersion already formatted as a string according to GroupNameFormat. You are then free to decide how the two values are combined.
If you did this for all of your controllers, that will display in the list as you've shown.  The actually API version is still 1.0 so the request URL would be /admin/secret?api-version=1 or whatever method of versioning you've configured.
Also be aware that you've changed the SubstitutionFormat to "V" which will only include the major version number. If you ever have an API version which includes a minor version, the wrong requests will be made from the Swagger UI. The default is "VVV", which will include the major version, minor version if it is not 0, and the status, if there is one. That is probably 99% of the cases. It is configurable in case you want something else like "F" or "VVVV", which will show all of the parts, all of the time, as long as they are present.


a

Codecov Report

Merging #1815 (532302f) into master (8062c2b) will decrease coverage by 4.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1815      +/-   ##
==========================================
- Coverage   95.05%   90.85%   -4.20%
==========================================
  Files         353      353
  Lines        3315     3315
  Branches      407      407
==========================================
- Hits         3151     3012     -139
- Misses         81      218     +137
- Partials       83       85       +2

Impacted Files
Coverage Δ

packages/client/lib/commands/ZRANGESTORE.ts
20.00% <0.00%> (-80.00%)
:arrow_down:

packages/client/lib/commands/GETEX.ts
21.42% <0.00%> (-78.58%)
:arrow_down:

packages/client/lib/commands/COPY.ts
33.33% <0.00%> (-66.67%)
:arrow_down:

packages/client/lib/commands/ZINTER.ts
36.36% <0.00%> (-63.64%)
:arrow_down:

packages/client/lib/commands/ZUNION.ts
40.00% <0.00%> (-60.00%)
:arrow_down:

packages/client/lib/commands/GEOSEARCHSTORE.ts
41.66% <0.00%> (-58.34%)
:arrow_down:

packages/client/lib/commands/XAUTOCLAIM.ts
44.44% <0.00%> (-55.56%)
:arrow_down:

packages/client/lib/commands/GEOSEARCH_WITH.ts
50.00% <0.00%> (-50.00%)
:arrow_down:

packages/client/lib/commands/LMOVE.ts
66.66% <0.00%> (-33.34%)
:arrow_down:

packages/client/lib/commands/BLMOVE.ts
66.66% <0.00%> (-33.34%)
:arrow_down:

... and 23 more

Continue to review full report at Codecov.

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


Upgrade PHPUnit

Q
A

Bug fix?
No

New feature?
No

BC breaks?
No

Deprecations?
No

Fixed tickets
N/A

Brought to you by PHP CS Fixer

Tests are falling.

Please, could you point where?
Travis is passing on all jobs.
Scrutinizer error is not on executing tests, but TEST SETUP FAILED with There was an error (Event-ID 753253043b4b4c4fb114847fc77591c0). I cannot fix internal Scrutinizer issues ;)

Hmm, okay. I will check it myself.


social auth redirect in configuration

would it be possible to have a different url in the config for the social auth redirection ?

I guess we could set a different base url for the social auth and social link since they might differ from the rest of the API requests.
Although technically you'll need to talk to the same backend.

Part of the 0.5.4 release

thanks !


[Feat] Add bonuses for the various attack types to the actor template

Add bonuses for:

Melee weapon attacks
Ranged weapon attacks
Melee spell attacks
Ranged spell attacks

We should also modify the existing bonuses.spell object, which looks like this:
"spell": {
  "attack": "",
  "dc": ""
}

With melee and ranged spell attacks getting their own bonuses, this can be replaced with a spellDC field, to match the bonuses.maneuverDC attribute. This will require a migration, but it's extremely simple.
When making this change, we must not forget to also update the spell tab's config dialog.

Added as part of 0b3fca45c58832bd6d32925777075cd79bb0b2cb. Closing this issue as completed.


I can't load a SharePoint project with gulp serve

Thank you for reporting an issue or suggesting an enhancement. We appreciate your feedback - to help the team to understand your needs, please complete the below template to ensure we have the necessary details to assist you.

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.
If you are planning to share a new feature request (enhancement / suggestion), please use SP Dev UserVoice at http://aka.ms/sp-dev-uservoice.

Expected or Desired Behavior

If you are reporting a bug, please describe the expected behavior. If you are suggesting an enhancement please describe thoroughly the enhancement, how it can be achieved, and expected benefit.

Observed Behavior

If you are reporting a bug, please describe the behavior you expected to occur when performing the action. If you are making a suggestion, you can delete this section.

Hello, I am getting started with SharePoint Framework and setting up the development environment so I have installed node 10, PowerShell, Chocolate, Boxstarter and VisualStudio. I also have installed Yeoman with npm install –g yo gulp and the generator SharePoint with npm install –g @microsoft/generator-sharepoint. I have created my Project with SharePoint Online only latest, component webpart and no JavaScript Framework.
Then, I have certificated gulp trust-dev-cert and I have launched gulp serve.
The error is the next:

I am working in Windows 10. What is wrong there?
Steps to Reproduce

If you are reporting a bug please describe the steps to reproduce the bug in sufficient detail to allow testing. Only way to fix things properly, is to have sufficient details to reproduce it. If you are making a suggestion, you can delete this section.

Submission Guidelines

All suggestions or bugs are welcome, please let us know what's on your mind.
If you are reporting an issue around any of the documents or articles, please ensure that you have clear > reference on the specific file or URL, which should be fixed.
If you have technical questions about the framework, we’ll be monitoring #spfx, #spfx-webparts, and > #spfx-tooling on (SharePoint StackExchange)[http://sharepoint.stackexchange.com/]. You can also > alternatively submit your question to (SharePoint Developer group)> [https://network.office.com/t5/SharePoint-Developer/bd-p/SharePointDev] at Office Network.
Remember to include sufficient details and context.
If you have multiple suggestions or bugs please submit them in separate bugs so we can track resolution.

Thanks for your contribution! Sharing is caring.

Please help

Interesting.  But yes, ideally use a straight forward path.  If you move your solution to c:\code... does everything work?

Current LTS version of NodeJS is 10.x. Notice that 9.x or 10.x versions are currently not supported with SharePoint Framework development.

@aagbarcelona Do you have tried Node 8?

It seems is working all. Thanks

Glad to hear.


[embedded] Use Builtin.RawPointer as the argument type on swift_release_n

The added test triggers a call to swift_release_n, which currently ends up causing an LLVM IR verification failure because swift_release_n is, at the IR level, expected to have a ptr argument, but UnsafePointer? in Swift code ends up lowering as i64. Let's avoid using it and instead use Builtin.RawPointer as the argument type, like we do in the other runtime refcounting functions.

@swift-ci please test


Implement a static search results page.

Rebased and applied some filter/sorting fixes, so it yields the same list as the find-as-you-type search:

Hah nice. Amusingly, yesterday, I implemented a javascript search for adrdox after doing a cgi style one first. I generally prefer the server-side generated thing, but the javascript one was requested for compatibility with github pages.

For ddox, I think, it was the other was around because dlang.org is just organized as a bunch of HTML files, so static generation was the priority. But apart from that I also prefer a server-side solution, and of course downloading the whole symbol file ~800KB eagerly isn't really nice either (~70KB gzipped).

Aye, I had the same problem. And the server side one can cache the indexes much better giving quite a nice speedup. Regardless, this will be good!

True, I didn't tackle indexes yet, because they really only get interesting on a more global scale (like on dpldocs.info). For a few thousand symbols, the search time isn't really noticeable yet. My next priority will have to be memory use, which is quite high right now with the uncompressed AST stored fully in memory.


Add an option to disable the question about creating the solution template folder

I personally don't use templates in every project, but vscode-solution-explorer keeps nagging me when opening a new workspace.

Thank you very much @remcoros!!
I'll merge it on the next release :)


Remove media_types video when not supported by code

The documentation states that smartadserver and rubicon are ready for video media types, but the spec in smartadserverBidAdapter.js and rubiconBidAdapter.js does not exports video as a valid media type.

@xavierleune - good catch for Smart and AOL, but Rubicon adapter supports it. https://github.com/prebid/Prebid.js/blob/master/modules/rubiconBidAdapter.js#L91
supportedMediaTypes: [BANNER, VIDEO],

Rather than removing video support documentation from anyone, I'd like to give them a chance to explain and fix the adapters. Will reach out to them.

Email sent to registered maintainer addresses. Also adding @vzhukovsky and @ytyukhnin as the most recent adapter updaters of AOL and Smart respectively.

Hi @bretg
Thanks for your feedback. Rubicon may be supporting it in the code, but this is not publicly released. They are in a test phase and it will not be released until Q2, so this adapter should not be listed as video ready IMO.
I think for smart & aol / verizon you should not be listing those adapters as video ready.

Heard back from AOL - they're ok removing video. But Rubicon wants to keep theirs. Smart wanted to be clear they support a custom version of outstream. So for now I'm trying this compromise:
https://github.com/prebid/prebid.github.io/pull/1194
I'll work with Smart to see if we can them to support outstream in a standard way.
In the meantime, thanks for caring about the quality of the documentation @xavierleune
Will leave this open until you get a chance to look at the other PR and confirm you're satisfied with this as a first pass.

Thanks for your new pull request, this seems fine to me. I'm closing this pull request as it's now resolved with #1194


tsv-summarize: "total line" when grouping.

Only a small ER this time. ;)
I hit a case where I'd like to have a "total" line in addition to the output of my --group-by.  I can get it by removing the --group-by option and rerunning the input, but it'd be nice if I could get them both in one go.

Makes sense. Not sure if it's a small ER or not :)  Depends on how I structured that part of the code. Hopefully small. I'll look into it.


[SPARK-48725][SQL] Integrate CollationAwareUTF8String.lowerCaseCodePoints into string expressions

What changes were proposed in this pull request?
Use CollationAwareUTF8String.lowerCaseCodePoints logic to properly lowercase strings according to UTF8_LCASE collation, instead of relying on UTF8String.toLowerCase() method calls.
Why are the changes needed?
Avoid correctness issues with respect to code-point logic in UTF8_LCASE (arising when Java to performs string lowercasing) and ensure consistent results.
Does this PR introduce any user-facing change?
Yes, collation aware string function implementations will now rely on CollationAwareUTF8String string lowercasing for UTF8_LCASE collation, instead of UTF8String logic (which resorts to Java's implementation).
How was this patch tested?
Existing tests, with some new cases in CollationSupportSuite.
6 expressions are affected by this change: contains, instr, find_in_set, replace, locate, substr_index.
Was this patch authored or co-authored using generative AI tooling?
No.

thanks, merging to master!


Add connectrpc/swift v0.8.0

Is there also a connect-swift-mocks we need to add?

Is there also a connect-swift-mocks we need to add?

yup building it now https://github.com/bufbuild/plugins/pull/817


Hide Activity usage with CompositionLocal

Because CompositionLocals are cool and should be used for everything and the kitchen sink.

Have some formatting changes too, why not 🤷♂

@ashdavies good time to merge, no conflicts 🔥


foundationdb: 7.1.32 -> 7.3.56

Includes fixes for GCC 14 and using non‐vendored libraries. The alternative to bumping to the next major version here would be to always build with Clang, which might not be a terrible idea anyway as it’s what upstream use for their official binaries on all platforms (or to patch 7.1 for the missing #include and potentially other issues, but it seemed like we were overdue for an update anyway and there’s a clean upgrade path).
Everything builds and the NixOS Stalwart Mail test passes, though I don’t know if that exercises FoundationDB or not. cc @happysalada @onny @oddlama @Pandapip1 as Stalwart Mail maintainers.
cc @thoughtpolice – are you still using/maintaining this? Before this the package hadn’t seen an update since May 2023, which seems a bit worrying for database software (though I guess FoundationDB is probably more stable than most). It seems like the only in‐tree user is Stalwart Mail.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:fast_forward: 1 package blacklisted:

nixos-install-tools

:white_check_mark: 10 packages built:

foundationdb
foundationdb.debug
foundationdb.dev
foundationdb.lib
foundationdb.pythonsrc
python312Packages.foundationdb73
python312Packages.foundationdb73.dist
python313Packages.foundationdb73
python313Packages.foundationdb73.dist
stalwart-mail

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[ ] 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

[x] 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/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[x] (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.

Foundationd tags latest as 7.3.43 and everything after as prerelease. Ive been using 7.3.43 at day job but there have been some problems with it (making a multi region deployment) that dont happen with 7.1
All in all im not against updating to 7.3, but their official documentation still defaults to 7.1
7.3 is not the latest stable i think.
Personally i would use 7.3.43 rather than 56. I dont feel too strongly about that though, happy to merge the PR as is. We should just be aware that there might still be rough spots with 7.3 that are not in 7.1

Oops, yes, I even saw that some of the versions were pre‐releases but just forgot when deciding to try 7.3. I’ll put it at the actual latest stable 7.3. I believe that 7.3.43 is considered stable and production‐ready per https://github.com/apple/foundationdb?tab=readme-ov-file#getting-started, and 7.1 is in maintenance mode (until 7.5, presumably). https://apple.github.io/foundationdb/ seems to point to 7.3.57; do you have a pointer to where they’re recommending 7.1 as a default?
I’m not opposed to sticking on 7.1 either but we’ll either have to match their use of Clang (to hide the error) or effectively maintain compatibility for GCC 14 ourselves (which might not be too bad if the churn is low enough). It seemed like getting up to date early in the 25.05 cycle and dropping some of our patches would be preferable to doing that work, but as I’m not a FoundationDB user myself I don’t know how much churn upgrades cause.

Changing to 7.3 sounds good to me.
Last time i checked the docs , the default was 7.1, they must have changed it since then.
Very weird that the doc shows 7.3.57 even though they say its a pre release.
7.3.43 is fhe safe choice i feel but i dont feel too strongly about it.

Adjusted to 7.3.42 (not .43, as that’s the AVX version – why do they do it like that??); Stalwart continues to build.
@happysalada Are you interested in maintaining this package, by the way? Seems like it could use some attention for version bumps.

Thank you.
I think i read on the roadmap that there is a big update planned for march 2025.
Ill try to make the update at that time and add myself as a maintainer. I cant comlmit right now though, but im hoping the situation will get better at day job until then .

This PR looks good to me, not sure why ofborg is not evaluating.

ofborg eval is very slow these days. The GitHub Actions evaluation passed.


Add existing Claim to PublishedClaims

So this is the editing of live / public claims

Back on it! Think the problem is that the reasons within the argument groups aren't in the correct format for the db to save. Will explore

Whoop -Claim.findByIdAndUpdate works, will probably have to use this guy more often. But for now - it's just updating one side of the arguments, would be great if it could do both


Cleanup repos and branches for flask and nodejs backend applications

[ ] Create and new repo for each backend application

[x] Combine the flask backend and GSA-Database

[x] https://github.com/nanoMFG/GSA-Database/issues/30
[x] Merge up-to-date branches into main branches for each repo
[ ] Document development workflow for each repo (how to use branches, when to merge, etc.)

Didn't create new repos except for GSA-website which is the shell for the production application.  Made new issues for documenting the workflow for each repo


Remove height rule that breaks examples on Firefox

The rule breaks the examples documentation page.
With the rule, the examples show like this

Without the rule

Tested on Chrome and the rule doesn't change anything.

The documentation needs to be rebuilt for this to work, right? @whawker

Yes, although I'm about to publish version 3.3.0 along with a new Highmaps package, so will do it then.

@whawker Okay, cool.


Use --include-tags-regex to exluce shortcut tags

I want to exclude this line.
https://github.com/yykamei/block-merge-based-on-time/pull/28/files#diff-06572a96a58dc510037d5efa622f9bec8519bc1beab13c9f251e97e657a9d4edR13

OK!
https://github.com/yykamei/block-merge-based-on-time/pull/30/files


A very good future development suggestion for flutter

flutter对于ui已经快做到极致了，但是后台服务和富文本编辑实在太复杂和太弱了！
比如我想用flutter开发一个定时提醒功能，时间到了，就弹出自定义页面，类似于系统闹钟（不是通知功能），现有的解决方法极其复杂，效果也不好。
现在的app，其实主要就是web的移动版，那么对于富文本编辑器就非常的依赖了，基本都是用webview去实现的，效率很低，webview毕竟很庞大。
为什么flutter不去开发一个类似微软IE浏览器提供的document.execCommand命令，简直简单好用到极点！那些flutter富文本框架反而很复杂。
真心希望在flutter工作的中国同胞，能提醒一下flutter的开发方向：漂亮的UI，强大后台功能，简单！甚至取代安卓！
如果都要调用安卓才能实现，我用flutter干嘛呢，哎！
Translation:
Flutter is almost perfect for ui, but the background service and rich text editing are too complicated and too weak!
For example, I want to use flutter to develop a timed reminder function. When the time is up, a custom page will pop up, which is similar to a system alarm clock (not a notification function). The existing solutions are extremely complicated and the effect is not good.
The current app, in fact, is mainly the mobile version of the web, so it is very dependent on the rich text editor, which is basically implemented with webview, which is very inefficient, after all, webview is huge.
Why doesn't flutter develop a document.execCommand command similar to the document.execCommand provided by Microsoft Internet Explorer? It's so simple and easy to use! Those flutter rich text frameworks are rather complicated.
I sincerely hope that Chinese compatriots working in flutter can remind us of the development direction of flutter: beautiful UI, powerful background functions, and simplicity! Even replace Android!
If you have to call Android to achieve it, what am I doing with flutter, hey!

flutter 2.2.2 会因为各种原因卡在flutter doctor，这就是个坑，这个坑填平后再说吧，我在使用flutter1.X时，没遇到，现在遇到了，flutter upgrade, sdkmanager update, 各种操作之后，还是会卡在Running "flutter pub get" in flutter_tools...， 唉，简直无语了。

@freedemocracy

Why doesn't flutter develop a document.execCommand command

I am not too sure if you are referring to this, but check out https://api.flutter.dev/flutter/dart-html/Document/execCommand.html
Also, regarding richtext editor, check out this open request, https://github.com/flutter/flutter/issues/12675
I am going ahead and closing this as duplicate. If you disagree, write in comments and I'll reopen it.


Update index.tsx

@Kieran-McIntyre, thank you for reply, I changed pull request. Can you please check?

Details

Hi @aytajj. Thanks for making that change. The unit tests are still failing. Can you please fix them? Once they are updated, I will take another look. Thanks!

Dear @Kieran-McIntyre, I think it will work now. Can you please check?

Dear @Kieran-McIntyre, I think it will work now. Can you please check?

Hi @aytajj. The tests are still failing. Can you please run the tests locally before pushing your commit to ensure they're passing? Run command yarn test in your terminal.

Hi @Kieran-McIntyre, I am sorry,  letterListContainerStyle must be optional.  I added question mark. Test was successful. Thank you

@Kieran-McIntyre can you please check? All tests was successful.

@Kieran-McIntyre can you please check? All tests was successful.

Thanks @aytajj. One last thing, please could you update the README.md with the new prop?

Hi @Kieran-McIntyre, I've updated the pull request, please check it out. Thanks.

Hi, @Kieran-McIntyre, Would you please check?

Hi, @Kieran-McIntyre, Would you please check?

@aytajj  this has now been merged and published. Your change can be found in v2.1.0. Thanks!

@Kieran-McIntyre , Thank you!


Kroxylicious Release

Release API version 0.1.0 Release version 0.1.0

Closing until https://github.com/kroxylicious/kroxylicious/pull/215 is merged at which point I will re-run (and thus regernate this PR)


fix: Add tags variable to public registry resource

Description

Add tags variable to public registry resource

Motivation and Context

Resolves #39

Breaking Changes

No

How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[ ] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

This PR is included in version 2.2.1 :tada:


this doc seems a little bit out of date and there are some mistakes

[Enter feedback here]
I am one of the support engineers, one of our cx raised ticket to this doc and we find there are some mistakes in this section

My Microsoft Alias: leisun

I would like to give some feedback that I found some mistakes in this doc. It would be good if you verify and correct them.

(1) Under “consumer. properties” file  (it looks like we do not use old consumer API, so the zk host is not supported now)
     Change from
      “zookeeper.connect=PRIMARY_ZKHOSTS”   to
       “bootstrap.servers=PRIMARY_BROKERHOSTS”

(2) Under “Create topics” section
     Change from
    “export PRIMARY_ZKHOSTS='ZOOKEEPER_IP_ADDRESS1:2181, ZOOKEEPER_IP_ADDRESS2:2181, ZOOKEEPER_IP_ADDRESS3:2181'” to
     “export PRIMARY_ZKHOSTS='ZOOKEEPER_IP_ADDRESS1:2181,ZOOKEEPER_IP_ADDRESS2:2181,ZOOKEEPER_IP_ADDRESS3:2181'”
     (remove the “space” between IP address)

(3)Under “Start MirrorMaker” section
   Change from
  “/usr/hdp/current/kafka-broker/bin/kafka-console-producer.sh --broker-list $SOURCE_BROKERHOSTS --topic testtopic”   to
   “/usr/hdp/current/kafka-broker/bin/kafka-console-producer.sh --broker-list $PRIMARY_BROKERHOSTS --topic testtopic”

(4)Under “Start MirrorMaker” section
  Change from
  “/usr/hdp/current/kafka-broker/bin/kafka-console-consumer.sh --bootstrap-server $SECONDARY_ZKHOSTS --topic testtopic --from-beginning”    to
  “/usr/hdp/current/kafka-broker/bin/kafka-console-consumer.sh --bootstrap-server $SECONDARY_BROKERHOSTS --topic testtopic --from-beginning”

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 59e5fa6f-ff5f-7c88-99f1-295896a5354f
Version Independent ID: 7d27c2a6-6b75-b3c1-c3db-54ce1523b2f9
Content: Mirror Apache Kafka topics - Azure HDInsight
Content Source: articles/hdinsight/kafka/apache-kafka-mirroring.md
Service: hdinsight
GitHub Login: @JasonWHowell
Microsoft Alias: jasonh

@sunscorch  Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Thanks @sunscorch it would be helpful if you could click edit on the top of the docs page and make the changes. Thanks so much, Jason

@sunscorch, We are working on Kafka MirrorMaker 2.0 Guide, which will take care of the issues reported. The new doc which will be available in next week.

@sunscorch
We have come up with a new page for Kafka Mirror Maker 2. Hope this resolves your concern.
https://docs.microsoft.com/en-us/azure/hdinsight/kafka/kafka-mirrormaker-2-0-guide

As this is covered in the new Kafka Mirror Maker 2 page as provided above, closing this issue.
#please-close


How does one plug in a binary serializer?

I have created a binary Serializer based on the (somewhat out of date) info in the Index documentation.
I created a jar with the serializer - and put it (and its dependancies) in the server lib directory.
I have this code(scala) to test the index:
OBinarySerializerFactory.getInstance().registerSerializer(new CFISerializer(), null);
val index = db.getMetadata.getIndexManager.createIndex("cfi","UNIQUE",new ORuntimeKeyIndexDefinitionCFI,null,null,null)
But when I try to use it I get this:
com.orientechnologies.orient.core.exception.OConfigurationException: Runtime index definition cannot find binary serializer with id=100. Assure to plug custom serializer into the server.
at com.orientechnologies.orient.core.index.ORuntimeKeyIndexDefinition.(ORuntimeKeyIndexDefinition.java:48)
at com.orientechnologies.orient.core.sql.OCommandExecutorSQLCreateIndex.execute(OCommandExecutorSQLCreateIndex.java:255)
...
I am afraid I am stumped... How do I "plug it into the server?"
Sorry If I missed something in the docs - but I have spent a few hours scratching my head.
Thanks!
Russ

hi @russwyte,
It seems all right, the only thing i don't understand is when you register that plugin, to work correctly the plugin should  be loaded at the moment that the database is open.
Can You tell me how do you deploy the plugin?

@tglman I am not confident I know the correct way to do that. Where can I find out how?

here is a guide on the plugins coding: http://orientdb.com/docs/last/Extend-Server.html which case did you use?

Closed because no answer after a while. In case please comment or reopen this.


feat(docs/application): :sparkles: Add Java Records concept

Closes: #164
I've signed the CLA

Sorry setters are obviously not possible due to their immutable design. Getters are important though!

Done. Currently, the changes don't show up for me in the PR view, but when you look at the branch, they're already there. Probably some bug with GitHub. I could imagine that they already show up when you see this.


Remove the uses of context receiver

Context receiver seems to be deprecated, so we need to remove them.
reference -> https://github.com/JetBrains/kotlin/commit/af91a814eda12160005464276fbbc7e0e2d873fd

Context receivers are now deprecated and renamed as context parameters.
The migration is not quite hard. We can do it in a small effort.
The code with context receivers like below:
context(MyContext)
fun hoge()

can be replaced with:
context(_: MyContext)
fun hoge()

_ means that we implicitly use the context parameter. Like before, we can access member properties or member functions without specifying the name.
The one thing we need to care about is, we can't have class-level context parameters.
For example, we can't write like:
context(_: MyContext)
class A

If we want to do this, we have to mark the factory function for the class with context like below:
class A(val context: MyContext) {
    companion object {
        context(_: MyContext)
        operator fun invoke() = A(context)
    }
}


Don't convert mixed glyph values to ints.

Doing this conversion visibly changes the appearance of some composite glyphs, since e.g. a component scale of 1.6 would be rounded to 2. What doesn't make sense is why this was working with FontLab....

lgtm.


PyTennessee 2015 Post

Starting my PyTennessee post off right by doing it in the open. Not ready for merge, would like to kick this out by Wednesday.

:shipit:


[Feature Request] File picker with scrollbar

Is your feature request related to a problem? Please describe.
When selecting multiple files at once, the file list exceeds the screen and cannot be selected anymore
Describe the solution you'd like
A scrollbar would be necessary.

Done! Will be part of the next release!


Refactor oauth2/oidc metadata generation

Define a set of metadata providers which describe info relevant to a given specification, and then use the providers to build both

.well-known/oauth-authorization-server
.well-known/openid-configuration

close with #457


docs: deprecate native-routing-cidr from v1.10

Given that the change to deprecate the native-routing-cidr option in
favor of ipv4-native-routing-cidr was supposed to be backported to
v1.10, update the docs to move the deprecation notice under the v1.10
section.

Change looks good, but is it OK to deprecate an option retrospectively after a major version has been released?

I'd say it's OK as this change is not going to break any existing setup

Will it trigger warnings if the old option name is used?

Will it trigger warnings if the old option name is used?

Yes:
vagrant@k8s1:~$ sudo journalctl -u cilium | grep native
Jun 29 15:19:44 k8s1 cilium-agent[8189]: Flag --native-routing-cidr has been deprecated, This option will be removed in v1.12


GH-1611 remove unnecessary fat jar build from runtime pom

GitHub issue resolved: #1611
Briefly describe the changes proposed in this PR:

rdf4j-runtime is now a pom-packaged dependency
removed shaded jar build process (I still think that got in there by mistake, it's never intentionally been a shaded jar before)
tested that this behaves as expected by having a local scratchpad app that used rdf4j-runtime as its only dependency.

I'm not sure how to test this myself other than what you did. I can see that there was no jar created for the runtime, which I think is correct. LGTM.
After you've merged this I can try to create a new release 3.0.2 with my script.

I'm not sure how to test this myself other than what you did. I can see that there was no jar created for the runtime, which I think is correct. LGTM.
After you've merged this I can try to create a new release 3.0.2 with my script.

Thanks Havard. Happy for you to do a 3.0.2 release with the script, but, but there's one or two other fixes I'd like to get into this release. Shall we say Thursday? Let's continue this on the dev list.


How to get query params if pretty URLs are used

I have 'enablePrettyUrl' => true, 'showScriptName' => false, and 'news/<type:\w+>' => 'news/index' rule in my urlManager config. While using FPM approach, I can get type param (when accessing /news/some-type URL) by calling Yii::$app->request->get('type'). When using yii\Psr7\web\Request and your bridge, I get nulls on every get() or post() call.
Could you please describe how to get it working?

Hi @maximal,
Take a look at the tests directory, and verify you're using a compatible PSR-7/15 Dispatcher. php-fpm isn't a PSR dispatcher so you'll need to use something like RoadRunner, Diactoros,
A working example is there:
config: https://github.com/charlesportwoodii/yii2-psr7-bridge/blob/master/tests/config/config.php
return [
    // Other flags
    'components' => [
        'request' => [
            'class' => \yii\Psr7\web\Request::class,
        ],
        'response' => [
            'class' => \yii\Psr7\web\Response::class
        ],
        // Other components
    ]
];

https://github.com/charlesportwoodii/yii2-psr7-bridge/blob/master/tests/ApplicationTest.php#L80

I haven't touched this repo in a few years. Last I checked Road Runner 1.8 worked, and there was a MR added for 2.5 that worked as well. Maybe try locking spiral/roadrunner package to one of those instead of their development branch? spiral/roadrunner gets pulled in by this repo already.

I just spun up the yii2-app-basic app using road-runner and it worked. Here's my configuration an the resulting screenshot. I pulled all this from either rr's documentation or the docs from this one.
.rr.yaml
---
http:
  address: "<IP_ADDRESS>:8081"
  pool.debug: true
  middleware: [ "static" ]
  static:
    dir: "./web"
rpc:
  enable: true
server:
  command: ./web/roadrunner
  env:
    YII_ALIAS_WEB: "http://<IP_ADDRESS>:8081"
    YII_ALIAS_WEBROOT: ./web

web/roadrunner
#!/usr/bin/env php
<?php

ini_set('display_errors', 'stderr');

// Set your normal YII_ definitions
defined('YII_DEBUG') or define('YII_DEBUG', true);
// Alternatives set this in your rr.yaml file
//defined('YII_DEBUG') or define('YII_DEBUG', \getenv('YII_DEBUG'));

defined('YII_ENV') or define('YII_ENV', 'dev');
// Alternatives set this in your rr.yaml file
//defined('YII_ENV') or define('YII_ENV', \getenv('YII_ENV'));

require __DIR__ . '/../vendor/autoload.php';
require __DIR__ . '/../vendor/yiisoft/yii2/Yii.php';

$worker = Spiral\RoadRunner\Worker::create();
$psrServerFactory = new Laminas\Diactoros\ServerRequestFactory();
$psrStreamFactory = new Laminas\Diactoros\StreamFactory();
$psrUploadFileFactory = new Laminas\Diactoros\UploadedFileFactory();
$psr7 = new Spiral\RoadRunner\Http\PSR7Worker($worker, $psrServerFactory, $psrStreamFactory, $psrUploadFileFactory);

$config = require __DIR__ . '/../config/web.php';

$application = (new \yii\Psr7\web\Application($config));

// Handle each request in a loop
try {
    while ($request = $psr7->waitRequest()) {
        if (($request instanceof Psr\Http\Message\ServerRequestInterface)) {
            try {
                $response = $application->handle($request);
                $psr7->respond($response);
            } catch (\Throwable $e) {
                $psr7->getWorker()->error((string)$e);
            }

            if ($application->clean()) {
                $psr7->getWorker()->stop();
                return;
            }
        }
    }
} catch (\Throwable $e) {
    $psr7->getWorker()->error((string)$e);
}

composer.json
"require": {
        "php": ">=5.6.0",
        "yidas/yii2-bower-asset": "<IP_ADDRESS>",
        "yiisoft/yii2": "~2.0.14",
        "yiisoft/yii2-bootstrap4": "~2.0.0",
        "yiisoft/yii2-swiftmailer": "~2.0.0 || ~2.1.0",
        "charlesportwoodii/yii2-psr7-bridge": "dev-master",
        "spiral/roadrunner": "^2.9"
    }

web/config.php
[
 'aliases' => [
        '@bower' => '@vendor/yidas/yii2-bower-asset/bower',
        '@npm'   => '@vendor/npm-asset',
    ],
'request' => [
            'class' => \yii\Psr7\web\Request::class,
            'enableCookieValidation' => false,
            'enableCsrfValidation' => false,
            'enableCsrfCookie' => false,
            'cookieValidationKey' => 'foo',
            'parsers' => [
                'application/json' => \yii\web\JsonParser::class,
            ]
        ],
        'response' => [
            'class' => \yii\Psr7\web\Response::class,
            'charset' => 'UTF-8'
        ],
]

This is also the latest spiral/roadrunner.
Double check your configuration

It's pulled in: https://github.com/charlesportwoodii/yii2-psr7-bridge/blob/master/composer.json#L17.
The example implementation uses it. You don't have to use Diactoros you can use any PSR dispatcher you want - the example provided is just as an example implementation.

Tried both implementation. Couldn’t get standard Yii2 paging widgets working, they’re using ->get() actively.

You're using the runner and configuration I provided, correct? The example Dispatcher/rr worker is very tied to laminas/laminas-diactoros so if you aren't using that exact Dispatcher and folder structure you'll need to make your own. The examples are provided for illustrative purposes only, but they do work -
I can also verify that the request object works just fine with both pretty URLs. See the following gif and code blocks. As you can see the URL parameters change and this works just fine with enablePrettyUrls set and the script name hidden.

// SiteController.php
public function actionPager()
    {
        $provider = new \yii\data\ArrayDataProvider([
            'allModels' => [
                [ 1, "test1"<EMAIL_ADDRESS>                [ 2, "test2"<EMAIL_ADDRESS>                [ 3, "test3"<EMAIL_ADDRESS>                [ 4, "test4"<EMAIL_ADDRESS>                [ 5, "test5"<EMAIL_ADDRESS>                [ 6, "test6"<EMAIL_ADDRESS>                [ 7, "test7"<EMAIL_ADDRESS>                [ 8, "test8"<EMAIL_ADDRESS>                [ 9, "test9"<EMAIL_ADDRESS>            ],
            'sort' => [
                'attributes' => ['id', 'username', 'email'],
            ],
            'pagination' => [
                'pageSize' => 10,
            ],
        ]);

        $provider->pagination->pageSize=2;

        return $this->render('pager', [
            'dataProvider' => $provider
        ]);
    }

// views/pager.php
<?php

/** @var yii\web\View $this */

$this->title = 'My Yii Application';
echo \yii\grid\GridView::widget([
    'dataProvider' => $dataProvider,
]);

echo \yii\widgets\LinkPager::widget([
    'pagination' => $dataProvider->pagination,
]);

All yii/psr7/web/Request does is translate the PSR7 request object into the appropriate Yii2 models, so if it's not doing that, then something is wrong with your runner/dispatcher. Have you tried the code I provided?
A few things I recommend you try:

Use the exact .rr.yml, composer.json, and web/roadrunner worker I provided in the previous comment and verify your config is setup correctly.
If you're using a different dispatcher, verify you're pushing the PSR7 object into the psr7/web/Application instance exported by this package.
Try running this on the yii-web-basic application to see how it behaves and it get working there.
Check the raw PSR7 object in Yii2 (Yii::$app->request->getPsr7Request()) and verify that the PSR7 object has your request parameters. If it doesn't then something isn't quite right with your dispatcher.

Try these first, especially just testing this in the yii-web-basic app and verify it meets your needs before trying it in your other app. This is alpha software but it does work I use it rather extensively in several private projects.
Regarding:

This bridge’s Yii::$app->request->get('type') for parameters in URLs returns null. Standard Yii2 Request implementation returns type_name.

I don't think yii\web\Request::get() has ever returned named parameters, I believe the standard way to achieve this is to pass them as action arguments, like so
public function actionIndex($type) {}

maps to
'news/<type:\s+>' => 'admin/news/index'

yii\web\Request::get() is only documented to return $_GET parameters.

I recommend reviewing the Yii2 documentation, and making sure you've actually go thing setup correctly. Sounds like there's other issues you need to address first before getting into this.

Seems like the problem is in yii\widgets\ListView. For some reason I don’t understand it doesn’t initiate its ->pager->pagination property with the params I passed.
Pagination::createUrl:
public function createUrl($page, $pageSize = null, $absolute = false)
{
	$page = (int) $page;
	$pageSize = (int) $pageSize;

	//
	// Here it doesn’t see my URL params
	//

	if (($params = $this->params) === null) {
		$request = Yii::$app->getRequest();
		$params = $request instanceof Request ? $request->getQueryParams() : [];
	}
	if ($page > 0 || $page == 0 && $this->forcePageParam) {
		$params[$this->pageParam] = $page + 1;
	} else {
		unset($params[$this->pageParam]);
	}
	if ($pageSize <= 0) {
		$pageSize = $this->getPageSize();
	}
	if ($pageSize != $this->defaultPageSize) {
		$params[$this->pageSizeParam] = $pageSize;
	} else {
		unset($params[$this->pageSizeParam]);
	}
	$params[0] = $this->route === null ? Yii::$app->controller->getRoute() : $this->route;
	$urlManager = $this->urlManager === null ? Yii::$app->getUrlManager() : $this->urlManager;
	if ($absolute) {
		return $urlManager->createAbsoluteUrl($params);
	}

	return $urlManager->createUrl($params);
}

Investigating further.

Maybe @samdark could take a look?

@maximal Please review #10 and see if it resolves your issue and thanks for your patience. I didn't understand the issue you were reporting as the URL Parameters != Query Parameters but Yii2 does some (imo bad) weird mapping resulting in them being returned with yii\web\Request::getQueryParameters().
I've updated the issue title to make that more clear, but please review the MR to see if it covers your needs.


zh-trans: add cloud-controller-manager.md

zh-trans: add cloud-controller-manager.md

@AdamDang I revoke my suggestion related to the title translation. We should leave cloud-controller-manager as is because it is the name of a binary, however we still should translate Cloud Controller Manager. Sorry for the confusion caused.

/lgtm
defer to @xiangpengzhao for approval

/approve


Build Server URL > Jenkins URL

Instead of using a custom 'Build Server URL' parameter, we can't use the built in 'Jenkins URL'(Manage Jenkins > Jenkins Location)?
related #3

This makes sense.  It would take away the need to configure it and just use the Jenkins fronted URL.

https://github.com/jenkinsci/slack-plugin/blob/2da9e23204d5b561d0499d4c8d9159780a1e1aba/src/main/java/jenkins/plugins/slack/SlackNotifier.java#L62
Seems should be changed there.

I do not actively develop this plugin.  I will maintain pull requests and releases from now on.  Please feel free to contribute this feature!

If you would like to be invited to https://jenkins-slack-plugin.slack.com/ for testing then just ask.


Rename report to qc-report and update qc report to 0.9.0

Description
This PR removes references to "webreport" and renames those to qc report.
The report template is updated to 0.9.0. This includes:

Rename "report" supertitle to "qc report"
Add Reads per UMI figure in sequencing panel
Replace "Median antibody molecules per cell" to "Average  antibody molecules per cell" for consistency
Hide "data missing" placeholder messages for not yet implemented features.

Fixes: EXE-1229,EXE-1230,EXE-1231,EXE-1232
Type of change
Please delete options that are not relevant.

[X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
A new test case has been added for generating a frequency table of the reads per molecule.

[x] tests/report/test_data_collection.py

PR checklist:

[ ] This comment contains a description of changes (with reason).
[ ] My code follows the style guidelines of this project
[ ] Make sure your code lints, is well-formatted and type checks
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If you've added a new stage - have you followed the pipeline CLI conventions in the USAGE guide
[ ] README.md is updated
[ ] If a new tool or package is included, update poetry.lock, the conda recipe and cited it properly
[ ] Include any new authors/contributors
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and documentation and corrected any misspellings
[ ] Usage Documentation is updated
[ ] If you are doing a release, or a significant change to the code, update CHANGELOG.md

I will wait with merging this untill tomorrow to make sure the report template is the exact same version as a the release in pixelator-qc-report.


Set plain text language mode when saving as plain text

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.74.1
OS Version: Windows 11

Steps to Reproduce:

Set the files.defaultLanguage setting to ${activeEditorLanguage}

Open a JavaScript file
Open a new file with the command: workbench.action.files.newUntitledFile
Save the file with a .txt file extension
Notice that the file still has javascript syntax highlighting and is still formatted as a JavaScript file.

If I use the command 'Change Language Mode' and then specify the language mode as a plain text file, the JavaScript wiggly underlines go away and the JavaScript syntax highlighting goes away, but it's annoying to have to do this every time I create a new plain text file.

@TylerLeonhardt thanks! do we now have a way to distinguish an explicit language mode from a programmatic one? I remember we added this for language detection? Wonder if I could maybe apply some of that logic here.

@bpasero this is how we trigger detection for auto-detection:
https://github.com/microsoft/vscode/blob/d97f2c1aebbaa8ee3cf926e2810579c2d007ed86/src/vs/workbench/services/textfile/common/textFileEditorModel.ts#L630-L634
I think the file extension is a good signal IMO

Yeah though I am a little bit nervous to break existing assumptions. There can be equally users that like the current behaviour...

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_smiling_face: This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Parse glpk solution file even if column bounds are empty

Fixes #24 by considering only a minimal length for a column specification. Also gives clearer messages what part of a solution file couldn't be parsed.

All good, thank you !


chore #1030: support access to /api-violations/{externalId}

Closes #1030

Codecov Report

Merging #1031 into master will not change coverage.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master    #1031   +/-   ##
=========================================
  Coverage     85.14%   85.14%
  Complexity     1038     1038
=========================================
  Files           175      175
  Lines          2834     2834
  Branches        474      474
=========================================
  Hits           2413     2413
  Misses          175      175
  Partials        246      246

Impacted Files
Coverage Δ
Complexity Δ

.../zalando/zally/configuration/OAuthConfiguration.kt
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

:+1:

:+1:


Unsolicited incoming messages

Is it possible to pick up on and react to incoming messages? For instance if I start an activity using my remote, Harmony.Net receives the messages and fires events based on them?
I had this implemented in my XMPP app for this, but I'm new to clientwebsocket and not sure how to just listen to incoming messages so I can react to them.
For instance be notified when an activity is started from the remote instead of from Harmony.net

Unfortunately, the WebSocket API doesn't send any messages when an event is started from an outside source.
Instead, the official app requests the current state of the Hub every 5 seconds or so. You can do this through the asynchronous Hub.GetRunningActivity() method, which returns either the active activity or null if there is none running.

That is part of the reason my library does some of the stuff it does with TaskCompletionSource and background listeners (https://github.com/i8beef/HarmonyHub/blob/master/src/HarmonyHub/Client.cs). You might be able to shoehorn that in here with one SPECIFICALLY listening for Activity stuff since its the only real push based thing the Harmony does most people care about... if you wanted something more complete though, something like that might set you up for later if they ever add stuff.
A polling mechanism could work here as a stop gap though.
Just a heads up @jlynch630 in case you're wondering why the sudden interest in your library, Logitech knee-capped XMPP today, and your library seems to be last man standing for functional implementations in the world 😄

Maybe there is something up with ClientWebSocket then, If I connect to the WebSocket from a websocket tester, I definitely get updates from the hub in real time.

@i8beef I actually was starting work on grabbing events from the hub for the Home Automation side (Philips Hue, Nest, Harmony Extender). All those updates were also sent over the XMPP stream and I believe they are on the Web Socket implementation, but I haven't vetted that yet. I'm waiting until I get basic functionality back.

Really? I tried it with just logging packets from the app. Let me try with manually connecting to the websocket and get back to you.

@jlynch630 I am using the WebSocket Test Client chrome extension and just tested again to be sure. For instance when you start or stop an activity, you should get 4 messages giving you the different states (starting to change....changing, then one saying the activity is changed and another saying the state is changed). In my XMPP implementation I actually allow users to fire off events on the 3 states of an activity (and therefore 3 states of powering down)
I unfortunately can't use your library as is because I can't use a .net standard library in my .net project, but I would be more than willing to help you test and get it all working. If we can get it working, I'll likely create a .net 4.5.3 project based off of it (I'm restricted by the plugin framework my HA system uses)

You'll notice there are other messages as well. For instance, I get this message when I turn on one of my Hue Light strips via the Hub:
{"type":"automation.state?notify","data":{"hue-[uniqueid]":{"color":{"mode":"ct","xy":{"y":0.3893,"x":0.4027},"temp":282,"hueSat":{"hue":9104,"sat":45}},"brightness":253,"on":true,"status":0}}}
There are lots of messages like this. I've build up a library and if you break down he APK you'll see there is a good amount listed there (for both sending commands. like starting a sleep timer and receiving like the HA message above)
*I removed a unique id from that string as I'm not sure if it can link back to my hue account somehow

That's a serious ask... but does make a case for a more robust background listener implementation.
How many unique message formats are you aware of like that? Are they all "type" plus an arbitrary "data" parameter? Is it feasible to really Type all those out, or are you looking at a leaky abstraction that takes dynamics or something to support that?

Doing that right now actually!
Let me know if you have any comments on this:
I've decided to start a background loop which constantly listens for messages and raises different events based on the type parameter (which is always present, so far as I am aware).
That being said, it makes it much more difficult to have an await this.ChangeChannel unless using the ever-so-ugly event to async method conversion. However, if you think that you'd prefer
await Hub.ChangeChannel("");
Hub.OnChannelChanged += ...

let me know.

They should be all Typed, but I think I was thinking more of the list of commands you can send The number of ones you can receive seems to be limited. However for the automation.state?notify, the "data" can change significantly depending on the device (Hue, Nest, Zigbee, etc), however his library would only have to fire that there was an automation event and pass on the data object and let the consumer handle identifying it.

@sirmeili The variety is why I avoided receiving automation states in the first place. I'll open a new issue for that though.

@sirmeili The variety is why I avoided receiving automation states in the first place. I'll open a new issue for that though.

I would set that at a low priority for now. I would rather get the basic non HA stuff working first.

That being said, it makes it much more difficult to have an await this.ChangeChannel unless using the ever-so-ugly event to async method conversion. However, if you think that you'd prefer

Yeah, that's actually why I split FireAndForget and RequestResponse off separately and let the caller decide which they want. The trick is the TaskCompletionSource stuff I have in my lib to let the separate listener thread complete the awaited task for the RequestResponse ones. My implementation would have to expanded on to support more "unsolicited" responses than just activity updates though. But I think the pattern would actually serve you pretty well here...
On the other hand, bypassing it and just saying you won't do request / response is way easier than that.

The only time I think I might need request/response is when getting the config. I use this when adding a new hub to my plugin so I can create the devices in my HA software, but I'm sure there is a way around that that I'm not seeing. Otherwise for Activity start/stop, you don't really need it if you set up your event handlers. I normally don't care where the request started from, just that the activity is changing.

The only time I think I might need request/response is when getting the config. I use this when adding a new hub to my plugin so I can create the devices in my HA software, but I'm sure there is a way around that that I'm not seeing. Otherwise for Activity start/stop, you don't really need it if you set up your event handlers. I normally don't care where the request started from, just that the activity is changing.

Now that I think of it, I won't need it on getting the config either. It will require a rework on how I handle that part of my code, but it would likely be better to just build/update devices when a new config is received no matter when.

Pings are handled by the .NET client libary automatically. But yeah I'll just start with switching to events and will update those other methods later if need be.

On the topic of pings and not related to this issue, but is it possible to throw an event if the library loses connection with the hub for any reason (power outage, network issue, etc). I currently do this in the XMPP and use it to kick off a reconnect process.
And is there a place for discussions that are not necessarily "Issues"?

Yes. However, .NET pings anywhere between 30s and 2min so it might be a little delayed, but when it does happen I could start reconnecting and also add an event for it cause why not

I'm going to close this for now unless there are any more issues. It's pretty much implemented


In KnownForBase class, Title is missing

Since Title is missing, I am unable to retrieve the title when sending an actor's name and needing to access what they are 'known_for'.
public abstract class KnownForBase
{
protected KnownForBase();
public string BackdropPath { get; set; }
public List GenreIds { get; set; }
public int Id { get; set; }
public MediaType MediaType { get; set; }
public string OriginalLanguage { get; set; }
public string Overview { get; set; }
public double Popularity { get; set; }
public string PosterPath { get; set; }
public double VoteAverage { get; set; }
public int VoteCount { get; set; }

Please submit a PR. :)

This isn't true, not every KnownFor object should have a title. TV shows have name instead of title, while movies have titles. It's present on the KnownForMovie object, which the library is casting to 😄 .
Is there a specific instance in which you're not receving the casted object? I just tested with GetPersonListAsync, which worked correctly.

If this was simply a misunderstanding this can be closed.


Task providers lists potential tasks from the .vscode-test directory

Issue Type: Bug
Repo

Create a new VS Code extension that uses vscode-test.
 I think excluding .vscode-test is a reasonable default
I'll look into the tsc task provider

Agreed. Nice find.


Make built towers transparent to improve selection

When other tower or item is dragged over the screen make all towers transparent. When player hovers dragged item/tower over some built tower, make it opaque in addition to the selection circle. Same we can make when tower is selected with the left click - all other towers should turn transparent to make selected tower more visible.

Fixed by showing selection outline above any sprite.


Add basic auth for the exposed dashboard

Users will be able to specify the username and password used to login to the dashboard.
Setting the username and password is mandatory when the user chooses to expose the dashboard.
This PR will help #186

Hello @walbertus,
Thanks for you work so far :-)
Let me know once you have a PR state we can test. Should I test the current version already?

Hi @SantoDE
You can test this PR now but you need to specify the password, as there's no mechanism to print generated password yet.

Updates:

Dashboard password is now mandatory

Issues:

Basic auth uses MD5, SHA1, or BCrypt for encryption (here). Helm support some methods related to cryptography (here).
These methods only intersect at htpasswd, which failed on pipeline with error function "htpasswd" not defined.

Do you have any suggestion about this @SantoDE ?

Hello @walbertus,
sorry for answering a little late :)
Hmm, not sure about your issue tough. Maybe it would be an option to use a sha1sum on the provided password and run this through a b64 enc before writing to the secret? Does it make sense?

Hello @SantoDE,
That's a nice idea, user can generate their own credentials using whatever encryption they choose.
Will update this

@walbertus can I give it another test? Did you found a way for the credential "delivery?

Hey @walbertus,
any updates?

Hi @SantoDE sorry I miss your comment
User-generated credentials are the only way I think, do you have any other suggestion?
Will resolve the conflict after work

Hey I kinda think, that userData sounds a little bit misleading as it suggests to provide some data by the user and not user auth related settings. And generated secrets by Helm should also provide an option to be self provided by secret name, since people use GitOps nowdays with sealed secrets or SOPS and really don't like to have those things be gitted as values.

I agree with you @CuBiC3D, at least if I understood correctly. Any ideas how we wanna move that forward?

I would just suggest to rename userData to something like usersString or users (one could probably expect an array here) and create useExistingSecret and secretName values to define those (first one could be omitted since an empty secret name can also be used as a condition).

I agree userData is misleading. users seems more appropriate since Traefik also uses users.
We can also use the useExistingSecret filled with the secret name
WDYT?

Sounds good in theory. Eager to see it :)


linesUpdate sets quantity to 0 when attempting to update a line item above the available inventory

What is the location of your example repository?
Internal
Which package or tool is having this issue?
hydrogen-react
What version of that package or tool are you using?
2023.4.0
What version of Remix are you using?
Using Next.js
Steps to Reproduce

Show all cart lines with useCart() and <CartLineProvider />
Create a quantity increase component using linesUpdate from useCart() for each line item
Test increasing the quantity for line items, going above the available inventory shown in Shopify

Expected Behavior
All cart lines should be allowed to attempt an update with a quantity above the available limit, without resetting the quantity to an impossible scenario (being a line item with a quantity of 0).
Actual Behavior
You should see that for the first line item the results are expected: If you try and update above the available amount, Shopify keeps the cart quantity at the available inventory limit 🥳
If you have more than one line item and try to increase the quantity of any item that's not the first line item, the results are bewildering: when you update above the available amount, Shopify changes the cart quantity to 0 but keeps it as a line item in the useCart() lines property 😕

Thanks @frehner! I'm not 100% positive where the underlying issue lies, but it's starting with the hydrogen-react useCart() hook that returns the linesUpdate method.
If that's simply hitting the storefront API under the hood, then I guess you're right. If not, I'd say it's still scoped to the hydrogen-react implementation.


variables ending in _lpdf interpreted as user defined densities

Summary:
From Ryan Giordano on the stan-users list:
This gives the compiler error PARSER EXPECTED: <probability function argument>:
data {
    real x;
}
parameters {
    real mu;
}
transformed parameters {
    real mu_lpdf;
    mu_lpdf = normal_lpdf(mu | 0, 1);
}

model {
    target += mu_lpdf;
}

Description:
The parser is almost certainly interpreting that as a user-defined probability function and looking for the arguments.  Should be easy to fix.
Reproducible Steps:
See above.
Current Version:
v2.12.0

As an aside, that program should just use _lpd or even _lp because the 'f' is for "function" and mu is not a function.

+1.  I wouldn’t at all be against not allowing variables ending with lpdf or lpmf.
On Nov 23, 2016, at 3:10 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:

As an aside, that program should just use _lpd or even _lp because the 'f' is for "function" and mu is not a function.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Wow--a triple negative!  That doesn't come along every day.

On Nov 23, 2016, at 6:55 PM, Michael Betancourt<EMAIL_ADDRESS>wrote:
+1. I wouldn’t at all be against not allowing variables ending with lpdf or lpmf.
On Nov 23, 2016, at 3:10 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:

As an aside, that program should just use _lpd or even _lp because the 'f' is for "function" and mu is not a function.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/stan-dev/stan/issues/2123#issuecomment-262658409, or mute the thread https://github.com/notifications/unsubscribe-auth/ADOBUSa0bg-NrRDJNPGG7fYAL8NXsmwgks5rBNJsgaJpZM4K6RGD.


Fix typo Update api.go

Description:
This pull request fixes a typo in the documentation of the MulAcc method in the API interface.
Problem
In the documentation, the example for the MulAcc method currently reads:
acopy = MulAcc(acopy, b, c)

This is inconsistent with other examples in the documentation, which explicitly reference the API object using the api. prefix. The omission of api. in this example could confuse readers, particularly those new to the interface, as they might not realize that MulAcc is intended to be invoked as a method of the API object.
Fix
The corrected line is:
acopy = api.MulAcc(acopy, b, c)

Importance

Clarity: Ensures consistency in the documentation, aligning this example with other examples.
Accuracy: Prevents potential confusion for developers interpreting the documentation.
User Experience: Enhances the overall readability and usability of the API documentation.

Please review and merge.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


x.y is not a valid version for toolchain install

took me a while to figure out I need version in x.y.z format.

As with some other features, this will require an index of all releases on the server to make work.
A related issue is that rustup doesn't ever do any fuzzy matching on semver - all versions are exact. This will become an important matter when Rust makes its first stable point release. There will probably be a reasonable expectation that rustup upgrades from X.Y.0 to X.Y.1 automatically.

Also, it's worth noting that the semver spec is a bit murky around x.y being equivalent to x.y.z, so the semver crate doesn't parse it as a version. There's a PR for it, but I haven't decided if I want to merge it or not.

Dupe of #794


Icon request: GOV.UK

URL of the site or app (required): https://www.gov.uk/
Logo resources for the icon (optional, but recommended): https://en.wikipedia.org/wiki/File:Gov.uk_logo.svg

I noticed HMRC was added in #136, but this isn't the same as GOV.UK (the broader login that includes more than just HMRC).

GOV.UK (the broader login that includes more than just HMRC)

Okay, I will replace it with this then if that's the case.

Replacing it makes sense. Would that break anything for existing users of the icon?
I think HMRC used its own login system in the past, but seems to have migrated fully to GOV.UK's login system.
The "sign in" link leads to GOV.UK's login: https://www.gov.uk/log-in-register-hmrc-online-services
And HMRC services are listed under: https://en.wikipedia.org/wiki/GOV.UK_Verify

Would that break anything for existing users of the icon?
It doesn't, the attached SVG of 2FA entry is stored to database and also included in the export JSON.

I decided not to remove HMRC icon but instead move it to "Outdated" category / directory.
https://github.com/aegis-icons/aegis-icons/blob/master/SVG/GOV.UK.svg
❓ How to add icon(s) to Aegis?


Lora extract for Flux finetunes/dreambooth

I'm working on it. Sorry for waiting.

take your time! Thank you for all your work!

+1 dreambooth looks amazing, model is 22gb

hopefully lora will be good.


[Table] Error while using Durable Functions with azurite

Which service(blob, file, queue, table) does this issue concern?
Table
Which version of the Azurite was used?
3.12.0
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
npm
What's the Node.js version?
12.16.3
What problem was encountered?
Couldn't extract path from URL in sub-Request
Steps to reproduce the issue?
Log below
PUT http://<IP_ADDRESS>:10002/queuesdev/funcpcappdevHistory(PartitionKey='2d2c8fe4-d3a6-438f-aa83-382d93ee9569:ca',RowKey='0000000000000000') HTTP/1.1
Accept: application/json;odata=minimalmetadata
Content-Type: application/json
DataServiceVersion: 3.0;

{"EventId":-1,"IsPlayed":false,"_Timestamp":"2021-05-28T19:28:47.3971577Z","_Timestamp@odata.type":"Edm.DateTime","EventType":"OrchestratorStarted","ExecutionId":"796d2897909c4c11aec3768df50feb95"}
 ErrorStack="Error: Couldn't extract path from URL in sub-Request:\n \r\n\r\nPUT http://<IP_ADDRESS>:10002/queuesdev/funcpcappdevHistory(PartitionKey='2d2c8fe4-d3a6-438f-aa83-382d93ee9569:ca',RowKey='0000000000000000') HTTP/1.1\r\nAccept: application/json;odata=minimalmetadata\r\nContent-Type: application/json\r\nDataServiceVersion: 3.0;\r\n\r\n{\"EventId\":-1,\"IsPlayed\":false,\"_Timestamp\":\"2021-05-28T19:28:47.3971577Z\",\"_Timestamp@odata.type\":\"Edm.DateTime\",\"EventType\":\"OrchestratorStarted\",\"ExecutionId\":\"796d2897909c4c11aec3768df50feb95\"}\r\n\n    at C:\\Users\\junior\\AppData\\Local\\Yarn\\Data\\global\\node_modules\\azurite\\dist\\src\\table\\batch\\TableBatchSerialization.js:71:23\n    at Array.map (<anonymous>)\n    at TableBatchSerialization.deserializeBatchRequest (C:\\Users\\junior\\AppData\\Local\\Yarn\\Data\\global\\node_modules\\azurite\\dist\\src\\table\\batch\\TableBatchSerialization.js:57:45)\n    at TableBatchOrchestrator.processBatchRequestAndSerializeResponse (C:\\Users\\junior\\AppData\\Local\\Yarn\\Data\\global\\node_modules\\azurite\\dist\\src\\table\\batch\\TableBatchOrchestrator.js:43:51)\n    at TableHandler.batch (C:\\Users\\junior\\AppData\\Local\\Yarn\\Data\\global\\node_modules\\azurite\\dist\\src\\table\\handlers\\TableHandler.js:400:50)\n    at runMicrotasks (<anonymous>)\n    at processTicksAndRejections (internal/process/task_queues.js:97:5)"

Have you found a mitigation/solution?
No

Sounds like a Batch sub operation deserialization issue, @edwin-huber able to help on this? Thanks!

Thanks, I'll take a look...

Investigating https://github.com/Azure/Azurite/blob/51baf445d1e2532659e16a90505c4ff8a9349ea1/src/table/batch/TableBatchSerialization.ts#L88 I see the hardcoded string devstoreaccount1. In my use case I have multiple accounts set up to mimic my production environment, so that hardcoded string is not part of it.

thanks, I'll update the regex


How Do I know if an Embedded document is loaded or newly added ?

Hi
public class Order
{
    public List<OrderDetail> Details { get; set; } = new List<OrderDetail>();
}

// has no Id - is embedded document
public class OrderDetail
{
    public int Quantity { get; set; }
    [DbRef("products")]
    public Product { get; set; }
}

public class Product
{
    public int ProductId { get; set; }
    public string ProductName { get; set; }
    public decimal Price { get; set; }
}

After loading an Order with its embedded documents, I want to change an OrderDetail via UI
how do I know if the OrderDetail in question is loaded or newly added as long as there is not an Id?
Thanks for the great project :)

When you save one Order, there is difference between loaded/newly OrderDetail.. all items will be saved inside datafile. If you want track this, you must implement some check property using [BsonIgnore] in OrderDetail

@mbdavid  I am getting confused, can you explain more? can give me a pseudo code ?
Thanks for your time

I've figured it out how to do this, add not mapping property : IsLoaded to OrderDetail, when loading an order I mark  all details as loaded, when saving I know if it newly or loaded document, is what you mean Sir.
Thanks in advance

If you want detect if an instance was loaded from database and not fill by your code, you can add a property to control this:
public class Order
{
    public int Id { get; set; }
    public List<OrderDetail> Details { get; set; } = new List<OrderDetail>();

    [BsonIgnore]
    public bool IsNew { get; set; }

    public static Order Create()
    {
        return new Order { IsNew = true };
    }
}

Now, you can just use Order.Create() static method to create new instances... and IsNew will be true. When loaded from database, this property remains false (BsonIgnore will mark as not serialized)

@mbdavid there is a misunderstanding, I apologize if I did not well explain my problem, I do not mean the Main document(Order) but the I mean subdocuments(OrderDetail), how do Know if they are loaded or newly added
thanks Sir for your time

Hi @toumir, the ideia are the same. You can just add this IsNew property  in OrderDetail and use a static Create method to create instances of OrderDetail with IsNew = true. When engine load List of OrderDetail, each instance will have IsNew = false because was ignored when serialize.

@mbdavid  yes I figure it out thank you Sir :)


Add nanobind_bazel v2.4.0

Mirrors nanobind v2.4.0. Note that no nanobind v2.3.0 was released due to a mistake in the release process.

I guess Bazel 6.x can go from the presubmit, but Bazel 7.x now also fails (missing --std="c++17"), which is confusing to me.

Decided to set the test target in the presubmit to the newly exposed @nanobind_bazel//:nanobind, which was the "headline feature" for v2.4.0.
(For clarification: This is an alias pointing to the @nanobind repo extension - it was there since the beginning, because it's the whole point of nanobind_bazel, but it was not accessible by downstream users until this release.)

Do you have an idea on how to inject C++17 as a standard into the Bazel 7.x job? I don't have it in my copts / cxxopts explicitly.