SPFx - Integrate bootstrap with WebPart

What type of issue is this?
Question
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Target SharePoint environment
SharePoint Online
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[X] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version
SPFx version
Node.js version
etc

Issue description
Hi,
Currently I am struggling integrating the bootstrap framework into my custom webpart.
I have watched tons of tutorials online but all resulting in the same error.
My Code Repo:
https://github.com/kevinvugts/test-spfx
When creating a style.module.scss  file containing my default bootstrap variables overwrites the gulp builder fails with the following error:
 exited with code 2
Error - [webpack] 'dist':
./lib/webparts/webpartDgdmHelloworld/components/Theme.module.css (./node_modules/css-loader/dist/cjs.js!./node_modules/postcss-loader/src??postcss!./lib/webparts/webpartDgdmHelloworld/components/Theme.module.css)
Module build failed (from ./node_modules/postcss-loader/src/index.js):
SyntaxError
(40:4) Missed semicolon
 @ ./lib/webparts/webpartDgdmHelloworld/components/Theme.module.css 1:14-155
 @ ./lib/webparts/webpartDgdmHelloworld/components/Theme.module.scss.js
 @ ./lib/webparts/webpartDgdmHelloworld/components/WebpartDgdmHelloworld.js
 @ ./lib/webparts/webpartDgdmHelloworld/WebpartDgdmHelloworldWebPart.js

The weird part is that there is no missing semicolon at all. When disabling the last line in the scss file the error disappears but bootstrap will not be loaded.
Of course I am able to load bootstrap with the     SPComponentLoader.loadCss("https://stackpath.bootstrapcdn.com/bootstrap/4.4.1/css/bootstrap.min.css");
However this is not desired since I want to overwrite certain variables of bootstrap before it's loaded.
My question
How can I integrate the bootstrap framework & React-bootstrap library with my SPFX webpart?
When importing components from React-bootstrap library I am getting the following errors:
Error - [tsc] node_modules/@popperjs/core/lib/createPopper.d.ts(1,13): error TS1005: '=' expected.
Error - [tsc] node_modules/@popperjs/core/lib/createPopper.d.ts(1,73): error TS1005: ';' expected.

Active GitHub thread related to this question: https://stackoverflow.com/questions/67924028/how-to-add-bootstrap-to-sharepoint-spfx-webpart

Hi @kevinvugts - I did some testing and I seem to have found a way around. The problem with popper is that it does not provide a types library.
Basically, while SPFx can include external resources using the SPComponentLoader  as outlined here, it also requires adding the respective types library as a dev dependency.
Both bootstrap and jQuery provide type libraries @types/bootstrap @types/jquery which can be installed as dev deppendencies while popper does not. To get around that, I installed popper itself as a dev dependency.
Hope this helps.
Bass

Hi @kevinvugts - I did some testing and I seem to have found a way around. The problem with popper is that it does not provide a types library.
Basically, while SPFx can include external resources using the SPComponentLoader as outlined here, it also requires adding the respective types library as a dev dependency.
Both bootstrap and jQuery provide type libraries @types/bootstrap @types/jquery which can be installed as dev deppendencies while popper does not. To get around that, I installed popper itself as a dev dependency.
Hope this helps.
Bass

Hi Bass,
Could you provide me with some sample code and the way you configured Popper into SPFX?
Thanks a lot already for your input!
Best,
Kevin

Hey Kevin,
Here are the detailed steps:

Install the following libraries in your project (note the save-dev as they are dev dependencies)

npm i @types/bootstrap @types/jquery @popperjs/core --save-dev

Update config\config.json to include jQuery and bootstrap as external libraries

"externals": {
  "jquery": {
    "path": "node_modules/jquery/dist/jquery.min.js",
    "globalName": "jQuery"
  },
  "bootstrap": {
    "path": "node_modules/bootstrap/dist/js/bootstrap.min.js",
    "globalName": "bootstrap"
  }
},

Add an init function to the Parent WebPart (the one that extends BaseClientSideWebPart)

protected onInit(): Promise<void> {
  SPComponentLoader.loadCss(
    'https://stackpath.bootstrapcdn.com/bootstrap/4.1.0/css/bootstrap.min.css'
  );
  SPComponentLoader.loadScript(
    'https://code.jquery.com/jquery-3.3.1.slim.min.js'
  );
  SPComponentLoader.loadScript(
    'https://cdnjs.cloudflare.com/ajax/libs/popper.js/1.14.0/umd/popper.min.js'
  );
  SPComponentLoader.loadScript(
    'https://stackpath.bootstrapcdn.com/bootstrap/4.1.0/js/bootstrap.min.js'
  );

  return super.onInit();
}

Once the setup is done, elements should be able to apply bootstrap - I used the below render function to test
return (

      This is a primary alert—check it out!

      This is a secondary alert—check it out!

      This is a success alert—check it out!

      This is a danger alert—check it out!

      This is a warning alert—check it out!

      This is a info alert—check it out!

      This is a light alert—check it out!

      This is a dark alert—check it out!

);

Hey Kevin,
Here are the detailed steps:

Install the following libraries in your project (note the save-dev as they are dev dependencies)

npm i @types/bootstrap @types/jquery @popperjs/core --save-dev

Update config\config.json to include jQuery and bootstrap as external libraries

"externals": {
  "jquery": {
    "path": "node_modules/jquery/dist/jquery.min.js",
    "globalName": "jQuery"
  },
  "bootstrap": {
    "path": "node_modules/bootstrap/dist/js/bootstrap.min.js",
    "globalName": "bootstrap"
  }
},

Add an init function to the Parent WebPart (the one that extends BaseClientSideWebPart)

protected onInit(): Promise<void> {
  SPComponentLoader.loadCss(
    'https://stackpath.bootstrapcdn.com/bootstrap/4.1.0/css/bootstrap.min.css'
  );
  SPComponentLoader.loadScript(
    'https://code.jquery.com/jquery-3.3.1.slim.min.js'
  );
  SPComponentLoader.loadScript(
    'https://cdnjs.cloudflare.com/ajax/libs/popper.js/1.14.0/umd/popper.min.js'
  );
  SPComponentLoader.loadScript(
    'https://stackpath.bootstrapcdn.com/bootstrap/4.1.0/js/bootstrap.min.js'
  );

  return super.onInit();
}

Once the setup is done, elements should be able to apply bootstrap - I used the below render function to test
return (

      This is a primary alert—check it out!

      This is a secondary alert—check it out!

      This is a success alert—check it out!

      This is a danger alert—check it out!

      This is a warning alert—check it out!

      This is a info alert—check it out!

      This is a light alert—check it out!

      This is a dark alert—check it out!

);

Reading through my snippet, I can see I loaded popper.min.js but actually installed @popperjs/core - might be best to align these two to the same version. I also hear that unpkg is the place to go so it might be good to update the URLs in the SPComponentLoader section to use it

Thanks for your reply. However I already fixed the issue by these instructions. Thanks anyways!
Closing off....


iOS crash right after a successful subscription

Please use Discussion board if you want get some help out of it. Please report issue if there is a bug.

Version of react-native-iap
6.3.0
Version of react-native
0.63.2
Platforms you faced the error (IOS or Android or both?)
iOS
Expected behavior
Get a SubscriptionPurchase after calling RNIap.requestSubscription or at least an error.
Actual behavior
A native crash visible in Xcode.
Tested environment (Emulator? Real Device?)
Tested on both
Steps to reproduce the behavior
const [subscription]: RNIap.Subscription[] = await RNIap.getSubscriptions([
    productId,
  ]);
console.log(subscription); // OK
const purchase = await RNIap.requestSubscription(subscription.productId);

Xcode logs about the crash:
2021-07-07 15:33:30.876687+0200 MyApp[1059:199421]

 Purchase Successful !!

.
2021-07-07 15:33:30.876724+0200 MyApp[1059:199421]

 requestReceiptDataWithBlock with force refresh: NO

.
2021-07-07 15:33:30.877045+0200 MyApp[1059:199421] *** Terminating app due to uncaught exception 'NSRangeException', reason: '*** -[__NSArrayI_Transfer objectAtIndex:]: index 0 beyond bounds for empty array'
*** First throw call stack:
(0x19d226754 0x1b1ced7a8 0x19d298a18 0x19d23e274 0x102768e00 0x106cfc0b4 0x106cfdde0 0x106d0e0a8 0x19d1a0398 0x19d19a270 0x19d199360 0x1b47d7734 0x19fc14584 0x19fc19df4 0x102770efc 0x19ce55cf8)
libc++abi: terminating with uncaught exception of type NSException
*** Terminating app due to uncaught exception 'NSRangeException', reason: '*** -[__NSArrayI_Transfer objectAtIndex:]: index 0 beyond bounds for empty array'
terminating with uncaught exception of type NSException

Resolved, it was a just a conflict with an old in app purchase library used on my project.

Resolved, it was a just a conflict with an old in app purchase library used on my project.

How did you solve it?

I just removed the other (and unused) library, previously installed by another developer.


feat(#571): TestResourcesPropertiesFactory supports user defined meta annotations

See #571

If direct annotation on test class is not found, then look for meta annotations.
Fixes typo on instantiateProvider.

Thanks to you for the quick review! Could you please label it as type: improvement?


chore: use Rollup instead of Parcel

Changelog
Summary
Per our last dev discussion, we are going to use Rollup instead of Parcel for JavaScript bundling. This PR setups up Rollup to transpile and minify files as needed to distribute CJS, ESM, and UMD files.  This would be a breaking change as we have changed the file locations.  Blocked by PR #13.
Added

Rollup with Babel and Terser plugins
Rollup configuration to generate the different file formats (only transpiles CJS and UMD formats)

Updated

package.json to reference the 3 different formats
moved [root]/index.js to src/index.js since we have Rollup creating a ESM file under the dist directory

Removed

Parcel
Removed non-dist files from NPM (use files properties)

Code review

[x] The code must be reviewed by at least one Maintainer

Code quality
Manual testing to be completed by reviewers

[ ] Documentation is understandable

Compliance requirements

[x] The pull request must have a link to GitHub issue

Related to https://github.ibm.com/Watson-Health/design-library/issues/5914
Definition of done

The code has been reviewed by the Maintainer
The code has passed quality checks

85% code coverage
Passed the following tests:

Unit

Documents are updated

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Text is getting stretched/compressed when width is changed.

Hello, I am trying to change the text width in the _init() state (I tried other states as well), but the text is stretched/compressed based if the new width is bigger/smaller than the one in the template.
Here a snippet coming from an example in your docs.
`class TextDemo extends lng.Application {
static _template() {
return {
x: 50,
y: 50,
Text: {
w:100,
text: {
fontSize: 36,
textAlign: 'left',
maxLines: 2,
text: 'Text a little too long so this part won't be visible',
wordWrap: false,
textColor: 0xffff00ff,
}
}
}
}
_init() {
this._text = this.tag('Text');
this._text.patch({
  w: 500
});

}
}
const options = {stage: {w: window.innerWidth, h: window.innerHeight, useImageWorker: false}};
const app = new TextDemo(options);
document.body.appendChild(app.stage.getCanvas());`

Then I saw this fixed issue: https://github.com/WebPlatformForEmbedded/Lightning/issues/24
So, Is it an actual issue or am I doing something wrong?
Thanks

Hi @achille1789,
Can you tell us more about your use case? Why do you want to change the width of a text element?
Whether this is a bug or not is actually not that straightforward to answer. Text is actually a texture of an element so in many cases it should be treated as such (especially when element is to be animated in various ways like rotation, grow/shrink etc.). However, there are some cases where it makes more sense for it to behave like a sub element rather than a texture.
Best regards

Hi @g-zachar , thank you for your quick response.
Basically, I am creating a component that has to be inserted in a library so it can be shared on different projects. In particular it will be a menu item that it will be inserted in a menu list.
I would like to set a default width in the component and give the possibility to change its width when imported based on the specific requirements.
Currently I am able to change all other props, when I import it, except for the width.
Thanks in advance

Hi @achille1789,
When designing a list component, I would go with something like this:
https://jsfiddle.net/jhczsbxp/
Let me know if you have any further questions.
Best regards

@achille1789
Alright, this definitely can be considered a bug. You can use this.tag('Label').w = w => 200 as a workaround for now.
I've updated my example:
https://jsfiddle.net/e3h95wzk/1/
Best regards

Great!
Looking forward to get the actual fix.
Thanks a lot again!
Best regards


Replace BitmapInfo with IDrawableImage

Not sure about IDrawableImage as name.
The if svg or bitmap logic should be replaced with something better. Perhaps there should be logic to automatically select an ImageRenderer.

If you want to circumvent the if's and the interface, then you could use a SKDrawable. That is a Skia object, that knows how to draw itself on a canvas. It has 3 main functions that must be implemented: OnGetBounds(), OnDraw() and OnSnapshot(). OnSnapshot() is normally a call of OnDraw(). So this object provides all functions you implement in IDrawableImage or need to implement.
If you want to draw a SKDrawable, you could use SKCanvas.DrawDrawable(), which calls behind the scenes the Draw function of the object.


Log rotation?

Does the base vaultwarden image do log rotation? Or is there log rotation otherwise set up? I set it up on my fork of the old setup, and slowly trying to PR my changes into this new setup so I can get caught up with your latest changes.

This is what I found on the vaultwarden wiki about logrotate: https://github.com/dani-garcia/vaultwarden/wiki/Logrotate-example, however the logging entry says they only log to stdout by default.
Docker logs the stdout|stederr from the containers to json files, but docker compose can set the log driver and log rotation:
https://medium.com/free-code-camp/how-to-setup-log-rotation-for-a-docker-container-a508093912b2


Added "Resolution" section

Added "Resolution" section with the basic analysis step.

@tedhudek - Another PR that is ready to go live when you have a moment.

@LuigiBruno - This PR looks fine, although this is not a frequent stop code.

OK. Thank you @DOMARS.


Please help a new Julia user with an error in the 00-introduction notebook.

I’m and new user of Julia and I am very interested in the “Turing Tutorials”.
I have installed Julia 1.9.0 and I have followed the instructions on loading the tutorial notebooks.
After adding the necessary packages. I then did a “Restart & Run All” under the Kernel menu of the “00-introduction” notebook.  The execution got down to cell 11 and stopped with an error.

I don't understand this error mesage.
Any  suggestions will be greatly appreciated.
Charles Brauer
<EMAIL_ADDRESS>

Hi Charles
Did you come right with this? My first thought is that Turing.jl hasn't been loaded when this cell is run. If not, could you please provide the code you ran up until this point and I can try help debug?
Thanks,
Patrick

Hi Patrick,
As you requested, I have attached the notebook.
Charles
From: Patrick Moehrke @.>
Sent: Monday, June 12, 2023 12:15 PM
To: TuringLang/TuringTutorials @.>
Cc: Charles Brauer @.>; Author @.>
Subject: Re: [TuringLang/TuringTutorials] Please help a new Julia user with an error in the 00-introduction notebook. (Issue #390)
Hi Charles
Did you come right with this? My first thought is that Turing.jl hasn't been loaded when this cell is run. If not, could you please provide the code you ran up until this point and I can try help debug?
Thanks,
Patrick
—
Reply to this email directly, view it on GitHub https://github.com/TuringLang/TuringTutorials/issues/390#issuecomment-1587932062 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKBS4VXV7YSMOQGTDTC3GLXK5TENANCNFSM6AAAAAAYIAO72A .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AAKBS4SN4GBV5PY5HRO77BTXK5TENA5CNFSM6AAAAAAYIAO72CWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTS6UXVZ4.gif Message ID: @.*** @.***> >

Hi @CBrauer
I don't believe the notebook is attached as I think the GitHub only retains the body of the email when replying to an Issue. Could you please add the code contained in your notebook in plain text that I can test it on my side?
Thanks.
Patrick

Hi Charles
Thank you, I had missed that it was a clean copy you're using. Unfortunately I don't receive your attatchments through GitHub Issues, but screenshots work. However, I suspect Turing isn't being loaded properly. Alternatively, Jupyter Notebooks isn't playing nice with macros.
Here is a minimum viable product you can test:
using Turing
using MCMCChains
using Distributions
using Random
using StatsPlots

# Create some coin flip data
N = 25
data = rand(Bernoulli(0.5), N);

# Check the average number of Heads
mean(data)

# Define our model and priors
@model function coin_flip(y)
    n = length(y)
    p ~ Beta(1, 1)
    y ~ filldist(Bernoulli(p), n)
    return Nothing
end

# Control how many samples we take
n_samples = 1_000
n_chains = 4

# Sample using NUTS, given our observations
chain = sample(coin_flip(data), NUTS(), MCMCThreads(), n_samples, n_chains)

# Plot the prior after sampling
plot(chain)

# Produce a summary table of our posteriors containing its mean, standard deviation, etc.
describe(chain)

Hope this helps!
Regards
Patrick

Hi Patrick,
I did as you suggested, and I put the code in separate cells.  Here is a screen capture of my running notebook:

Your code crashed the notebook.  The execution never gets past the second cell.
You said that I should place the code in separate cells to see the plot output, and describe output.  What if I executed the "test.jl" script in the Julia REPL? Would you expect the REPL to give the plot and describe print correctly?  Well, it doesn't.
Here is what happens when I execute your code in the REPL:
julia> include("test.jl")
┌ Warning: Only a single thread available: MCMC chains are not sampled in parallel
└ @ AbstractMCMC C:\Julia-1.9.1\bin\packages\AbstractMCMC\bE6VB\src\sample.jl:296

┌ Info: Found initial step size
└   ϵ = 1.6
┌ Info: Found initial step size
└   ϵ = 0.8
┌ Info: Found initial step size
└   ϵ = 1.6
┌ Info: Found initial step size
└   ϵ = 0.8
Sampling (1 threads) 100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| Time: 0:00:12
done.
julia>

As a new user to Julia, I'm getting rather discouraged.
Charles


make cache_dir configurable

In our installation, /var is mounted with noexec, so I had to change the cache directory.
Default stays the same, but with this PR it's configurable.

We would also be interested in having this applied.

Thanks for the PR @rndmh3ro!


samples: drivers: led_pwm: Update sample to remove usage of DT_INST

Update sample to use DT_COMPAT_GET_ANY_STATUS_OKAY in order to remove
usage of DT_INST.
Signed-off-by: Benjamin Björnsson<EMAIL_ADDRESS>

What about device_get_binding?

Good point, I've updated.

@Mani-Sadhasivam when looking at this sample I get the feeling that the led API should be extended to support getting the number of leds? All API functions except write_channels takes the led argument but there's no explicit way of getting what range this argument supports.

You need to update sample.yaml to match the output change you made, this will fix CI failure:
diff --git a/samples/drivers/led_pwm/sample.yaml b/samples/drivers/led_pwm/sample.yaml
index c6beedfa8d..fe9a3845ff 100644
--- a/samples/drivers/led_pwm/sample.yaml
+++ b/samples/drivers/led_pwm/sample.yaml
@@ -13,7 +13,6 @@ tests:
       type: multi_line
       ordered: true
       regex:
-        - "Found device \\S+"
         - "Testing LED \\d+"
         - "Turned on"
         - "Turned off"

Thanks!


[Resources] Import APIs for subscriptions.json file

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify

the change is to add APIs that already exist in an existing API version

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.

The service changes are already in production

When you expect to publish swagger? Please provide date, or month to public if date is not available yet.

Sooner the better

If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[x] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@Tiano2017 This requires an ARM review, and you must fix avocado failures

Hi @markcowl, the Avocado failures seem to be false alarm. The examples are referenced from the new subscriptions.json file. Besides that, I'll work on getting ARM review.

Hi @markcowl, can you help take another look? I got the ARM API sign off. The Avocado failures seem to be false alarm. The examples are being referenced from the new subscriptions.json file.

@markcowl I updated the x-ms-examples, but still getting the avocado failures..

@markcowl  Also, regarding the error on breaking change. The property is already added in service side a few versions back, so what would be the right process here. Should I still create a new swagger version to include that change? That doesn't seem to be meaningful.


Required is missing

Ideally need to pass it through from the schema.

try clicking submit or tabbing off the field

You're missing the point. TextField has a required property, which shows a * in the UX to the user to indicate that the field is required.
Unfortunately, yup doesn't make it easy to get this information, but I can see it deep in the tree.

Oh, you want a custom label. That falls outside of this library. Not everyone wants the star. Material ui does not do this out of the box either

Nope, I don't want a custom label. I want to set the required property on a MUI TextField.

Hmmm. there is nothing preventing it from being passed though.
Seems to be working as expected in the sandbox?
https://codesandbox.io/s/915qlr56rp

How is it working as expected in the sandbox? Doesn't look like the screenshot above at all.

So the placeholder is working
Can you provide a codesandbox example?

did you add
        <Field
          name="email"
          type="email"
          label="Email"
          required={true}
          component={UppercasingTextField}
        />```

You're not understanding, get it from the schema. Why should I have to declare it twice?

That is not a feature in formik as far I am aware. And I don’t think it belongs in the library

Of course formik doesn't know anything about MUI. That is why you created your library.
However, formik definitely knows about Yup, as it is a tightly integrated dependency for handling validation.
So you take the schema from yup, run schema.describe() on it and then use that output to populate the required property on the <TextField/>.
But that said, forget about it. You don't seem interested in fixing your library. I guess I'll roll my own.


[prometheus-blackbox-exporter] Fixing minor syntax error

What this PR does / why we need it:
When setting resources in another values file, this warning is seen:
2018/10/23 22:19:42 warning: skipped value for resources: Not a table.

Defaulting to {} removes the warning.
Checklist

[x] DCO signed
[x] Chart Version bumped

Signed-off-by: Dave Henderson<EMAIL_ADDRESS>

/ok-to-test
/assign

/lgtm

/ok-to-test
/lgtm


sample code not working on IE 11

HI,
I copy the entire index.html from https://raw.githubusercontent.com/aratcliffe/Leaflet.contextmenu/master/examples/index.html
then run node app.js
The map shows up with markers. However, Upon right click, the default Windows menu shows up instead of popup contextmenu. Same issue even I changed to 1.2.0 leaflet.
Did I miss anything?
Thanks
Henry

It looks like you forgot to copy it's dependencies. Maybe check our console and see if there are any errors.


allow multiple pathways in VideoPlaylist

[x] design layout
[ ] implement new layout in component
[ ] refactor to make layout configurable via props

first layout design:


Class Not Found

Hello
I am trying to use your library but get the error
Lcobucci\JWT\Signer\Rsa\Sha256' not found in client.php:6 S
I have tried playing around with the folder structure but no luck
The folder structure downloaded via composer does not have src in it
Please can this be fixed asap?
Cheers
Paul

@paulinuk What version are you using?

The version that installs with composer
I don’t see a version number
I am basically looking for an example that I can use for authenticating with an internal identity server using OAuth open id connect with Implicity flow

Can you inspect your composer.lock file and report back the version of lcobucci/jwt that is installed?

V.0.2.0
I need something that will exchange an id token for an access token
Does your library support that when in implicit mode?

Sorry it’s 3.3.1 was looking at the wrong one
My php is version 5.4 which I can’t update

5.4.16

@paulinuk that version of the JWT library indeed contains the class in question.
https://github.com/lcobucci/jwt/tree/3.3/src/Signer/Rsa
Can you post the full stack trace of the issue you are having?

Full disclosure that I have no tried this library on PHP 5.4.
According to the documentation here https://github.com/thephpleague/oauth2-client/tree/2.0.0 the minimum version supported for the oauth2 client is PHP 5.5. This library is simply an extension of the oauth2-client from thephpleague. Thus I don't believe this will work for your specific use case.


Optimize Animator.RenderFrameAsync, also resolve issues #186

This fix will use more memory per frame, but extremely faster, especially when rendering large gif files.

Hi @chglove,
Thanks for your PR! I really like the change. It's much simpler and easier to understand, and much more efficient. Nice work!
Let me try it, and if I don't see any issue, I'll merge and release an updated version.

This fix will use more memory per frame

Does it, though?
I think there's the same amount of data in _cachedFrameBytes, but a much smaller number of byte arrays (1 per frame instead of 1 per line), so it probably uses slightly less memory, and also puts less pressure on the garbage collector.
Anyway, this seems to work just fine.
Thanks again!


rsync exludes files that it should not

In the rsync command to send files to the container there is unexpected behavior.
rsync -a --delete-excluded --delete --exclude-from=<(git ls-files -o --directory) --exclude .gitlab-ci-local/ ./ .gitlab-ci-local/builds/${target}/

Relevant excerpts from rsync docs:

if the pattern starts with a / then it is anchored to a particular spot in the hierarchy of files, otherwise it is matched against the end of the pathname.

So if git ls-files -o --directory outputs "test" because there's a folder at the root of the project that is ignored, rsync will also ignore any path ending in "test" as well, leading to unintended and confusing behavior.
One additional example is with golang, where often ./cmd/mybinaryname is a directory in the project, and the .gitignore contains mybinaryname to avoid committing the binary. The result is that rsync will fail to sync the folder ./cmd/mybinaryname.
Unfortunately I haven't had time to try and patch it, but it looks like prefixing the output of git ls-files -o --directory with a / may be sufficient.

I've written a POC fix and manually verified that it fixes my issue. I'd like to add a regression test but frankly am a little lost in node land. @firecow Is there an existing test case that you could recommend as base for writing one?

Can you show me your fix, then I can better point you in the right direction...

Sorry, it was referenced above my comment.
https://github.com/icook/gitlab-ci-local/commit/2a2dcda4c3e94c7ec54680f59592fc038de817da

Add a .gitignore in tests/test-cases/image/ and ignore /test-file.txt
Change image <test-entrypoint> in tests/test-cases/image/integration.image.test.ts to the following
test("image <test-entrypoint>", async () => {
    const writeStreams = new MockWriteStreams();

    try {
        await fs.writeFile("tests/test-cases/image/test-file.txt", "I'm ignored\n");
        await handler({
            cwd: "tests/test-cases/image",
            job: ["test-entrypoint"],
            privileged: true,
        }, writeStreams);
    } finally {
        await fs.rm("tests/test-cases/image/test-file.txt", {force: true});
    }

    const expected = [
        chalk`{blueBright test-entrypoint         } {greenBright >} Hello from 'firecow/gitlab-ci-local-test-image' image entrypoint`,
        chalk`{blueBright test-entrypoint         } {greenBright >} I am epic multiline value`,
        chalk`{blueBright test-entrypoint         } {greenBright >} /builds/test-entrypoint`,
        chalk`{blueBright test-entrypoint         } {greenBright >} Test Entrypoint`,
        chalk`{blueBright test-entrypoint         } {greenBright >} I'm a test file`,
    ];
    expect(writeStreams.stdoutLines).toEqual(expect.arrayContaining(expected));
});

Revert your fix. Observe the test is failing as you want.
Apply your fix. Observe the test is passing.
Make PR, LETS GO !!! :smile: @icook

Awesome, thanks for the guidance! I spent some time on this but for some reason can't get the repro working yet. gitignore doesn't seem to be applying. I'll come back to it and let you know.

@icook Ask away and paste some errors, and we will get you to the finish line 🤣

Much appreciated, got it sorted. Didn't realize npm run build needed to be called after changing non-test sources in between test runs, so reverting my patch "wasn't working"

Sweet... Good job man 👍


Dual sdk

JIRA Link
N/A
Changelog / Description

Updates REST API with the SDK dual version to support 5.4 and 6.0 chain versions
reduces docker image size

Checklist -

[ ] New Feature ?
[ ] Updated swagger annotation (if API structure is changed) ?
[ ] Unit Test (if possible) ?
[ ] Updated the Readme.md (if required) ?

:tada: This PR is included in version 3.0.0-alpha.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.0.0-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Bug] "Mirror of X" does not change when updating Clone-URL on Mirror

as title
version: up to master

I'll have a look at it

found it


Omnisharp is red. Something about LoggerFactory.

Starting OmniSharp server at 12/24/2022, 8:27:48 AM
Target: c:\Users\timot\FreeRoam\FreeRoam.sln
OmniSharp server started.
Path: c:\Users\timot.vscode\extensions\ms-dotnettools.csharp-1.25.2-win32-x64.omnisharp\1.39.3-beta.24\OmniSharp.exe
PID: 21164
    Starting OmniSharp on Windows 6.2.9200.0 (x64)

    Checking the 'DOTNET_ROOT' environment variable to find a .NET SDK

    Using the 'dotnet' on the PATH.

    DotNetPath set to dotnet

    Located 1 MSBuild instance(s)
        1: Visual Studio Build Tools 2022 17.3.32811.315 17.3.0 - "C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\MSBuild\Current\Bin"

    Registered MSBuild instance: Visual Studio Build Tools 2022 17.3.32811.315 17.3.0 - "C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\MSBuild\Current\Bin"

[STDERR] System.Composition.Hosting.CompositionFailedException: No export was found for the contract 'ILoggerFactory'
-> required by import 'loggerFactory' of part 'DefaultProjectChangePublisher'
-> required by import 'projectConfigurationPublisher' of part 'MSBuildProjectManager'
-> required by import 'item' of part 'IMSBuildEventSink[]'
-> required by import 'eventSinks' of part 'ProjectSystem'
-> required by import 'item' of part 'IProjectSystem[]'
-> required by initial request for contract 'IProjectSystem[] { IsImportMany = True }'
at System.Composition.Hosting.Core.ExportDescriptorRegistryUpdate.CheckTarget(CompositionDependency dependency, HashSet1 checked, Stack1 checking)
at System.Composition.Hosting.Core.ExportDescriptorRegistryUpdate.CheckDependency(CompositionDependency dependency, HashSet1 checked, Stack1 checking)
at System.Composition.Hosting.Core.ExportDescriptorRegistryUpdate.CheckTarget(CompositionDependency dependency, HashSet1 checked, Stack1 checking)
at System.Composition.Hosting.Core.ExportDescriptorRegistryUpdate.CheckDependency(CompositionDependency dependency, HashSet1 checked, Stack1 checking)
at System.Composition.Hosting.Core.ExportDescriptorRegistryUpdate.CheckTarget(CompositionDependency dependency, HashSet1 checked, Stack1 checking)
at System.Composition.Hosting.Core.ExportDescriptorRegistryUpdate.CheckDependency(CompositionDependency dependency, HashSet1 checked, Stack1 checking)
at System.Composition.Hosting.Core.ExportDescriptorRegistryUpdate.CheckTarget(CompositionDependency dependency, HashSet1 checked, Stack1 checking)
at System.Composition.Hosting.Core.ExportDescriptorRegistryUpdate.CheckDependency(CompositionDependency dependency, HashSet1 checked, Stack1 checking)
at System.Composition.Hosting.Core.ExportDescriptorRegistryUpdate.CheckTarget(CompositionDependency dependency, HashSet1 checked, Stack1 checking)
at System.Composition.Hosting.Core.ExportDescriptorRegistryUpdate.CheckDependency(CompositionDependency dependency, HashSet1 checked, Stack1 checking)
at System.Composition.Hosting.Core.ExportDescriptorRegistryUpdate.CheckTarget(CompositionDependency dependency, HashSet1 checked, Stack1 checking)
at System.Composition.Hosting.Core.ExportDescriptorRegistryUpdate.Execute(CompositionContract contract)
at System.Composition.Hosting.Core.ExportDescriptorRegistry.TryGetSingleForExport(CompositionContract exportKey, ExportDescriptor& defaultForExport)
at System.Composition.Hosting.Core.LifetimeContext.TryGetExport(CompositionContract contract, Object& export)
at System.Composition.CompositionContext.GetExport(CompositionContract contract)
at System.Composition.CompositionContext.GetExports(Type exportType, String contractName)
at System.Composition.CompositionContext.GetExports[TExport](String contractName)
at OmniSharp.Stdio.Host.Initialize() in D:\a\1\s\src\OmniSharp.Stdio\Host.cs:line 59
at OmniSharp.Stdio.Host..ctor(TextReader input, ISharedTextWriter writer, IOmniSharpEnvironment environment, IServiceProvider serviceProvider, CompositionHostBuilder compositionHostBuilder, ILoggerFactory loggerFactory, CancellationTokenSource cancellationTokenSource) in D:\a\1\s\src\OmniSharp.Stdio\Host.cs:line 53
at OmniSharp.Stdio.Driver.Program.<>c__DisplayClass0_1.b__1() in D:\a\1\s\src\OmniSharp.Stdio.Driver\Program.cs:line 74
at McMaster.Extensions.CommandLineUtils.CommandLineApplication.<>c__DisplayClass146_0.b__0(CancellationToken _)
at McMaster.Extensions.CommandLineUtils.CommandLineApplication.d__157.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at McMaster.Extensions.CommandLineUtils.CommandLineApplication.Execute(String[] args)
at OmniSharp.HostHelpers.Start(Func`1 action) in D:\a\1\s\src\OmniSharp.Host\HostHelpers.cs:line 30
[ERROR] Error: OmniSharp server load timed out. Use the 'omnisharp.projectLoadTimeout' setting to override the default delay (one minute).

Setting omnisharp.path to empty string Fixed it.


Color attribute no longer works in FAB list [RC3]

Ionic version:
[ ] 1.x
[ x] 2.x
I'm submitting a ...
[ x] bug report
Current behavior:
The color attribute no longer works for buttons in a FAB list.
Expected behavior:
The color attribute should change the color.
Steps to reproduce:
Create a FAB with an <ion-fab-list> with buttons inside it, then try to set the color of the button, for example color="secondary".
Related code:
 <ion-fab center bottom>
    <button ion-fab color="primary"><ion-icon name="md-cog"></ion-icon></button>
    <ion-fab-list side="top">
      <button ion-fab color="secondary"><ion-icon name="md-log-out"></ion-icon></button>
    </ion-fab-list>
    <ion-fab-list side="left">
      <button ion-fab color="secondary"><ion-icon name="person"></ion-icon></button>
    </ion-fab-list>
    <ion-fab-list side="right">
      <button ion-fab color="secondary"><ion-icon name="information-circle-outline"> </ion-icon></button>
    </ion-fab-list>
  </ion-fab>

Other information:
I believe that this was broken with eb0b05df88946f1c662c65a707f603aa46ff53cb or c9fdbf9c10869c8ebf1f32dadbdd6c49fa3ce58f. It appears that .fab-in-list is overriding the colors.
Your system information:

Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.3
Ionic CLI Version: 2.1.13
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.46
ios-deploy version: 1.8.6
ios-sim version: 5.0.6
OS: macOS Sierra
Node Version: v6.3.1
Xcode version: Xcode 8.1 Build version 8B62```

Already fixed in nightly. Thanks for the report.

Quick question. I updated today to rc3 and i am getting the error. My fab list are no more with my color.
What do I have to do ?

@speedfl npm install ionic-angular@nightly

@speedfl Whoops, it's npm install --save ionic-angular@nightly

@Tyler-Darby thanks for the tips however with this new version the no-shadow is not working anymore on the ionic header + the solution proposed here https://github.com/driftyco/ionic/issues/8425#issuecomment-261790685 do not work anymore and labels are not displayed anymore.
Any solutions?
Best regards
Geoffrey MUSELLI


Epson TM-T20II profile

This printer has a 80mm (default) and 58mm configuration.
Column widths are available from page 94 on the tech reference guide, and code pages may differ from the default profile as well.

Hello,
I've been looking for printing support via Python for the EPSON TM-T20III printer. The ESC/POS library doesn't seem to have support for the TM-T20II or TM-T20III, but I found a pdf with the ESC/POS Command reference for the TM-T20 class of printers.
I would be open to providing a pull request to add support for this printer, but I would need a guide on how to do so.
For the time being, I'll provide the direct link to the PDF and upload it here for anyone looking for it.
Link to the original download
Epson TM-T20 ESC-POS Quick Reference Guide .pdf
Thank you.


Cannot run Ashes 2023

I can run this fine with the CLI:
flatpak run org.zdoom.GZDoom -iwad ~/Games/doom-wads/DOOM2.wad -file ~/games/doom-wads/Ashes-2023/Ashes2063Enriched2_23.pk2

But when I use CleanDoom, it doesn't want to run:

Mod: https://www.moddb.com/mods/ashes-2063/downloads/ashes-episode-1ashes-2063pk3-only-v22

Oh I see, I selected on line, but the checkboxes on the right all got checked....  I wonder if that's a good idea to check the boxes when adding mods


Enable usage for partial build

In a big software project you not always want to build and test the whole project. You sometimes only want to build specific subprojects and execute their tests. With the current implementation of pytest_discover_tests this does not seem to be possible - at least to my understanding. When running ctest (even with a regex that only matches the tests of one subproject) all CTestTestfile.cmake files in the project tree are checked and if at least one of the ${_tests_file}s (of other subprojects) was not built this will result in an inclusion error. One possible solution that I can think of would be to create the (empty) ${_tests_file} within pytest_discover_tests if it does not exist yet. What do you think?

@MarcelKoller, apologize for the very late reply!
This is a valid point. Many C++ libraries that provide Python bindings require partial builds to ship products without the bindings, so we don't want Pytest to run all the time.

When running ctest (even with a regex that only matches the tests of one subproject) all CTestTestfile.cmake files in the project tree are checked and if at least one of the ${_tests_file}s (of other subprojects) was not built this will result in an inclusion error.

Right, I was able to replicate the issue. I mostly create partial builds via a custom CMake conditional definitions, so I didn't consider checking what the '--target' option would do. Unlike 'gtest', the 'pytest_discover_tests' command is not set up to run "POST_BUILD" after the corresponding CMake target containing the tests has been built, so I didn't think that using an intermediate "include" script would be necessary. But it seems to fix this issue indeed.
I'll push a fix shortly!

This should have been resolved in #34 and is included in version 0.8.0.
Thanks for pointing out this limitation!


make sign_assertion and sign_response configurable

The saml2int profile actually requires the Assertion to be signed in addition to the response. We might want to make this configurable maybe...? however the default should be saml2int right?

Yes you're right. I'll update the PR to support signed assertions but turn it off as default

I went "nuts" and made both assertion and response signing configurable but with different defaults. My thinking is that we may need to support crazy corner-cases. I've also clarified documentation which was inconsistent with code in the use of idp_config/sp_config

The changes will not allow a per-SP configuration to be applied if there is no explicit default set because, for example,
sign_assertion = self.config['idp_config']['service']['idp']['policy']['default']['sign_assertion']
will throw a KeyError and then the line
sign_assertion = self.config['idp_config']['service']['idp']['policy'][resp_args['sp_entity_id']]['sign_assertion']
will never have a chance to be evaluated.
Was that intended or should there be two (2) try/except clauses each for sign_assertion and sign_response?

Damn, I should have spotted that. I will fix it.


refactor: standard-GEM

Main improvements in this PR:

feat:

adhere to standard-GEM, by refactoring folder tree and README.md (#232)

I hereby confirm that I have:

[x] Tested my code with all requirements for running the model
[x] Selected devel as a target branch (top left drop-down menu)
[x] If needed, asked first in the Gitter chat room about this PR

Awesome!
Looking at the /model/* files, I see they are named yeastGEM instead of the expected yeast-GEM. Any thoughts on the standard-GEM indication that

All model files must be named the same as the repository

?

I don't see a reason why not to adhere to the standard-GEM indication on this, so seems fine to correct this.

Everything looks good, except there is deletion of one entry of model xml file, if that is ok, then everything is fine.

@feiranl This is an artifact of the empty subSystems. I've now manually removed them (model=rmfield(model,'subSystems')) before running saveYeastModel. This will be fully fixed when subSystems are added (see #11).

@edkerk I'm curious, why was 0ef8f3f necessary?

Just for reference, coming back to naming of the model files. In commit 919d396, I also changed the model.id field from yeastGEM to yeast-GEM. However, - is not allowed in this ID in SBML, so that it either would end up with an invalid SBML file, or it would end up with a model.id that reads yeast__45__GEM if replaced with ASCII code as COBRA routinely does. This also raises issues when the model is then loaded with RAVEN or cobrapy instead as these do not automatically modify the IDs to replace ASCII codes. So best to prevent all of this, while standard-GEM also does not specify that the model.id should have a certain format, just the model filename. This is now fixed with 0ef8f3f and aab4e8d.

@mihai-sysbio Very good point, the SBML documentation is very clear on what is allowed, so it would just involve a few regex-checks.


Update Readme for Mac Gatekeeper problems

you mean the unidentified developer box? yea that'll happen if you don't have dev certificates from apple. easy fix

go to System Preferences>Security & Privacy>General
Just press open anyways under allow apps downloaded from
2a. or just for everything in general just change the setting to allow from "anywhere" (not recommended)

Yeah, but a lot of users aren't aware of it. I'm going to update the README to explain it.


Migrate to senzing/stream-producer

Is your feature request related to a problem? Please describe.
senzing/mock-data-generator has been deprecated. Use  senzing/stream-producer instead.
See https://github.com/Senzing/stream-producer/issues/3

Reference: https://github.com/Senzing/mock-data-generator/issues/44

Reference: https://github.com/Senzing/mock-data-generator/issues/44


Pod evaluation fails when more than one container is defined

Steps to reproduce:

create pod with more than one container

Evaluation of this fails when put in tests directory:
local kube = import "../kube.libsonnet";
local simple_validate = (import "test-simple-validate.pass.jsonnet").items_;
simple_validate {
  pod+: {
    spec+: {
      containers_: {
        first: kube.Container("first") { image: "nginx" },
        second: kube.Container("second") { image: "nginx" },
      },
    },
  },
}

Cause
in kube.jibsonnet:
PodSpec: {
    // The 'first' container is used in various defaults in k8s.
    local container_names = std.objectFields(self.containers_),
    default_container:: if std.length(container_names) > 1 then "default" else container_names[0]
    [...]
}

"default" as default_container should be used when std.length(container_names) < 1. In current code container_names[0] is used when container_names is empty.

My bad. When there are multiple containers in one Pod one of them should be named default or default_container should be overridden.


feat: generic iam authorization

Description of changes
This change introduces additional mode for IAM authorization.

Existing IAMAuthorizationConfig is moved to IdentityPoolAuthorizationConfig (except admin roles which are now deprecated)
New setting IAMAuthorizationConfig.enableIamAuthorizationMode is introduced and works as follows:

If not enabled (default) existing IAM rules execute without change (i.e. only IAM roles attached to Cognito Identity Pool work)
If setting is enabled

IAM roles attached to Cognito Identity Pool work as previously
Deprecated admin roles work as previously
Any other IAM principal is assumed to have access via it's IAM Policies and thus auth directive rules are skipped.

CDK / CloudFormation Parameters Changed

Issue #, if available

Description of how you validated changes
E2E tests https://us-east-1.console.aws.amazon.com/codesuite/codebuild/594813022831/projects/amplify-category-api-e2e-workflow/batch/amplify-category-api-e2e-workflow:14528517-b475-4a4c-bffe-120d98ce84fe?region=us-east-1 .
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Any CDK or CloudFormation parameter changes are called out explicitly

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Corresponding docs PR: https://github.com/aws-amplify/docs/pull/7056

Tests green except known schema_searchable problem. https://tiny.amazon.com/15wj5u0l/IsenLink


*.spec.ts: HttpClientModule --> HttpClientTestingModule

Possibly a solution to fix the build errors in travis.
Closes #846

CI is green, that's a good sign... 😂 ... I'll merge and if the next dozen builds or so are stable, please claim the bounty on bountysource.com! 🙏

CI is green, that's a good sign... 😂 ... I'll merge and if the next dozen builds or so are stable, please claim the bounty on bountysource.com! 🙏

I think putting the bounty back on the score board issue will be a better idea. I didn't do anything intelligible as such, like I just followed stack overflow. But I thank you a lot.

Score Board performance issue is solved, and I claimed my own pledged bounty... so go ahead, good stackoverflowing is definitely worth 10$ on this annoying CI issue...

Lots of Thanks in that case🙂😁


Create Pixmap from video element on gwt

You guys can surely guess why it's needed :)

Here a video that this change is working https://cdn.discordapp.com/attachments/348229413785305089/751348815910666260/LibGDX_application_-_Google_Chrome_2020-09-04_09-50-28.mp4

for reference, https://developer.mozilla.org/en-US/docs/Web/API/WebGL_API/Tutorial/Animating_textures_in_WebGL
@SimonIT could you please format only your changes? i understand files wasn't properly formatted before but it'll mess with history. If we really want to fix formatting for some files, i think it should be done in a separated PR.
Other than that it looks good to me. Thank you for your work on gdx-video :-)


Make MySQL secret key configurable

Today, when a customer informs a secret name to pull MySQL credentials, the Secret Key is fixed with MYSQL_PASSWORD.
For customers with dynamic secrets provisioning, this is a problem, given the secret key names are dynamically generated.

Whatever we implement here should be done for Bucket, Redis and any other credentials as well.

https://github.com/wandb/helm-charts/pull/257


Total amount causing issue with paging

Summary of problem or feature request
When I get the sum of amount it works great but once I'm using the pagination the total amount gets 0.
Laravel/PHP Code
->withQuery('total', function ($filteredQuery) {
return $filteredQuery->sum('Amount');
})```
Js Code
table.on('draw.dt', function (e) {
$('#total').text(LaravelDataTables["dataTableBuilder"].ajax.json().total)
});
System details

Operating System unbntu 19.04
PHP Version 7.1
Laravel Version 5.8
Laravel-Datatables Version 9.0

Any error on the response? Maybe try cloning the filteredQuery cause it's currently mutable.
->withQuery('total', function ($filteredQuery) {
    return (clone $filteredQuery)->sum('Amount');
})

I already tried it before but it didn't work, It works correctly only on the page the first page.
the query I get on the first page is:
select sum([Amount]) as aggregate from [Transaction]
and in other pages I got a query like this:
select * from (select sum([Amount]) as aggregate , row_number() over (order by (select 0)) as row_num from [Transaction]) as temp_table where row_num between 101 and 150 order by row_num
but can I listen to all events that are fired except paging? I think it will solve the problem if there is another one

I see, so the problem is paging are getting applied when doing the sum? Maybe try removing the paging applied on query before applying the sum? Is it possible to remove the limit applied on query? Will dig further when I got the chance.

I have the same issue. Is there any solution for this after 2 years?

I have the same issue. Is there any solution for this after 2 years?

any solve on that?
got same problem.

Any Update?? I got the same issue.

@yajra did you got any chance to find solution. i have same problem

There is a work around you guys can do. May be helpful for someone.  Compare the start if its 0 then return else return static value and in front check that condition.

"fnDrawCallback": function( oSettings ) {                 if( oSettings.json.total != 'no' ) {                     $('#total_donation').text(oSettings.json.total);                 }             },

There is a work around you guys can do. May be helpful for someone. Compare the start if its 0 then return else return static value and in front check that condition.

"fnDrawCallback": function( oSettings ) { if( oSettings.json.total != 'no' ) { $('#total_donation').text(oSettings.json.total); } },

This seems a great workaround! It works perfectly.
Really thanks man!!!!
If someone is using datatables as service, in method dataTable has to use
$startNo = $this->request->input("start")
because it's an inherited property.

This works fine for me:
->withQuery('total', function ($filteredQuery) {
return (clone $filteredQuery)->offset(0)->sum('Amount');
})
just add ->offset(0)


Experimental

This is an automated message:
You can preview this docs PR at http://371.docs-pr.validatedpatterns.io
Note that they get generated every five minutes, so please wait a bit.


Fixes tiny bug with selecting restored backup #8019

Fixes wrong determination of Last restored backup - #8019

Type of Change

[x] Bug fix (non-breaking change, fixes #8019  )
[ ] New feature (non-breaking change, adds functionality, fixes # )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[ ] Ran manual Pester test and has passed (.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Unit test is included
[ ] Documentation
[ ] Build system

Purpose
Approach
Use key field restorehistory.restore_history_id instead of backup_set_id which contains duplicates when you restore same backups
Commands to test

Screenshots

Learning

Thank you so much, @Zhunya! This looks reasonable to me. @Stuart-Moore can you please bless this change?

Can anyone push this further because this bug affects Restore-DbaDatabase

oh! this is restore history, nbd, ill merge, thanks again!


allow creating workspace during import

This allows the option to create a new workspace while importing data or tools. It also includes a comprehensive style update on those pages.
Fixes #333

@MattPutnam I definitely hear that, and had originally veered away from doing it that way. In the end, product+design wanted a very specific interaction pattern, which resulting in that particular arrangement of components. Assuming we want to replicate that pattern, it might be sufficient to parameterize the button text so it could be 'Export' in some cases. If not, then we can always pull out the Select (which is really just a very basic Select with workspace names).

This has been superseded by other PRs, closing.


NavigationView: Settings button repositions itself with some delay

Opening this bug on behalf of @fede9876, from https://github.com/MicrosoftDocs/winrt-api/issues/881.
When resizing the height of the NavigationView, the Settings button reposition itself to the correct position with some delay.
The problem is very noticeable when positioning a PaneFooter element near the Settings button because the PaneFooter doesn't have this problem, so for a short time they overlap with each others.
Step to reproduce the problem:

Windows 10 October update (Build 17763)
create an empty project with Min version and Target version = "Windows 10, version 1809 (Build 17763)"
add this code between Grid tags in MainPane.xaml

<NavigationView IsSettingsVisible="True">
    <NavigationView.PaneFooter>
        <NavigationViewItem Content="My Pane Footer"/>
    </NavigationView.PaneFooter>
</NavigationView>

Here's a preview of the problem (the entity of the delay may vary between different devices).

The same issue is also occurring when the NavigationView is in PaneDisplayMode=Top mode.
When resizing, we are seeing a big delay for the settings button to update its position. As in the bug description, the PaneFooter content is not impacted by this.
Windows min version = 10.0.15063, max version = 10.0.18362
Nuget version:  2.2.190611001-prerelease

This is the behavior when it is on top mode:
https://gfycat.com/definitefixedinchworm

Still present in:
Windows: Windows 10 Home 1903 build 18362.239
NuGet package version: Microsoft.UI.Xaml 2.2.190830001
Below is a gif in top mode.

Thanks for letting us know! @licanhua please take a look?

Also @chingucoding this might be a relatively simple one if you have time?

Also @chingucoding this might be a relatively simple one if you have time?

I will look into this.

I don't know if it's still the right place to point this out by I've just installed Win11 22H2 (Build 22621.674) and the new Task Manager is affected by this exact issue.


Error: The hook api-health-checks is taking too long to load

Execute "node. / bin"/ konga.js prepare --adapter postgres --uri p ostgresql://localhost : 5432 / Kong "why did the following error occur
`[root@hecs-x-xlarge-2-linux-20200604174817 konga]# npm  start

<EMAIL_ADDRESS>start /home/konga
node --harmony app.js

(node:9614) Warning: Accessing non-existent property 'padLevels' of module exports inside circular dependency
(Use node --trace-warnings ... to show where the warning was created)
Error: The hook api-health-checks is taking too long to load.
Make sure it is triggering its initialize() callback, or else set sails.config.api-health-checks._hookTimeout to a higher value (currently 120000)     at Timeout.tooLong [as _onTimeout] (/home/konga/node_modules/sails/lib/app/private/loadHooks.js:85:21)     at listOnTimeout (internal/timers.js:549:17)     at processTimers (internal/timers.js:492:7)

@zxlxiaoyi - I am also facing same issue, have you found any workaround?

In case anyone else is facing same issue
I switched to node 12.14.1 and it's working now.


Update README.md

updated to match pull request (rename 40-interfaces, remove 05-firewall, and rename 05-apps)

@a3ilson  thanks for all these updates, I'll test them shortly 👍.


[action] update homebrew install for appledoc action for #17503

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
6454 examples, 0 failures
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
1200 files inspected, no offenses detected
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
The provided command was no longer valid. The repo behind it has been deleted.
https://github.com/fastlane/fastlane/issues/17503
Description
I replaced the link with a currently supported formulae.
https://formulae.brew.sh/formula/appledoc
Testing Steps

If necessary, uninstall your appledocs formula
Run fastlane run appledoc
Run provided brew command

@googlebot I signed it!

@googlebot I fixed it

@seanmcneil This looks good but looks like there might be an issue with the email used for the commit and the CLA signed 🤔

@seanmcneil This looks good but looks like there might be an issue with the email used for the commit and the CLA signed 🤔

Yes, it is likely due to me using a different computer, with different git settings, than my normal one. I'd likely have to restart everything.

@seanmcneil I’m sorry 😱 It would be easier if we didn’t have to have CLA but I don’t have any control over that 😔 If you are able are able to force push over this with a new commit with the other email or create another PR that would be great! If not, I can also create a PR for this change for you but I don’t want to take this contribution away from you 😇
Let me know your thoughts on how you’d like to proceed! ❤

@seanmcneil I’m sorry 😱 It would be easier if we didn’t have to have CLA but I don’t have any control over that 😔 If you are able are able to force push over this with a new commit with the other email or create another PR that would be great! If not, I can also create a PR for this change for you but I don’t want to take this contribution away from you 😇
Let me know your thoughts on how you’d like to proceed! ❤

Sorry for the trouble, but I eventually got everything aligned to make CLA happy. You can delete this & one other PR attempt, as they had unacceptable commits.

I have added a new PR with a valid CLA


Unable to load C# add-on script (even if newly created)

Godot version:
3.2.2.stable.mono.official
OS/device including version:
Windows 7 service pack 1
Issue description:
The engine is unable to read plugins/addons properly, at first I thought it was because I moved the plugin from a project to another, unaware of some kind of dependency in the code, but then I tried to create a new one from scratch and Godot showed me the message "Unable to load addon script from path: '(path of the addon)' There seems to be an error in the code, please check the syntax."
Steps to reproduce:
Project->Project Settings, select "Plugins" tab and click "Create"
fill the required fields in the form and choose C# as language
leave "Activate now?" on and click "Create"

As the documentation says:

Writing editor plugins is possible, but it is currently quite convoluted.

I don't know exactly how to do it though.

Same issue. The plugin.cfg file have no info about the current language to use. The main.cs plugin file generated or the one inside the documentation is not working.
Maybe godot tries to load the .cs file as a .gd file ?
When I restart Godot and go to "Project Settings --> Plugins" and click the Edit button of my plugin, the "language" dropdown reset to "GDscript" everytime. So I guess the plugin.cfg file is wrongly generated and/or loaded. I can't force it by adding a "language="Csharp" line inside the plugin.cfg file.

I've just noticed this occurring on my gdscript smoothing addon, but only when I delete the .import folder.

Unable to load addon script from path: 'res://addons/smoothing/smoothing_plugin.gd' There seems to be an error in the code, please check the syntax.

https://github.com/lawnjelly/smoothing-addon
plugin.cfg
[plugin]

name="Smoothing"
description="Smoothing nodes for fixed timestep interpolation."
author="Lawnjelly"
version="1.0.3"
script="smoothing_plugin.gd"

smoothing_plugin.gd:
tool
extends EditorPlugin

func _enter_tree():
	# Initialization of the plugin goes here
	# Add the new type with a name, a parent type, a script and an icon
	add_custom_type(
		"Smoothing",
		"Spatial",
		preload("smoothing.gd"),
		preload("smoothing.png")
		)

	add_custom_type(
		"Smoothing2D",
		"Node2D",
		preload("smoothing_2d.gd"),
		preload("smoothing_2d.png")
		)

func _exit_tree():
	# Clean-up of the plugin goes here
	# Always remember to remove it from the engine when deactivated
	remove_custom_type("Smoothing")
	remove_custom_type("Smoothing2D")

I pretty much followed the guide in the documentation to make it, not sure what is causing the warning. The addon seems to work ok though.
https://docs.godotengine.org/en/3.2/tutorials/plugins/editor/making_plugins.html

Having this issue on C# addons too.
Created a new one that literally empty and drops the syntax error and don't allow to be enabled.

so like what @Trystan-SA had said at the end was they added "'language="Csharp'" to the plugin.cfg but for some reason it needs to be "language="C-sharp" instead so it looks kinda like this
[plugin]
name="My Custom Node"
description=""
author=""
version=""
language="C-sharp"
script="CustomNode.cs"

Just wanted to weigh in that this is still occurring on v3.2.3.stable.mono.official.
Tried creating a blank plugin, but get this error when trying to enable it:

Build the project once, than it's able to enable it.

How I solved it:
Write the editor plugin script in GDScript, the node extension script can still be CS
tool
extends EditorPlugin

func _enter_tree():
    # Initialization of the plugin goes here.
    # Add the new type with a name, a parent type, a script and an icon.
    add_custom_type("MyButton", "Button", preload("MyButton.cs"), preload("icon.png"))

func _exit_tree():
    # Clean-up of the plugin goes here.
    # Always remember to remove it from the engine when deactivated.
    remove_custom_type("MyButton")

using Godot;
using System;

[Tool]
public class MyButton : Button
{
	public override void _EnterTree()
	{
		Connect("pressed", this, "clicked");
	}

	public void clicked()
	{
		GD.Print("You clicked me!");
	}
}

By just running the project once after you create the plugin then it can be enabled

By just running the project once after you create the plugin then it can be enabled

Build the project once, than it's able to enable it.

Sadly this only solves the problem if you have created the plugin in that very project, if you try to move the addons in other projects or delete the .mono folder where you originally created the plugin they won't load regardless building the project.

By just running the project once after you create the plugin then it can be enabled

Didn't work for me. Currently there seems to be no way to start even an empty plugin if it is written in C#. I'm getting this error message:
Unable to load addon script from path: 'res://addons/custom_node_loader/CustomNodes.cs'. This might be due to a code error in that script.
Disabling the addon at 'res://addons/custom_node_loader/plugin.cfg' to prevent further errors.
No further indication about what may be wrong with it and nothing in the logs either.

So we had a stab on this in the Godot Discord and the following steps seem to fix it:

open your .csproj file
remove the RootNamespace  and AssemblyName tags
save your .csproj file
rebuild solution in Godot
now you can activate the plugin

You can re-add RootNamespace and AssemblyName after you have activated the plugin.

* remove the `RootNamespace`  and `AssemblyName` tags

Removing AssemblyName is enough. IINW this is because the project assembly is always loaded via its default name.

I have been facing the same issue for a long time now, here is my solution/workaround:

Project Settings -> Plugins -> Edit -> Change Language to C#
Delete existing .sln & .csproj files (only if the plugin is moved. if newly created there won't be any)
Project -> Tools -> C# -> Build (it always fails in my case but creates new .sln and .csproj)
Open and rebuild the new .csproj in an external editor. (I only used VS so far)

It is really great to be able to easily create C# plugins for Godot. I hope we see this being closed soon.

I am having the same issue in 4.0.1. None of the workarounds above seem to work. This is on a fresh windows install.

I am also having this same issue on 4.0.1. Fresh install. Fresh project, after following the Making Main Screen Plugins tutorial. I couldn't get this to work in the 4 beta as well (I've never used 3 before) ~ but ~ I figured it was just because it was beta.
I wonder if its because of capitalization issues? ~ Godot seems to want everything lowercase ~ Visual Studio wants everything Camel Cased.

I'm still getting this problem on 4.0.1 as well. Implementing the exact thing from the docs. Making sure i build my project before activation etc.

The problem is clearly capitalization, but not on the the moment on activation but compilation/build.
If you compile the plugin under the "Addons" folder you will be unable to activate it.
But if you compile it under "addons" folder you will be able to activate it, even if you later rename it to "Addons"

@AndreSacilotto Thank you for that one. I was running into this issue on Godot 4.1 stable and was at a loss. It seems this issue still remains for now. Renaming the "Addons" folder to "addons" did the trick.

This is still an issue in Windows with C#, and very confusing when you're just starting with Godot. Renaming the folder, addon file and script file to lower case, solves the issue. Tested on Godot 4.2 Dev 4.

I tried everything described above on Godot 4.1.2 version and the error is still there. I even created a plugin from the editor button following the documentation for C# but it still won't load.
I tried:

Changed upper to lower
Rebuild the project
Create the solutions again
Create a new project from scratch

I mean, my .NET installation should be right because I can run the .cs scripts without problem in Godot. My plugin.cs does not have any syntax error on the file:
#if TOOLS
using Godot;
using System;
[Tool]
public partial class GodotParadiseHealthComponent : EditorPlugin
{
public override void _EnterTree()
{
AddCustomType("HealthComponent",
"Node",
GD.Load

Hi, Tried to rename to lowercase on both folders and files. It did not resolve the issue :/.
Godot v4.2.stable.mono - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1070 (NVIDIA; <IP_ADDRESS>99) - AMD Ryzen 5 5600X 6-Core Processor (12 Threads)
I had the same issue today, and try every methods in this post. Finally it worked.

Change the folder's name to lowercase
Project Settings -> Plugins -> Edit -> Change Language to C#
Add language="C-sharp" in plugin.cfg
Build

don't name your main plugin file as "Plugin.cs". Name it as .. in my case - LayoutDebuggerPlugin.

Don't forget to change plugin.cfg

I had the same issue as well and fixed it by changing:

Renaming "addons" folder -> "Addons"

The issue is still occurring in v4.3.stable.mono.official [77dcf97d8] and C#
Here the contents of the .cfg

i have everything in lowercase, i changed to c-sharp in project settings > plugins

the .cs file seems ok too:

when I click on reload, the failed unloading assemblies occurs (probably because i have another addons running that uses .NET json)

after rebuild, if i go to project settings > plugins > edit, i can still see the language is c-sharp... but when I reload the editor, it changes it the language back to GDScript, all the time. I don't know how to prevent Godot from doing this.
I tried creating the addon from scratch but it still fails.


WIP: add classification setup

Coverage increased (+0.2%) to 50.0% when pulling c8169b00a231a433fc7d149065b5d20dd09c5103 on changsiyao:classification-setup into b294e9dcf3d51b4081570a03989003d9a25faa9a on berkeley-stat222:master.

@jarrodmillman Can you check if my setup for the classification project is fine? And if it is, would you merge it for us? Thanks!

Coverage increased (+0.2%) to 50.0% when pulling f072c6b13983a89768cfbebc958f1fbd6bffe4fe on changsiyao:classification-setup into b294e9dcf3d51b4081570a03989003d9a25faa9a on berkeley-stat222:master.

@MengfeiJiang Can you also take a look and see if I can change this to MRG? Thanks

The structure looks fine to me. @changsiyao

Good. We will pull from the new branch then.

@jarrodmillman Could you please take a look and merge if it is ok? Thanks!


Create specs based on given examples

This commit brings a new set of tests that have been based on some scenarios,
so that we can improve the calculatios of multiple convictions (same proceedings or separte proceedings)
I've marked a test as pending for same proceedings since we haven't tackled yet relevant orders.
Although another test showed that we are marking all separate convictions as
never spent if there's a never spent conviction in one of the groups.
This is not correct, only the convictions that overlap would be considered never spent,
any convictions which are spent before should not be considered "never spent".
It may be better to bring the screenshot explaining the scenario with 4 convictions

This Pull Request will fail but I'd like to confirm this before attempting a fix on the calculator.
Story: https://trello.com/c/W2K1PV43/40-dc-calculations-for-separate-proceedings

Hey yep, I'm aware of the never spent / indefinite being overzealous and taking over other spent days 🤣
As part of another PR I've done a bit of refactor and sorting and should be now be easier to implement this special handling hopefully in some future PR.
With the refactor some of the multiples calculator interface has changed so instead of passing a check_group you now pass the proceeding itself. And no need to call #process! either.

Hey yep, I'm aware of the never spent / indefinite being overzealous and taking over other spent days 🤣
As part of another PR I've done a bit of refactor and sorting and should be now be easier to implement this special handling hopefully in some future PR.
With the refactor some of the multiples calculator interface has changed so instead of passing a check_group you now pass the proceeding itself. And no need to call #process! either.

 [DC] Calculations for Separate Proceedings

 [DC] Calculations for Separate Proceedings

 [DC] Multiples - 'Same' proceedings scenarios for calculations testing

This PR will be closed, only the example tests from separate convictions should be used.
Relevant orders has been covered in #438


Warn when /usr/share/doc/openvpn missing

My server had the file /etc/dpkg/dpkg.cfg.d/01_nodoc which contained ignore rules for the /usr/share/doc directory. Seems like digitalocean added that file.
This script failed with
./vpn: line 239: cd: /usr/share/doc/openvpn*/*ample*/sample-config-files: No such file or directory
gzip: server.conf.gz: No such file or directory
cp: cannot stat ‘server.conf’: No such file or directory
sed: can't read server.conf: No such file or directory
sed: can't read server.conf: No such file or directory
sed: can't read server.conf: No such file or directory
sed: can't read server.conf: No such file or directory
sed: can't read server.conf: No such file or directory
sed: can't read server.conf: No such file or directory```

I think a check if that folder exists with a message about the possible cause would be helpful

Also, it might make sense to add `set -e` to the beginning of the script.

You sure this was done by DigitalOcean? It would be dumb.
I just checked with their following images, none of them had this file:

Ubuntu 14.04 x64
Debian 8.1 x64
CentOS 7 x64

I had Debian 7, and upgraded to Debian 8 yesterday. Might also have been the upgrade process.. And I can't find any information about it either.. weird. It was called 01_nodoc and looked kind of like this: https://unix.stackexchange.com/questions/128884/can-i-disable-all-man-pages

So it was not a clean image. If someone can reproduce this after dist-upgrading a clean image, let me know and I will reopen the issue.


Allow inline conditional subflows

Currently all conditional flows must require a separate flow file.
For small conditional flows like dismissing a popup, allowing some permission, tapping on an element only if it's visible etc, creating a complete new file just to write a couple of tapOn commands is more of a hassle rather than convenience.
Conditional flows should also support inline commands just like repeat does. Wherein commands to be executed can be written in the main flow itself.

This feature is now supported.
Please checkout our documentation and upgrade to the latest version of Maestro.


wallet - clearing pending transactions

When we change somethinig in the ownership, we call "clearPending()" (https://github.com/ethereum/dapp-bin/blob/master/wallet/wallet2.sol#L147). But when doing this, the map with the transaction in the "wallet" contract (https://github.com/ethereum/dapp-bin/blob/master/wallet/wallet2.sol#L301) stays there and does not get cleared. Here an example as a StateTest: https://gist.github.com/CJentzsch/a751c6d6d6bb5c893363

fixed by now


144 minor code refactoring

Current coverage is 94.90% (diff: 76.47%)

Merging #145 into master will decrease coverage by 0.03%

@@             master       #145   diff @@
==========================================
  Files            28         28
  Lines           831        825     -6
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits            789        783     -6
  Misses           42         42
  Partials          0          0

Powered by Codecov. Last update ab0872e...32211ba


Add gutter icons for classes and members navigation

In order to navigate into class hierarchy, it would be nice to have gutter icons at the beginning of a line that contains for instance:

inherited classes (click goes to go parent class)
overridden members (click goes to go parent member)
...

As an example, see what IntelliJ is doing for Dart language

One option would be to add behaviours (on hover, on click...) in gutter icons from DecorationRenderOptions.

cc @jrieken it sounds like another use case of breadcrumb.

Yeah, the outline/breadcrumb information could be sufficient here

I would like to see gutter icons in vscode.
But after reading this, I see that I must switch to IntelliJ.

Yeah, the outline/breadcrumb information could be sufficient here

pretty sad that his proposal was shot so quick down because it's far from being solved by breadcrumbs


Feature/fixing and condition for tags

This PR fixes the issue where if we try to use the filter as is:X+Y, it will return true even if it includes only one of the tags, while it should work as an AND condition.

Looks good to me.
@anshitbansal21 would you mind updating the readme accordingly?

@abhinaba-ghosh This is already documented in the README like -
 mocha --tags "is:X+Y"

Do you want me to add anything else?

@abhinaba-ghosh This is already documented in the README like -
 mocha --tags "is:X+Y"

Do you want me to add anything else?

Make sense! I thought if you want to add the logic description. But it can be done outside of the PR scope too.

The change should be available in<EMAIL_ADDRESS>🎉


[INTEL MKL] Added support for common utility functions used by MKL-DNN enabled kernels for MKL-DNN v1.0

Since this PR builds up on https://github.com/tensorflow/tensorflow/pull/30186, it will be reopened once https://github.com/tensorflow/tensorflow/pull/30186 gets merged into the master.


cannot delete adapter

unfortunately my devices are not supported by the local api, so I have to uninstal the adapter.
However this doesn't work.
`$ iobroker del govee-local --debug
Delete adapter "govee-local"
Error deleting adapter govee-local from disk: Cannot find module 'iobroker.govee-local/io-package.json'Require stack:-<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>/opt/iobroker/node_modules/iobroker.js-controller/iobroker.js
You might have to delete it yourself!
Process exited with code 0`
ioBroker is up to date.
Node.js v18.16.0
NPM 9.5.1
Admin 6.12.0

I assume, this is not a problem of the adapter, but your specific installation ...

txs for the feedback.
Any other adapter can be installed or uninstalled, except this one. However, I'll simply leave it deactivated for the time being.


Appstudio update hacbs-test-project-gsfa

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-gsfa-on-pull-request-
creating pipelinerun hacbs-test-project-gsfa-on-pull-request- in namespace rhtap-demo-iwwl-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-gsfa-on-pull-request-
creating pipelinerun hacbs-test-project-gsfa-on-pull-request- in namespace rhtap-demo-wmog-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


Fix commit message ordering in test

This was clearly wrong, and failed when tested (locally).  I am not able to
explain why the test in the workflow passed.

I missed that too. Wonder how the tests passed in Github..

I missed that too. Wonder how the tests passed in Github..

#27 🤦🏽

Thanks!
Pulling.  This is irrelevant to the 0.43.0 release, so I will not re-release.

Thanks!
Pulling.  This is irrelevant to the 0.43.0 release, so I will not re-release.


Bump osbuilder

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Fedora uki failure is a known issues fixed in https://github.com/kairos-io/kairos/pull/2549


Apply basic formatting to default stack name

In v0.6.0-alpha.2, running rio up without specifying name while in ~/Desktop directory fails with:
FATA[0000] Stack.rio.cattle.io "Desktop" is invalid: metadata.name: Invalid value: "Desktop": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')

Would be nice to apply some formatting to the default name for the user to prevent this error

Initial milestone set to v1.0 - Backlog. Please triage and update as needed.

fixed in v0.6.0-alpha3. Name are converted to lowercase now.

Validated the fix in v0.6.0-alpha.3


[FIX] Home 404 & logo hover fixed

Home 404 & Logo hover fixed

Fixes #490

1. Home page link 404 working now

2. Logo hover effect fixed

Video clip : [ FIXED in this PR ]
https://github.com/user-attachments/assets/ec47571e-1f1a-481b-aaac-af6f45ac843b

Hey @mansiruhil13 , I am merging this PR and adding the same labels you added to this issue.


Download All is not working in dev or production

Lambda does not have the correct permissions to access the bucket.
listObjectsV2 error AccessDenied: Access Denied

closed by #27


FORNO-555: Add start media import command

Description
Adds a new vip media-import command to start a new media import
Steps to Test
Make sure the docker containers for GOOP, Parker and MMS are running locally on your computer.

Check out PR.
Run npm run build
Run VIP_PROXY="" API_HOST=http://localhost:4000 node ./dist/bin/vip-import-media.js @1000.production --url https://beruberuberu-test.go-vip.co/wp-content/uploads/2021/03/wp-content.zip
Verify the command is showing showing the correct environment and URL
Check MMS logs to confirm that files are being uploaded via the mock file service
Verify the command stops checking status after the import is COMPLETED

@hanifn , actually I was thinking, we need to add a disclaimer that if the directory structure inside the archive begins with /wp-content/uploads then we will extract all the files after that root path and import it. If the archive does not begin with /wp-content/uploads (and only that - we are not considering only /uploads) then we will just take the content as is.

we need to add a disclaimer

@saroshaga  Hmm.. Should we add this as part of the confirmation message? I'm thinking that this sort of disclaimer should be shown before the confirmation check

we need to add a disclaimer

@saroshaga Hmm.. Should we add this as part of the confirmation message? I'm thinking that this sort of disclaimer should be shown before the confirmation check

Makes sense, thanks!

Then it will have to be another confirmation check , wouldn't it?

Yeah I've thought about that. I'm just not sure about how to do that in args 🤔

Then it will have to be another confirmation check , wouldn't it?

Yeah I've thought about that. I'm just not sure about how to do that in args 🤔

I think having it as part of the same prompt would also be fine, we just to make HIGHLIGHT it :D

I think having it as part of the same prompt would also be fine, we just to HIGHLIGHT it very well :D

@saroshaga Added in f399935b1c5bfeebe551e6e36bfce13ee322e927
What do you think?

🚢


Make Quiz Exercises Test run for TAs more visible when exercise is not released

Currently, tutors can see quiz exercises, but they cannot test them if they are not yet released. To get a feeling for the quiz and to be able to (easily) check the quiz setup and content, it would be good if tutors would be able to participate in quizzes before they are released, similar to the coding exercises.

UPDATE: I just noticed that there is this functionality under Course Administration - might be good to move it to the overview so that it's accessible the same way as programming exercises, right now it's very hard to find (and none of us tutors knew about it)

We have made the arrangements of the buttons a bit more obvious (I hope so at least) in the latest release.


[Android][TextInput]ShowSoftInputOnFocus has no effect

[Android][TextInput]ShowSoftInputOnFocus has no effect

I am having the same issue


Feature/final touch

Dear Team,
I hope this message finds you well. I am pleased to present the latest updates for the "feature/final-touch" branch. These changes bring a more professional and polished appearance to the project. Here's a summary of the enhancements:

Button Hover Effect: Improved the visual appeal of buttons by introducing a glossy hover effect. The buttons now gracefully transition to a slightly lighter shade of green with a subtle box-shadow, creating a more engaging user experience.

Updated Styling: Made consistent changes to border colors across various CSS files (Profile.css, nav.css, RocketList.css, Mission.css) to ensure a cohesive design throughout the application.

Readme Refinement: Revamped the Readme file to provide comprehensive project information. Added a well-structured Table of Contents for easy navigation and highlighted essential sections, such as "About the Project," "Tech Stack," "Live Demo," "Getting Started," "Contributing," and more.

Live Demo Link: Incorporated a live demo link, allowing stakeholders to access and experience the project effortlessly.

I kindly request the team's esteemed review and consideration for merging the "feature/final-touch" branch into the Dev repository. These updates significantly enhance the project's professionalism and user experience.
Thank you for your time and collaboration in making this project exceptional!
Warm regards,
@Zafron047

@zuhebahmed88091 I have made the necessary changes. Please kindly check the changes. 😸

@zuhebahmed88091 Thanks for the review and your valuable suggestions.


Pick Literate Code Format for EDA

Hey folks,
Is there a team preference between .ipynb or .Rmd format for the EDA?
I am partial to Jupyter notebook but am easy either way.

Second vote for Jupyter notebook.

I agree, I think Jupyter notebook will work well.


Material 3 Ripple

Can we have documentation of the Material 3 ripple effect? It's such an important aspect of Material Design, yet there is nothing, not even a single mention about its specs on material.io
Sure, we can try to reverse engineer it from here, but it's not very easy

We'll be adding usage documentation for <md-ripple>. Unfortunately the mio design spec is lacking a bit, but the appropriate article is about applying states for state layers.

Thank you very much, looking forward to it


codex example can't find the model

     Running `/Users/robert/src/rust/openai/async-openai/examples/target/debug/codex`
Error: ApiError(ApiError { message: "The model: `code-davinci-002` does not exist", type: "invalid_request_error", param: None, code: Some(String("model_not_found")) })

OpenAI deprecated codex models https://platform.openai.com/docs/models/codex
I have an open PR #67 to remove this codex example.
I haven't tried but codex example should work using recent GPT models?


Brightness settings for LIFX less then 100% dims them to 0%

Hi Isaac, I am using this app simply to control my LIFX lights at work to set light colors manually. Sadly, my work wont allow the auto integration for Teams Presence. So, I manually pick red and green colors based on being in a meeting or not. I go to the settings tab, and change brightness to 80% (then click save) and then go to Set Light Colors tab, choose my color, then click Set Color. When I do this, my lights appear to turn off, but in reality it seems to be setting them to 0% (so they appear off). I go into the LIFX app and can see they are still on and I check their current brightness and can see they are set to 1%. In reality, they are not set to 1% because the lights simply are not on. Anyhow, hoping this is something that can be addressed in future updates. I'd like to be able to set my proper brightness.

Disregard, I just noticed there was a new version of the application that addressed this issue! Thanks


Consider porting buffer implementation to atom/text-buffer

https://github.com/atom/text-buffer/blob/master/src/text-buffer.coffee
See also https://github.com/atom/highlights

Closed as of b38d44e.


Enable authorise

Work in progress
Potential fix for
https://github.com/ably/ably-flutter/issues/459
https://github.com/ably/ably-flutter/issues/444
https://github.com/ably/ably-flutter/issues/439

https://github.com/ably/ably-java/issues/926 currently blocking this for Android


Config options missing in docs

The docs seem to be incomplete, no config options.

This is the case for all components and API's

Same here. I imagine it's related to yesterday's update.

This snapshot may help: https://web.archive.org/web/20221127093425/https://astro-imagetools-docs.vercel.app/en/introduction

Thanks for reporting the issue @silveltman . Closing this since the issue has been resolved.


Tabs dissapear with delay when keyboard is open

When keyboard is open there's around 1second delay before tabs disappear. Initially they are attached to the top of keyboard. But if I set my app to Fullscreen mode, then tabs disappear much quicker.
Same issue with body.keyboard-open class. It is added with delay after keyboard is open

Greetings @Yuripetusko!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Greetings @Yuripetusko!
I've closed this issue because my sensors indicated it was old and inactive, and may have already been fixed in recent versions of Ionic. However, if you are still experiencing this issue, please feel free to reopen this issue by creating a new one, and include any examples and other necessary information, so that we can look into it further.
Thank you for allowing me to assist you.


MSBuild is broken in the SDK 4.7.2 Images

Steps to reproduce the issue

powershell
docker run --rm mcr.microsoft.com/dotnet/framework/sdk:4.7.2-windowsservercore-ltsc2019 msbuild -version
$lastExitCode

Expected behavior
The version number of MSBuild appears and the container exits. The output of $lastExitCode is 0.
Actual behavior
The container exits. There is no MSBuild information. The output of $lastExitCode is -1073741502.
Additional information
I've read through https://github.com/MicrosoftDocs/visualstudio-docs/issues/4772, https://github.com/microsoft/dotnet-framework-docker/issues/548 and a few others. Those issues seem to indicate that I am supposed to update my host. My host has every patch applied to it that Windows Update shows.
  See this announcement: https://github.com/microsoft/dotnet-framework-docker/issues/428


Use clang-format-12 in presubmits

We have to have some pinned version and there isn't really a strong
reason it has to be the same as our oldest supported version of clang.
Most devs have more recent clang versions, so the later version is
likely to be closer to what people have locally. clang-format-12 is
the newest version installed by default on the GitHub managed runners,
so it also saves us some apt nonsense.
Fixes https://github.com/google/iree/9154

looks like clang-format-12 is not found in some runs https://github.com/google/iree/pull/9477

Oh no! It's only installed on ubuntu20.04 runners, not the 18.04 we're using. And I'd guess git-clang-format exits early if it doesn't detect any relevant changes. Reverting