Terra-Paginator needs to sync new selectedPage value

Issue Description
Currently the Paginator does not re-sync the selectedPage props when that prop is updated. We have a search box which rebinds the information and replaces it in the table.

Issue Type

[ ] Bug

Expected Behavior
Terra should implement a componentWillReceiveProps  or the new getDerivedStateFromProps to sync the selectedPage to the correct value
Current Behavior

Steps to Reproduce

Create a function that will change how many items are in the paginator
Go to the next page
Change the function to return less or more items, essentially filtering them
Component is in an incorrect state

Environment

Component Version: 1.4.0
Browser Name and Version: All
Operating System and version (desktop or mobile): All

To give some context on my initial design: I had intended for the selectedPage option to be used for setting the initial selected page and the selected page for full-page refreshes.

Here is what our proposal is.
This is the same scenario as a controlled vs uncontrolled input element. We need to create a version of the paginator that completely offshoots all the state handling of the current page off to the consumer. This will take in a page, and require onChange functions to be handled by the consumer, which are activated when the page on a paginator is changed.

That makes sense for giving consumers a lot of control over the paginator. However because the totalPage changed I feel like that it would make sense if the calculation did math to check if the selectedPage was more than the totalPages and then reset the selectedPage back to the passed in value. If that value is over the boundary perhaps set it back to one.
Here's a gist that kind of demonstrates what we're doing in our component.
https://gist.github.com/CoryMcDonald/2d6fc0e58e08ef379e2414712313158f

It's very difficult to have a component that switches between controlled and uncontrolled state. Mixing the two makes it hard to keep the state of the parent and state of the component in sync with each other, especially when the component is trying to change the value based on what was passed in.


Preserve rule status on update according to #1840.

Resolves #1840
Ran PG (#695)

LG but needs tests.

@mdeuser this would be a good one to pick up also for the controller - this fix may be correct but is lacking test coverage.

Closing for #3254.

@rabbah wanted to close I think.


Fix glitches with mobile scrolling

Possible causes include browser-specific CSS rendering and viewport optimizations for mobile.
See below:

This issue has been resolved with #20 and #25.


chore: upgrade react to 17.0.2 [WIP]

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Hi @lilykuang! I see you removed react-virtualized from package.json.
Should we rewrite existed code which uses this lib?

@EugeneTorap I removed react-virtualized for testing purpose. I am actually waiting to see if react-virtualized will release a new version since the support for v17 and v18 https://github.com/bvaughn/react-virtualized/pull/1740 is merged

They don't update the lib during 2 years. I've started rewriting FilterableTable in order to use our new AntD table.

sounds good. thank you for the heads up @EugeneTorap

@lilykuang I see there's apparently only two failing tests left here. As this upgrade is very critical for dependency management, I propose just disabling those two tests for now if we can ensure the underlying components are working with the new version. Thoughts @michael-s-molina @kgabryje @eschutho ?

It would be great if we could merge this PR as part of the 4.0 initiative, during the breaking window. During that period, we'll merge many PRs that introduce breaking changes and will need to do a full test of the application, which would be a great opportunity to test the impact of this PR.
If you agree, just add a card to the 4.0 project board about upgrading React to 17 😉

/testenv up

@michael-s-molina @lilykuang Do we know for sure if there are any breaking changes for this PR or are we thinking of putting it into 4.0 just to be extra cautious?

@michael-s-molina @lilykuang Do we know for sure if there are any breaking changes for this PR or are we thinking of putting it into 4.0 just to be extra cautious?

We added the v4.0 label to merge it during the breaking window and reuse the tests/stabilization period given that this impacts the whole application and will require a full test. Is just to be extra cautious and optimize efforts 😉

/testenv up

Wondering if we should re-open this against the main fork so more people can collab on it. I don't mind doing the rebasing and re-opening if people are going to push on this.

Wondering if we should re-open this against the main fork so more people can collab on it. I don't mind doing the rebasing and re-opening if people are going to push on this.

@mistercrunch This was punted to 5.0 because of the following:

Histogram and Event flow charts use old, unsupported library @data-ui, which is not compatible with React 17. Since there are no easy replacements, we need to rewrite those plugins using a different library or deprecate them before moving on with React 17 upgrade.

So I think we need to resolve some prerequisites before continuing the work. @kgabryje will know more.

Any hacks to support conflicting reacts? Plugins-in-iframe-mode?

hey @michael-s-molina @kgabryje, any news in this area?

Plugins-in-iframe-mode?

There are actually a lot of reasons to do this... but I've tried it begore, and it requires some major refactoring. It ain't gonna be easy until we've reached the end of the theming road. Hopefully we'll get there soon :)


lock / unlock folder and files

Description of the new feature / enhancement
a feature that lock / unlock / encrypt folder and files with a password like winrar has
Scenario when this would be used?
folders and arquives
Supporting information

Not planned for PowerToys.
Refer to https://github.com/microsoft/PowerToys/issues/21759#issuecomment-1304392992


Modify test cases to support changes

Related PRs https://github.com/wso2/carbon-identity-framework/pull/3705
Related Issues Fixes wso2/product-is#12402

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/1198450460

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/1198450460
Status: failure

The issue causes this PR is reverted.


Migrate from Ethers to Alloy

I was to deep into the build to do this before releasing but we should migrate ethers to alloy

Will assign to myself as this is a big task and has a lot of chances to break tons of stuff if done wrong

any updates on it?

No I will take the hit shortly - are you waiting for this migration? @vivekascoder

hey sir any idea when you get to work on this? @joshstevens19

I had a stab at this but it had so many breaking changes and alloy documentation for migration is not super clear so stopped but I 100% want to do this it’s just about time @kevinfaveri


Need to pass auth header and having issues

Hello - currently trying to use reportportal via CLI and I need a way to pass an auth header. I did not see headers as a specific option for newman and report portal so was attempting to use restClientConfig like this:
newman run {{myPostmanUrl}} --folder '$(area)' --reporter-htmlextra-darkTheme --reporter-htmlextra-logs --reporter-htmlextra-testPaging --reporter-htmlextra-omitHeaders --reporter-htmlextra-noSyntaxHighlighting --reporter-htmlextra-title {{myTitle}} -r htmlextra,testrail,@reportportal/agent-js-postman --timeout-script 4000000 -n 1 -e {{myPostmanEnv}}
--reporter-@reportportal/agent-js-postman-debug=true
--reporter-@reportportal/agent-js-postman-endpoint={{myReportPortalUrl}}
--reporter-@reportportal/agent-js-postman-token={{reportPortalToken}}
--reporter-@reportportal/agent-js-postman-launch={{myLaunchName}}
--reporter-@reportportal/agent-js-postman-project={{reportPortalProject}}
--reporter-@reportportal/agent-s-postman-description={{testDescription}}
--reporter-@reportportal/agent-js-postman-restClientConfig="{rejectUnauthorized:false,params: {access_token: 'XXXX'},headers: { Authorization: 'Basic XXXXX }}" \
When I try this I get a 401 from Report Portal but I know my credentials are correct. Any ideas?

Hello @w100frt !
Could you describe your case in more detail?
Why do you need to pass authentication headers? If the token is present, it should be sufficient to report the data.


cannot create resource "clusterroles" while installing "ELK" via rancher

Failed to install app mca-phdzs-p-rpkn6. Error: release mca-phdzs-p-rpkn6 failed: clusterroles.rbac.authorization.k8s.io is forbidden: User "u-zwzqejpnll" cannot create resource "clusterroles" in API group "rbac.authorization.k8s.io" at the cluster scope
I'm really a noob right now :) So I have no idea what user "u-zwzqejpnll"  is.  I logged into rancher via "admin". So I guess I can do anything.
I tried the quicksetup quide ... cluster is running as far as I see.

Useful
Info

Versions
Rancher v2.3.3 UI: v2.3.25

Route
apps-tab.detail

Ähmm ... I think I got it :D
I changed "Available Roles" from "Project" to "Cluster" and now it works.
Sorry!

hello the same problem for me, how you solved this? can you explain clearly?


Pinot hitting stuck partitions while consuming transactionally

We have a Pinot cluster that consumes using isolation level READ_COMMITTED from an exactly-once Flink app upstream. This Flink app publishes using Kafka transactions. The app got into a state where it crashed multiple times leading to many aborted transactions.
The Pinot cluster had many stuck partitions around the same time as the app crashlooping. From Pinot's perspective, the poll call from the consumer was returning no records since all the messages that were consumed were aborted.
We'd see repeated log lines from Pinot for a stuck segment like this. This would repeat indefinitely.
[2024-01-22 23:59:57.269842] INFO [KafkaConsumer] [table__1234__10346__20240122T1755Z:25] [Consumer clientId=consumer-1234, groupId=consumer-1234] Seeking to offset<PHONE_NUMBER>5 for partition table-1234
[2024-01-22 23:59:57.169762] INFO [LLRealtimeSegmentDataManager_table__1234__10346__20240122T1755Z] [table__1234__10346__20240122T1755Z:25] Consumed 0 events from (rate:0.0/s), currentOffset=16548330245, numRowsConsumedSoFar=0, numRowsIndexedSoFar=0

We confirmed that it was stuck on aborted messages by turning on DEBUG logging and seeing these messages from the Kafka consumer. The consumer did this for the entire poll and then would timeout.
DEBUG [Fetcher] [table__1234__10346__20240122T1755Z:25] [Consumer clientId=consumer-1234, groupId=consumer-1234] Skipping aborted record batch from partition topic-1234 with producerId 12191437 and offsets<PHONE_NUMBER> to<PHONE_NUMBER>

To remediate, we increased stream.kafka.fetch.timeout.millis to a larger value.
We're wondering if there's a way for Pinot to detect this case. This might require Kafka changes to support since it seems like the Kafka consumer does all of the filtering for aborted transactions in the consumer library here and does not expose a way to check if all the messages were aborted so Pinot would not know to advance its offsets. https://github.com/apache/kafka/blob/65596d0a2368399c77423d6ca8fca0c15dd64840/clients/src/main/java/org/apache/kafka/clients/consumer/internals/CompletedFetch.java#L205-L221

@dang-stripe
Hi David,
Initially, I thought that we were not making a progress because of Pinot side consumer recreation due to idle.timeout.millis expiration (this is pinot side check where we recreate the consumer if no record gets consumed for a long time. Default value is 3 minutes)
But, you mentioned that increasing stream.kafka.fetch.timeout.millis helped to unblock. If this is the case, Kafka client's client.poll() call actually timed out over and over again. In this case, this case needs to be handled from the Kafka client but it's a little bit of an edge case since the client was behaving as expected based on the provided timeout.
One graceful handling would be:

Kafka client should return empty records for message batch and bump up the offset instead of keep trying to fetch.

The workaround would be the following:

Bump up the stream.kafka.fetch.timeout.millis to fully skip the offsets.
Or, pause & resume with latest to skip records, backfill the missing data.

Thanks @snleee. I think increasing the fetch timeout or pausing + resume is a reasonable workaround.
I don't see anything in the Kafka consumer library that'd allow us to get the offset from a poll call that returns no messages, so this seems like an ask for Kafka to support an API like that. Otherwise Pinot can't actually tell the different between an actual empty batch or an empty batch with only aborted messages.


Use HTTPS endpoint of Betaseries

Betaseries.com recommends the use of the HTTPS endpoint. Edited the plugin accordingly.
Motivation for changes:
https://www.betaseries.com/api/requetes states the HTTPS is preferred over plain-text HTTP.
Detailed changes:

changed API_URL_PREFIX to use HTTPS
also changed the links to betaseries.com documentation to https.

Thanks


Could we link the commit hash on the PR commit page to the commit-specific page?

I'm reporting it to GitHub, please do so too, even if just by copy-pasting this:
Commit link shortening bug: PR commit links only link to the commit

This is a PR commit link:

https://github.com/sindresorhus/refined-github/pull/1119/commits/0f51dfe30eb36e60369a188786d88b6e421f1543

Paste that into a comment, you'll see that it's shortened to "0f51dfe" and it will link to:

https://github.com/sindresorhus/refined-github/commit/0f51dfe30eb36e60369a188786d88b6e421f1543

Which is a link outside the PR, it's a different view.

Bug initially reported in: https://github.com/sindresorhus/refined-github/issues/1053

Does the GitHub bug block the original request? I'm not sure I entirely follow what the problem is but there are already .patch and .diff links that work correctly; the commit link is almost the same so it feels like it should be possible to extract it somehow and use it to linkify the commit hash as well?

Check out @hkdobrev this link:
https://github.com/sindresorhus/refined-github/pull/1045/commits/b7137144f3df9a77f80c78ca63bf43322b2cf621

Refined GitHub is taking care of linkifying it so it works correctly.
Now here's the same link pasted directly without wrapping it in <pre>:
https://github.com/sindresorhus/refined-github/pull/1045/commits/b7137144f3df9a77f80c78ca63bf43322b2cf621
As you can see, GitHub shortens it to be a commit link... but forgets the PR part. Opening it will not show any link to the PR #1045

Sorry, I might be slower today but I think the feature request is this (@hkdobrev, please correct me if I'm wrong):

How is this related to linkification inside the comment box?

That's not what he's asking

@borekb I might have missed that point entirely.

Could we link the commit hash on the PR commit page to the commit-specific page?

@hkdobrev to clarify, do you want to link this

To this page?

If yes, why?

@borekb I think you're right, I got distracted by the GitHub bug.

Thanks for confirming, in my case, the reasons are similar to the OP:

The commit page has slightly different (sometimes preferable) rendering
The commit might be in multiple PRs, the commit page links to them
There might be conversation for the plain commit
The commit page shows commit parent

It's a small convenience, in my opinion similar to linkifying a branch name on the PR page.

good first issue, I might take it to contribute back to this awesome project! Can I start working on it or does this issue need further confirmation that this is something that you, maintainers, are interested in happening?

Yeah you can go ahead

PR: #1429. Most of the time, I didn't know what I was doing :) But making live changes to a Chrome extension was an interesting experience.

@lukehefson can we solve this long-standing papercut? 😅  https://github.com/sindresorhus/refined-github/issues/1053#issuecomment-375200239

I'll dig in!


Store PR item display error

Issue: When issuing out OSMO1 CHAMBER CLEANER, there's display barcode error under STORE-PR.
Is the item name too long?

[SOLVED] The comma between plunger and wire threw off my code. There are few items with long name that has commas, you need to shorten them and remove the commas.


[7.x] Allow route helper to return the router instance on empty arguments.

This PR will allow route helper to return the router instance on empty arguments.
One simple use case is for checking if a route exists inside a view. This PR addresses the warning on IDE when using the Route facade. It will also help the IDE for the auto-completion of methods available in the Route.
Instead of using the facade in view, we can now use the helper to eliminate an IDE warning Undefined class Route.
@if(Route::has('login'))
...
@endif

to
@if(route()->has('login'))
...
@endif

No plans to merge this I don't think.


Fix adding a translation CSV results in errors on initial import for many types of resources

Fixes #93704
@mihe was right, #92320 did not fix the problem. In fact, there were two problems with CSV importation, and #92320 solved only one of them.
The remaining problem was caused by the creation of the translation file in res://, which triggered update_files and then _update_pending_scene_groups. This caused _update_pending_scene_groups to load all the scenes during the first scan before all the resources were imported. On the first scan, without the .godot folder, all the scenes are processed. _update_pending_scene_groups must be called after all resources are loaded.
At first, I tested only adding !first_scan condition before _process_update_pending. It worked when starting the MRP project without the .godot folder. But if I started an empty project and pasted the CSV file and a scene with resources at the same time, the same problem occurred. So the fix is to never call _process_update_pending while scanning. Anway, it should be called at the end of the scanning, once all the resources are loaded.
Note: I think #93723 could also fix this problem because with these modifications, scene are not loaded anymore to find the groups.
Tested with the MRP from #93704 and #83200.

I can confirm that this does fix #93704, and as far as I can tell _process_update_pending and the filesystem_changed signal are both called once the importing is finished, so the change looks reasonable, assuming _update_file_icon_path isn't dependent on either of them in some weird way.

Thanks!


v0.8.2

One thing to consider for the future is if we could add a build task to prs. I'm fairly certain this would compile without error, but having a task would remove that concern while reviewing.


Widen logging dependency constraint

This should get merged in @brianegan. Needing to use a fork of this branch in my project due to the latest stable release of Flutter introducing Dart 3. Otherwise, I come across dependency issues with plugins using newer versions of the logging package.


internal error: entered unreachable code

Bug description
got a PrismaClientRustPanicError
at In.request (/usr/src/app/node_modules/.prisma/sqlite/runtime/library.js:122:5896)\n    at async l (/usr/src/app/node_modules/.prisma/sqlite/runtime/library.js:127:11167)\n    at async main (/usr/src/app/build/cmd/debit.js:66:13)","name":"PrismaClientRustPanicError","clientVersion":"5.13.0"}

How to reproduce

Expected behavior
No response
Prisma information

// Add your schema.prisma

// Add your code using Prisma Client

Environment & setup

OS:
Database:
Node.js version:

Prisma Version

Hi @fjdvchain, unfortunately this issue doesn't tell us any information that can be useful to reproduce this panic.
I'm thus closing it, but I recommend you to open a new one by filling out this template. Thanks!


adjustIntConst fix: adjustBigIntConst

Codecov Report

Merging #49 (90929a8) into main (36e71c0) will decrease coverage by 0.12%.
The diff coverage is 65.00%.

@@            Coverage Diff             @@
##             main      #49      +/-   ##
==========================================
- Coverage   86.47%   86.34%   -0.13%
==========================================
  Files          10       10
  Lines        2876     2893      +17
==========================================
+ Hits         2487     2498      +11
- Misses        314      319       +5
- Partials       75       76       +1

Impacted Files
Coverage Δ

cl/codebuild.go
83.96% <65.00%> (-0.76%)
:arrow_down:

Continue to review full report at Codecov.

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


[WIP] Attempt to enable a WorkerType attribute

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.

Description
THIS IS ONLY A PROPOSAL,
This is how it could look like if we want to consider re-adding the WorkerType attribute for MonoBehaviours.
Thoughts/concerns?
Shall we do it, if so, is this way OK?
Example usage:
namespace Assets.Gamelogic.Shaders
{
    // Enable this MonoBehaviour on client workers only
    [WorkerType(WorkerUtils.UnityClient)]
    public class ReapplyShaders : MonoBehaviour
    {
        private Renderer[] renderers;

        private void Awake()
        {
            renderers = GetComponentsInChildren<Renderer>();
        }

        private void Start()
        {
            // ...
        }
    }
}

Tests
Tested locally on a port of Pirates.
MonoBehaviours do get enabled/disabled depending on which worker type they are on even if they inject nothing. The prefab preprocessor correctly disables them along with other behaviours even if they inject nothing.
Documentation
Prototype phase, no doc yet

I think this looks good. Separate discussion for if we want this or if we can think of something better, but this doesn't look like it has any unexpected difficulties.

Sounds like the approach is approved, will resolve issues in PR and prepare for merge :)


[MemoryLeaks] solve several memory leaks --sync

Synchro PR from https://github.com/medInria/medInria-public/pull/1130 , https://github.com/medInria/medInria-public/pull/1163 and https://github.com/medInria/medInria-public/pull/1161.
This PR solves also a problem with https://github.com/Inria-Asclepios/medInria-public/pull/790, i removed the deletion of the pointers, which are stored in static, and created a problem in pipelines. I'll fix it in medInria3.4 also.
Replace https://github.com/Inria-Asclepios/medInria-public/pull/785
To be solved with https://github.com/Inria-Asclepios/music/pull/1031
:m:

This is a sync AND additional changes in a single commit. Could you indicate which files contain the changes that need to be reviewed?


Stemcell upgrades should be included in selective deployments

In order to handle "off-release" stem cells (stem cells which come out in-between tile version releases), this pipeline should include a resource to apply the most up-to-date stem cell version

We can upload stemcells without associating changes using this feature in om-linux: https://github.com/pivotal-cf/om/commit/16d387b8beb661f20fc5cd5d8af5ff878b597282
then need to associate those uploaded stemcells to whatever product we want to change:
https://github.com/pivotal-cf/om/issues/159

initial iteration. "associate stemcell to product" would be a task within "apply changes"

When performing "associate stemcell version to product", you should be using the version pulled from the stemcell pivnet product. DO NOT query the opsman for all the available versions, because then we might not be applying the exactly correct stemcell version. We only want to apply the stemcell version which definitely passed upload-stemcell

Success. Uploading stemcell is successful && it does not automatically apply the stemcell to ALL associated products

Success. We're now able to automatically upgrade tiles, stemcells, and perform selective-deployments on those upgrades. We've gone from taking ~8-20 hours for ONE change to as little as 15 minutes for a change.


Bump @defichain/playground dependencies

What kind of PR is this?:

/kind dependencies

Codecov Report

Merging #319 (31c36e4) into main (6b327d0) will increase coverage by 13.44%.
The diff coverage is 66.66%.

@@             Coverage Diff             @@
##             main     #319       +/-   ##
===========================================
+ Coverage   67.39%   80.84%   +13.44%
===========================================
  Files          53       53
  Lines         595      595
  Branches      130      130
===========================================
+ Hits          401      481       +80
+ Misses        191      109       -82
- Partials        3        5        +2

Impacted Files
Coverage Δ

src/layouts/contexts/WhaleContext.tsx
66.66% <66.66%> (+7.40%)
:arrow_up:

src/pages/blocks/index.tsx
100.00% <0.00%> (+9.09%)
:arrow_up:

src/components/icons/assets/index.ts
61.53% <0.00%> (+23.07%)
:arrow_up:

src/components/commons/Link.tsx
100.00% <0.00%> (+33.33%)
:arrow_up:

src/components/dex/PoolPairs.tsx
71.42% <0.00%> (+71.42%)
:arrow_up:

src/components/commons/AdaptiveTable.tsx
100.00% <0.00%> (+71.42%)
:arrow_up:

src/pages/dex/index.tsx
100.00% <0.00%> (+100.00%)
:arrow_up:

src/pages/tokens/index.tsx
100.00% <0.00%> (+100.00%)
:arrow_up:

src/components/icons/assets/dBCH.tsx
100.00% <0.00%> (+100.00%)
:arrow_up:

... and 7 more

Continue to review full report at Codecov.

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


TypeError: tag[primaryAttributeKey].toLowerCase is not a function with external scripts in reactjs without gatsby

please help me out from this error:
`TypeError: tag[primaryAttributeKey].toLowerCase is not a function
(anonymous function)
node_modules/react-helmet/lib/HelmetUtils.js:141
138 |   return false;
139 | }
140 |

141 | var value = tag[primaryAttributeKey].toLowerCase();
| ^  142 |
143 | if (!approvedSeenTags[primaryAttributeKey]) {
144 |   approvedSeenTags[primaryAttributeKey] = {};`

used external javascripts for some animations and css change.
code :
import inspinia from "./assets/js/inspinia" class App extends Component {   render() {     return (       <>       <Helmet>         <script src={inspinia} type="text/javascript" />           </Helmet>

I have a similar issue and when looking at what is happening there seems to be an issue w/ an array:
on a console.log
tag, primaryAttributeKey =  { innerHTML: [ 'window.__INITIAL_STATE__ = ', ';' ] } innerHTML

So the primaryAttributeKey is innerHTML, which exists, but that value is an array.
I have the following test setup
 <Helmet>
     <script>
         window.__INITIAL_STATE__ = {};
      </script>
  </Helmet>

@Way-U-Need What version are you using?

It's happened with Helmet 5.2.1 and webpack 4.39.1 in my case

Also got this issue, using 5.2.1 and the following code, on a Gatsby app:
import React from 'react';
import Helmet from 'react-helmet';
import { AppLocale } from '../appTypes';

// prettier-ignore
const FrontLocaleRedirectionScript = ({ locale }: { locale: AppLocale }) => {
  return (
    <Helmet>
      <script type="text/javascript">
        alert("page locale = " + {locale});
      </script>
    </Helmet>
  );
};

export default FrontLocaleRedirectionScript;

TypeError: tag[primaryAttributeKey].toLowerCase is not a function

HelmetUtils.js:121

On my side I'm running into this issue too. Giving a little bit more context

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>...

Currently my team is on a major dependency upgrade and we run into this issue when upgrade the file-loader from v4.3.0 to v5.0.2.

In case anyone else is still running into this issue...If you use v5 of file-loader, you're going to have to set {options: { esModule: false } } for file-loader in your webpack config.
Due to defaulting to esModule for file-loader in v5, commonJS exports are now being exported as objects with a default property for compatibility with esModules. For HelmetJS, it means that the href and src attributes for links (tag[primaryAttributeKey]) are being exported as  { default: '/link-href'  } instead of /link-href.
For more on the general discussion around compatibility you can check out the thread here https://github.com/webpack/webpack/issues/4742

@kembly hi there ! Can't make it work.
Tried adding as stipulated in gatsby-node.js
exports.onCreateWebpackConfig = ({
  stage,
  rules,
  loaders,
  plugins,
  actions,
}) => {
  actions.setWebpackConfig({
    module: {
      rules: [
        {
          use: [
            {
              loader: "file-loader",
              options: {
                esModule: false,
              },
            },
          ],
        },
      ],
    },
  })
}

but throws me this
ERROR

There was an error compiling the html.js component for the development server.

See our docs page on debugging HTML builds for help https://gatsby.dev/debug-html TypeError: htmlComponentRenderer.default is not a function

  TypeError: htmlComponentRenderer.default is not a function

Any help would be appreaciated. For me it's the AMP Boilerplate code that is causing this.

I got this error when importing images and the solution was easy, I had to wrap the imported variable in a template literal
`

`

I managed to get round it by not using JSX:
const childrenAsString = (node) => {
  return (node.children || [])
            .map((ch) => (ch || {}).data)
            .filter(Boolean)
            .join('\n');
};

const HeaderScript = ({ node }) => {
  const innerHTML = node.attribs.src ? null : childrenAsString(node);

  return (
    <Helmet>
      { React.createElement(node.type, { innerHTML, ...node.attribs }, null) }
    </Helmet>
  );
};

Just ran into this error while trying to give 2 strings to a style tag, fixed it by making it one string. It makes sense too, because an array of children doesn't have a function toLowerCase. If you look at the docs, you'll see how they suggest doing it as well:
    {/* inline script elements */}
    <script type="application/ld+json">{`
        {
            "@context": "http://schema.org"
        }
    `}</script>

    {/* noscript elements */}
    <noscript>{`
        <link rel="stylesheet" type="text/css" href="foo.css" />
    `}</noscript>

    {/* inline style elements */}
    <style type="text/css">{`
        body {
            background-color: blue;
        }

        p {
            font-size: 12px;
        }
    `}</style>

Note that each of the style, script and noscript contents is a single string, Helmet only assumes that to be the type of the children

Just ran into this issue with an app bootstrapped with create-react-app 4 with --template typescript. In my case, I solved it by manually pointing to the default export from my file.
<Helmet>
    <link
        rel="icon"
        type="image/png"
        sizes="32x32"
        href={require(file)}
        data-react-helmet="true"
    />
<Helmet/>

Will not work, as @kembly pointed, the import points at {default:'/link'} as opposed to just /link. You can modify the webpack config (in my case with react-rewired or something), or just switch to require(file).default manually, as I did.

I got the error, trying:
Fix:
  <script
    type="application/ld+json"
    dangerouslySetInnerHTML={{ __html: JSON.stringify(data) }}
  />;


Notification not displaying in Notification Center for Android but can see it on console.

The notification is not displaying in the Notification Center but I can see it on console whenever notification is arrived. It only happens on Android though. iOS is working fine. I went through all the open and closed bugs but none  of the solutions are working.
Below is the version I am using.
"react-native": "0.64.3",
"react-native-notifications": "^4.3.0",
Also, notification logged twice when it I arrived.

I'm also facing the same problem, did you solved it?

Hey is anyone got solutions ?. I am also facing this issue, notifications arrived but it not visible in foreground state.while app is in background state then notification is visible there.

The notification is not displaying in the Notification Center but I can see it on console whenever notification is arrived. It only happens on Android though. iOS is working fine. I went through all the open and closed bugs but none of the solutions are working.
Below is the version I am using.
"react-native": "0.64.3", "react-native-notifications": "^4.3.0",
Also, notification logged twice when it I arrived.

Try "react-native-notifications": "4.1.2" will worked !

Hey gents, hopefully useful for anyone else who comes across this. You need to register for a channel when you're on Android, and you need to decorate the local notification payload with a channel id, and then it works great.
Not so much a library-specific thing rather than an Android-specific setup for notification - should be better documented but the library works fine. (I'm on 4.3.1)
Channel creation:
  Notifications.setNotificationChannel({
        channelId: 'my-notification-channel-id',
        name: 'My App',
        importance: 5,
        description: 'My Notification',
        enableLights: true,
        enableVibration: true,
        showBadge: true,
        vibrationPattern: [200, 1000, 500, 1000, 500],
  });

Code I used to add the channel id to incoming remote notifications, and to de-dupe the handling of them (local notification will trigger another received foreground notification.
  Notifications.registerNotificationReceivedForeground((notification: Notification, completion) => {
      if (callback)
        callback(notification);

      if (Platform.OS == 'android') {
        console.log(JSON.stringify(notification.payload));

        // check if we have a payload, and if a channel was supplied - if no channel then we have a remote push notification
        // if channel was supplied, it's a callback to the local notification we're creating inside of this if statement
        if (!notification.payload || (!notification.payload.android_channel_id && !notification.payload["gcm.notification.android_channel_id"])) {
          notification.payload.android_channel_id = "my-notification-channel-id";

          let nId = Notifications.postLocalNotification(notification.payload);
          console.log(nId);
        }
      }

      // Calling completion on iOS with `alert: true` will present the native iOS inApp notification.
      completion({alert: true, sound: true, badge: false});
    },
  );


gpw: command not found

Hi,
Seems that something else has changed as it ran ok yesterday after I made the changes you suggested. I needed to strip out the users and groups to create a brand new setup and am now getting this error:
[INFO] create-hackathon.sh->create_groups_and_users(): started... (20.00 seconds)
[INFO] create-hackathon.sh->create_team(): Creating team #1... (20.00 seconds)
Exception ignored in: <io.TextIOWrapper name='' mode='w' encoding='UTF-8'>
BrokenPipeError: [Errno 32] Broken pipe
./setenv.sh: line 90: gpw: command not found
GAM 4.95 - https://git.io/gam
Jay Lee<EMAIL_ADDRESS>Python 3.7.4 64-bit final
google-api-python-client 1.7.11
Linux Debian 9 Stretch x86_64
Path: /home/admin/bin/gam
Usage: gam [OPTIONS]...
GAM. Retrieve or set G Suite domain,
user, group and alias settings. Exhaustive list of commands
can be found at: https://github.com/jay0lee/GAM/wiki
Any ideas?
Thanks,
Mark

Please discuss this on the group rather than opening issues.


fix(docker): pin dlv version to v1.22.1

<EMAIL_ADDRESS>is incompatible with golang 1.19 which is used by the docker builder.
This change pins dlv version to @v1.22.1 in the builder's Dockerfile to avoid breaking the 'docker-deployment' target.
Fix #247

cc @jpbrucker


config: enable tichi for chaos-mesh/chaos-mesh

close https://github.com/ti-community-infra/configs/issues/110

/hold

[ ] waiting for chaos-mesh member approve
[ ] waiting ensure bot have permissions
[ ] waiting turn off sre-bot features

/lgtm

/merge

/unhold


Mention git-lfs prereq for game assets in readme

I was unaware of git-lfs when downloading & running for the first time, and therefore ran into #23. This should help others get past that road bump.

Hi! Thank you for creating the issue and the pull request. This will indeed be helpful for newcomers.
I'm ready to accept this PR, I've got only 1 minor request: could you move this new section a little bit higher (right above "Building")? Right now it splits the important information about building, and I think the section about lfs has more priority for a reader.

closes #23

I moved the section & removed two lines. One was unnecessary (git lfs fetch) and the other was misleading (git lfs ls-files doesn't necessarily show if the assets are real vs pointers--at least, I couldn't tell the difference at a glance.)


I'm trying to update the Spec and Status in Reconcile code, and only Spec were updated.
The apiserver-builder-alpha release is  v1.16.alpha.1
Here is my code:
func (r *ReconcileMachine) Reconcile(request reconcile.Request) (reconcile.Result, error) {
	// Fetch the Machine instance
	instance := &adav1.Machine{}
	err := r.Get(context.TODO(), request.NamespacedName, instance)
	if err != nil {
		if errors.IsNotFound(err) {
			// Object not found, return.  Created objects are automatically garbage collected.
			// For additional cleanup logic use finalizers.
			return reconcile.Result{}, nil
		}
		// Error reading the object - requeue the request.
		return reconcile.Result{}, err
	}

	// machine deepcopy
	md := instance.DeepCopy()
	md.Status.Status = "ready"
	md.Spec.IP = "<IP_ADDRESS>"
        err = r.Update(context.TODO(), md)
}

My question is:

What is the right way to update Spec and Status?
Should I use DeepCopy before update?

I would appreciate you very much if can give an answer or just give me a guideline link.

What is the right way to update Spec and Status?

yes, this is the expected behavior. spec and status are updated separatedly. moreover, the status update is supposed to be done by requesting corresponding sub-resource e.g.
https://github.com/kubernetes-sigs/controller-runtime/blob/4ba0a3b63c8580e39afb413da3209f5ffcb6b278/pkg/client/client.go#L182-L185

Should I use DeepCopy before update?

it depends on whether the instance is get from the cache or the real request payload. for the former case, yes, i think you should deepcopy in order to avoid mutating the cache entries. as for the latter, you're free to do in-place modification upon the resource.


Use regular URL to generated documentation for checks/open_api_check.py

Use regular URL (written in Markdown) to generated documentation for checks/open_api_check.py

Fixed in https://github.com/RedHatInsights/insights-results-aggregator-utils/pull/252


Angular 2 Inline html template highlight broken after update

hello
After upgrading to 11.0.10 this feature doesn't work anymore. if I install 11.0.1 its working fine.
not sure if this is a bug, or this is not supported anymore

This is happening to me too. Can this please be resolved?

Same problem hier :(

I had the same issue upgrading to 11.0.10. Going back to 11.0.3 fixed the issue.

Thanks @colingm, interesting. Currently not using Atom for any projects, but it's nice to know this is functionality is back 👍


Split the visualisation script

Cytoscape and microreact/grapetree/phandango use quite different paths. I think a better interface would be splitting this into two commands:
poppunk_visualise_tree
poppunk_visualise_network

Where the first makes all three outputs, the second does cytoscape. The network command may more naturally be part of the upcoming network package.
Would need to work out exactly where MST should fit into these (as it's kind of both).

Specifically, this package should do the visualisation for MST and cytoscape poppunk_visualise_network, and PopPUNK should do trees/microreact poppunk_visualise_tree.


⚠ Warning: Extra attributes from the server: style, class

Description:
When loading any of the pages, we received unexpected attributes: style and class error in the browser console.
Steps to Reproduce:

Load any page
Check browser console

What I've Tried:

Turning off all browser extensions
Happens in Firefox, Chrome, Edge, Brave. DOESN'T happen in Opera.

Screenshots:

Notes:
If anyone has encountered this issue or has a potential fix, please share in the comments below.

Solved here: https://github.com/radix-ui/themes/issues/22#issuecomment-1763201093


Language Drop Down Not working for Players

Describe the bug
As a DM I can see all languages in the drop down, when I logged with a test player, there are no other languages other than default Common for D&D5E, the player with ownership of the actor has access to 3 languages, Common Druidic and Gnomish both of which are not selectable in the dropdown at all.
To Reproduce
Steps to reproduce the behavior:

Load up a world using D&D5E with the speak as and polyglot and any dependencies.
Import a generic starting hero from the 5E Compendium, give ownership to a player
Player logs in goes to chat and sees only Common
Switch to the DM screen and the dropdown shows all languages.
Tried creating a new actor incase it was just bugged to no avail.
I tried reloading the entire application and that did not work

Expected behavior
The language dropdown should simply show all the languages that character knows,
Screenshots
If applicable, add screenshots to help explain your problem.

Game System and version D&D5E 2.2.1
Polyglot Version 1.9.1
Are you using Babele? No

Was the player assigned to the character? As in, does their name shows up like this in the players list?

Nope, and I just realized when I did assign a character its working as normal, I had thought assigning overship was enough my mistake.

It works, but only if the owned character is on a scene, and that can conflict with other owned characters since it will behave just like when a GM selects a token.


Convert event type to dropdown

Basic Information
When creating an event, the event type currently only accepts a string - but the only valid inputs for "event type" should already be in the database.

Looks like this is fixed already.


Try using windows-latest CI

See if our windows-2019 woes are solved now that backtraces are using
frame pointers instead of native APIs.

Ok looks like things are indeed green now!
Due to our repository configuration this can't land immediately though. The CI gates we're requiring have "windows-2019" in the name whereas in this PR they switched to "windows-latest". To land this I'll need to switch the gates to the "windows-latest" version as reported in this PR. When I do that switch though all other PRs will have to rebase on this to merge since they otherwise won't have a run on "windows-latest" to succeed.


Ensure all subcommands display a warning when --policy filters out all policies

Discussed in our call today.  Will update this with details as needed.

@kapilt - This warning message works currently for all subcommands that load policies.  I did notice some other behavioral problems though.  I'll outline those in two new issues and you comment on the potential fixes.

No action required here - I think this can be closed.


[Runtime DS] Add location header to create API from service response

Purpose

Fix https://github.com/wso2/apk/issues/1011

Codecov Report
Patch coverage has no change and project coverage change: +45.85 :tada:

Comparison is base (f5dc79e) 33.36% compared to head (1ce831f) 79.22%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1012       +/-   ##
===========================================
+ Coverage   33.36%   79.22%   +45.85%
===========================================
  Files         173       11      -162
  Lines       19248      592    -18656
  Branches     5706      257     -5449
===========================================
- Hits         6423      469     -5954
+ Misses      12705      123    -12582
+ Partials      120        0      -120

Flag
Coverage Δ

adapter
?

admin-domain-service
∅ <ø> (∅)

backoffice-domain-service
?

devportal-domain-service
∅ <ø> (∅)

idp-domain-service
79.22% <ø> (ø)

router
?

runtime-domain-service
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 162 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Fix line number API usage

This PR fixes the samples that are using the deprecated API of lineNumbers (boolean). They now use "on", "off" and "relative".

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

@nikeee thanks for updating the API usage ;)


[Perf] Small delay displaying RCTRootView after a push

I am attempting to integrate React Native into an existing Objective-C app. My goal is to replace a UIViewController's root UIView with an RCTRootView. Following these instructions https://facebook.github.io/react-native/docs/embedded-app.html, I am doing the following in viewDidLoad: of the view controller:
NSURL *jsCodeLocation = [NSURL URLWithString:@"http://localhost:8081/index.ios.bundle"];
RCTRootView *rootView = [[RCTRootView alloc] initWithBundleURL:jsCodeLocation moduleName:@"SimpleApp" launchOptions:nil];
rootView.frame = self.view.frame;
self.view = rootView;

Everything works fine, except that there is a very small (maybe half second) delay between when the view controller is pushed and the React Native view is actually displayed. I tried pre-bundling the javascript but there was no difference. I can only assume this is because the javascript is loaded asynchronously. Is there anyway to guarantee the view is loaded so that it is immediately seen upon a push transition?

There's an NSNotification called RCTJavaScriptDidLoadNotification that runs once the JS has finished downloading you could listen to. Alternatively you could write a native module that lets JS tell ObjC when your React view has mounted.

@ide I don't really care about knowing when the JS has finished downloading, because as far as I can tell that when will never be fast enough.
What you are suggesting is an interaction like this:

User taps a table view cell
Wait for the RCTJavaScriptDidLoadNotification to fire
Push detail view controller

To me, this is unacceptable. A native iOS user would expect there to be no delay between #1 and #3.

I understand you want to improve perceived performance. Two things you could do (separately or together) are:

Immediately push a view controller that displays a loading indicator. This way the UI immediately responds when the user taps the table cell and also lets them know the app is loading content and isn't stalled.
Preload a hidden RCTRootView in advance, mounting an empty component just to set up the JS environment. When the user taps the table cell and you know the content to display, and the RCTRootView has successfully finished loading (known via the technique in my first post), send an event from native to JS with the data you want to display and show the RCTRootView.

@marcshilling The JS can be preloaded by the bridge in advance of your RCTRootView being displayed. I suggest storing an RCTBridge instance in your app delegate and preloading the JS. You can then create a new RCTRootView on each tap, using the same bridge instance (via -[RCTRootView initWithBridge:]), and it should load much more quickly.

@nicklockwood's suggestion is really clean and should work well. ++

We should probably provide a different template for apps with multiple RCTRootViews. Or maybe just deprecate the RCTRootView convenience constructor that obscures the existence of RCTBridge.

:+1: on @nicklockwood's suggestion as well. I'm using this method in an existing iOS app and it has been working wonderfully for me.
I've noticed also that even with this method, there can still be a very slight delay between when the view is pushed and when the content gets rendered. So, in some cases, I'm taking a hybrid approach. Not only do I have the RCTBridge instance in my app delegate, but I also pre-load my RCTRootViews when the calling view loads. Then, when it's time to push the rootView, it's already been loaded and there is no delay at all. Of course, this isn't feasible in all situations (might not work in a UITableView), but it does work.

@dsibiski - any interest in updating the Embedded App docs to reflect your approach? It seems like you have a good amount of experience with it :smile:
@nicklockwood

We should probably provide a different template for apps with multiple RCTRootViews. Or maybe just deprecate the RCTRootView convenience constructor that obscures the existence of RCTBridge.

Should we spin this off into a separate issue?

You might be interested in the new RCTContentDidAppearNotification in the latest release btw :-)

@nicklockwood  Unfortunately the RCTBridge must be initialized on the main thread,  if I preload a bridge in my app delegate,  it will not only block my app's launch but also consume a lot of memory, any advice about this?

I'm surprised that RCTBridge blocks your app loading noticably. It's true that it has to be initialized on the main thread, but msot of what it does afterwards

You can load the bridge whenever you want, it doesn't have to be at app launch time. For example, you might load it on the screen before you present your RCTRootView controller so that by the time the user presses the button to display the React view, it's already loaded the JS. You could then throw it away again if they go somewhere else in the app.
To avoid the additional delay due to the JS app initialization, you could also try pre-creating the RCTRootView off-screen, and then set it as the root of your view controller when it's presented.

@nicklockwood Thanks for your advice, In my case,  the RCTRootView must be preloaded at app launch time, but my app is a pretty large app which is very sensitive about what can be done at  launch( RCTBridge's setup method now takes about 500ms,  it must be called on main thread, and if the bridge is not initialized, the JS can not be downloaded and loaded.)
So I'm looking forward to a setup API  which can be called  on a background thread completely, for example,   it can set up a context which have  downloaded and loaded the JS on the background thread , and when I come to main thread, I can initialize a bridge with the context, thus save a lot of time.

+1 if that makes sense :)

I'm working on an example repo that will cover more use cases than the one discussed in the docs. There are a couple very simple examples now, but I plan to add more advanced ones very soon.
https://github.com/dsibiski/react-native-embedded-app-example
I'll also try to work on the 'Embedded Apps' documentation to reflect some of the possible examples.

@dsibiski
I have been looking at your repo and will create some issues soon. One thing I noticed right away is that it's light on comments :)
Thank you and I also meant to update this thread with a link to it, but you beat me to it.
 I'll try to add some comments around the interesting bits. Good call.

There's also a loadingView you can set on RCTRootView to display a spinner while the React context is initialising. Feel free to reopen this task if you have any other questions about bridge sharing/preloading.

Let me cut to the chase, I'm trying to load an RCTRootView in a UITableViewCell. The data for the react view is coming from an API.
I have created the bridge in the AppDelegate as @nicklockwood mentioned. I am quickly initializing an RCTRootView when the detail view loads, and then supplying it with constraints to the UITableViewCell. When the API call is finished and I have the data, I send the RCTRootView the appProperties as data.
But at this point, the RCTRootView doesn't size to fit properly with the UITableViewCell. I am calling tableView.reloadData after a second or two for this to happen.
What am I doing wrong? How can the performance be improved here?


Link new ReVanced Extended package name

Description

Icons addition information

Icons linked

App Name (linked ytm.rvx.noname.exe to @drawable/youtube_music)
App Name (linked yt.rvx.noname.exe to @drawable/revanced_extended)

Advice needed
I've added several lines into the linking XML because I don't really know which activity is the main activity for RVX and because there are also several lines for the old RVX linkage.
Old RVX links:
<item component="ComponentInfo{app.rvx.android.youtube/com.google.android.youtube.app.honeycomb.Shell$HomeActivity}" drawable="revanced_extended" name="YouTube ReVanced Extended" />
<item component="ComponentInfo{app.rvx.android.youtube/com.google.android.youtube.app.honeycomb.ShellHomeActivity}" drawable="revanced_extended" name="YouTube ReVanced Extended" />

Links I've added:
<item component="ComponentInfo{yt.rvx.noname.exe/com.google.android.youtube.app.honeycomb.Shell$HomeActivity" drawable="revanced_extended" name="ReVanced Extended" />
<item component="ComponentInfo{yt.rvx.noname.exe/com.google.android.youtube.app.honeycomb.ShellHomeActivity}" drawable="revanced_extended" name="ReVanced Extended" />
<item component="ComponentInfo{yt.rvx.noname.exe/com.google.android.youtube.app.application.Shell$HomeActivity}" drawable="revanced_extended" name="ReVanced Extended" />
<item component="ComponentInfo{yt.rvx.noname.exe/com.google.android.apps.youtube.app.application.Shell_HomeActivity}" drawable="revanced_extended" name="ReVanced Extended" />

Examining logcat shows that when RVX starts, yt.rvx.noname.exe/com.google.android.youtube.app.honeycomb.Shell$HomeActivity is started (first line I've added links to it).
Contributor's checklist

[X] I have followed the Lawnicons Guidelines
[X] I have ensured that Lawnicons builds correctly
[X] I am willing to make changes to my icons if someone suggests changes

👍


Streaming: Surface exceptions like other paths to allow error handling in upper layers.

Fixes https://github.com/microsoft/botbuilder-dotnet/issues/5793
When we send outgoing activities in the adapter for http flows, if they fail, we let the exception get thrown so that it reaches the top level adapter. Cloud adapter+  streaming also does not catch, so that errors flow to the handlers.
However, outgoing activities on streaming when using the BotFrameworkHttpAdapter, we do catch the exceptions, log them and return null. The result is negative in a number of ways:
1- Errors are swallowed
2- Callers cannot retry or handler errors in code
3- Developers expect OnError trigger to get called but when using streaming it does not (OneVoice ran in to this)
This change makes all outgoing activities treat errors consistently (by not hiding them) and solve the problems above.

Pull Request Test Coverage Report for Build 260961

0 of 0   changed or added relevant lines in 0 files are covered.
14 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.02%) to 76.138%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Builder/Streaming/StreamingRequestHandler.cs
14
56.29%

Totals

Change from base Build 260124:
0.02%

Covered Lines:
23120

Relevant Lines:
30366

💛  - Coveralls


Add Color selection to project creation

This needs library support if I am not mistaken.

Yes and we need to ask backend first as well.

Color should be sent in int format. Int refers to a certain color code
Example in.
https://github.com/toggl/toggl_api_docs/blob/master/chapters/projects.md#create-project
    "0":  "#4dc3ff",
    "1":  "#bc85e6",
    "2":  "#df7baa",
    "3":  "#f68d38",
    "4":  "#b27636",
    "5":  "#8ab734",
    "6":  "#14a88e",
    "7":  "#268bb5",
    "8":  "#6668b4",
    "9":  "#a4506c",
    "10": "#67412c",
    "11": "#3c6526",
    "12": "#094558",
    "13": "#bc2d07",
    "14": "#999999"

added to library

UI part is done and seems to mostly work. It does seem however that the library does not like the colour the UI tells it and changes it back to some other colour a few seconds later (maybe it receives a colour from the backend?)
@tanel: could you look into this?

ok will take a look asap!


Got trouble when integrate with ATT&CK Navigator

Hi I'm trying to integrate Workbench with Navigator, I changed the configration file /src/assets/config.json in Navigator server and the config looks like this.
{
    "versions": [
        {
            "name": "ATT&CK Workbench Data",
            "domains": [
                {
                    "name": "Enterprise",
                    "data": ["http://localhost:3000/api/stix-bundles/?domain=enterprise-attack"]
                },
                {
                    "name": "Mobile",
                    "data": ["http://localhost:3000/api/stix-bundles/?domain=mobile-attack"]
                },
                {
                    "name": "ICS",
                    "data": ["http://localhost:3000/api/stix-bundles/?domain=ics-attack"]
                }
            ]
        }
    ]
}

after this i clicked create new Layer in Navigator, i can see the config related to Workbench, but after i click create it just rolling and rolling without show any notice or result..

Then I found an error through devtools. is there any knowns issue related to this?

Hi @dickens88,
This is really more of an ATT&CK Navigator issue so I'm closing it in favor of tracking the fix in https://github.com/mitre-attack/attack-navigator/issues/324 (which is the same issue duplicated on that repo).


Go to production

Hi,
I'm using this library in few my projects for some time (over 3 months) and everythings works fine.
Maybe it is time to go from testing/dev to production?
By production I mean:

change package name from dev to org.jboss,
add this library into maven repository; so far I have build step in my projects, to download it from repository and build.

@Ryszard-Trojnacki You've answered one of the questions on my mind, I had been wondering if anyone was using the library in their projects. Any issues come up getting it added to your projects?
I'd like to see a published jar as well.
Also, it might be nice to see if we can get it working with Jitpack. I attempted to use Jitpack to pull it into an experimental project yesterday and Jitpack had problems building it; I ran out of time before I identified what the issue was. :)

Any issues come up getting it added to your projects?

I'm using RESTEasy Guice for a long time - for few years in several projects. Before I had to stay with RESTEasy 4.7.9.
Few months ago, when you published this new version of RESTEasy, I migrated two of my projects and so far everything works fine. I didn't experience any problems and everything works like it should be, like it worked in old RESTEasy.
But two things:

I don't use @RequestScoped,
I configure this by code in Jetty:

                var restEasy = injector.getInstance(GuiceResteasyBootstrapServletContextListener.class);
                main.addEventListener(restEasy);
                main.setInitParameter("resteasy.guice.modules", RestModule.class.getName());
                main.addServlet(HttpServletDispatcher.class, "/api/*");

Also, it might be nice to see if we can get it working with Jitpack.

I didn't know that there is this service (jitpack) and there is https://jitpack.io/#resteasy/resteasy-guice package and it works for me.
repositories {
    mavenCentral()
    maven { url 'https://jitpack.io' }
}
dependencies {
    implementation group: 'dev.resteasy.guice', name: 'resteasy-guice', version: '1.0.0.Alpha1'
}

I will be happy help.

Ah interesting! I didn't think to try that as a group. That worked for me, thank you! :)

@Ryszard-Trojnacki , I'll let @jamezp comment here regarding pushing artifacts for this project in the maven repo.
I'd like to clarify though that the "dev" in dev.resteasy.* package has nothing to do with the expected stability/maturity level of the software, while it's really connected with the RESTEasy domain name (https://resteasy.dev/).

I apologize, I must have missed this question for some reason.
As @asoldano said, the "dev" part of the package name is stable and it's already been deployed to Maven Central https://repo1.maven.org/maven2/dev/resteasy/guice/resteasy-guice/. The dev.resteasy.guice is the new package name.
There will be migration steps in order to use this, and renaming the packages is one of them. I've filed https://github.com/resteasy/resteasy-guice/issues/16 to update to README to reflect this.

Works for me (even better with dev package). Thank you.

No problem @Ryszard-Trojnacki. Once there is some usage and everyone seems satisfied, I'm happy to do a 1.0.0.Final release as well.


⚠ AGEG website (redirect) has degraded performance

In f2f160a, AGEG website (redirect) (https://ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2207 ms

Resolved: AGEG website (redirect) performance has improved in be623f8.


[translation] add source url to DocumentStatusDetail

The service has started returning a property called source_path that links back to the source document on DocumentStatusDetail. We should expose this in the client library and consider naming implications now that we have a url for both source and target on that return type -> source_url and target_url

can't find the source_path on DocumentStatusDetail generated model only the path attr
should i add it manually?!

Ah right, we'll need to wait until the swagger is updated to include source_path and then regenerate on it to get this new field. Let me check on that.

we need to update the samples once code re-regeneration is done!
this is in order to use source_path instead of translated_document_url in some cases
as current code samples conflate the two!

Very true! I'm going to open an issue to update samples.

completed in #17719


Split configs in @ts-engine/eslint-config

When running lint in a project without react you get this warning:
Warning: React version was set to "detect" in eslint-plugin-react settings, but the "react" package is not installed. Assuming latest React version for linting.

Break out react linting into different ruleset:
extends: [
  "@ts-engine/eslint-config/recommended",
  "@ts-engine/eslint-config/react"
]

Linked to #49


Software closed because an error occured

I am unable to access my sdcard as whenever I try to open it, it closes and gives me an error.
v1.1 with fw 8.1.0 with atmosphere v0.9.4 on emummc.
Error occurs with both applet and title redirection.

Can confirm
v1.1, 9.0.0, v0.9.4 sysnand

Seems to happen with windows primarily.

When attempting to test on Atmosphere 0.9.4 - HOS 9.0.0 with commit 364eb I get the following backtrace: https://pastebin.com/phMXXeaj
Unsure what the heck HID has to do with Windows - will continue debugging.

Well based on the lines where it crashes with, it would seem some sort of memory corruption issue I guess? monkaS

That seems to just be a bug from threading, when I remove that thread it definitely looks like some memory corruption is happening.

Here is the difference between Linux and Windows when the OS requests the contents of a directory... Windows seems to be requesting the contents of objects that have handles out of range (12289, etc), and this is causing incorrect memory accesses. Can do a quick fix just checking the range of handle accesses, but want to try and find out first why Windows is so bad. (might be bad handling of some other MTP request for Windows)

https://github.com/retronx-team/mtp-server-nx/blob/master/source/MtpServer.cpp#L603
seems there was a note to fix this already. Windows MTP appears to be getting drunk and asking for object refs of object handle 0x3000 that doesn't exist. Will just do a somewhat hacky fix for now unless someone has better insight on what is happening.

It appears even with the proper fix for the above using your fix-windows branch, crashing is still occurring with Windows 10 and fw 9.0.0. Looking at the crash logs and comparing with the elf, it appears this is still likely a memory corruption issue of some kind.
01569855307_010061d00db74000.log

@Adubbz As it turns out, the Android code is quite bugged, I don't even know why it properly works on any android device...

Am working on troubleshooting this with someone else as I seem to be the only person who does not experience this issue. First glance it might possibly be an issue with the modifications for Switch support (although why it only now popped up I have no clue).

Yep, happening to me too. Might swap back to the old, old mtp app for now.

happens to me too

Confirming this issue as well. Windows 10.


Release v0.3.12

A PR to release the fix to restore filenames to anonymous functions.

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

Closing this PR as this change got released in the v0.4.0 release.


Bower Issue Running gulp

Hi i have this issue happening when i try to run it with gulp.
Error in plugin 'run-sequence(dev)'
Message:
The bower package jquery has no bower configuration file.

Did you sudo npm install gulp -g && sudo npm install bower -g && npm install && bower install, especially the bower commands?

Yes issue is resolved and completely fine! Thanks you


Running in VPC namespace

Question
How to run it alongside other applications in different namespaces in a different namespace  within  a VPC?
Considerations
I currently use it as a service with cluster IP and the vault fails to init and unseal.

Hey @PrahathessRaghavan
I've done some thinking about this and also socialized with some folks internally. Our recommendation is that you should have a dedicated cluster for Vault and always communicate through the load balancer.

@sethvargo thank you for getting back It'll be much useful learning for me if you can share why that's is so , if this is not a big ask .
Thanks

A lot of it revolves around TLS and upgrades. By using the load balancer, you don't have to trust all possible IPs and DNS addresses. It also helps because you can swap clusters out to do upgrades more easily.


How to configure TLS certs in client app

My app can access our non-TLS, development PostgreSQL instance. But when attempting to test against a production environment, I am forced to use TLS access. So far, I've been unsuccessful in locating an example on how to configure my client cert and key .pem files. I have the correct .pem files, but I'm missing how to specify these files in the 'ConnectionConfiguration()'.
A brief example would be greatly appreciated.

Are you authenticating the server, client, or both?  If the server, then the server must have a chain of trust back to a root-CA that's available on the machine.  I would like to add self-signed server certificate support (see #17).  If you are trying to do client authentication, that's supported by the underlying BlueSSLService but not supported in this library I believe.

I'm guessing the client, as the server is a Google SQL Cloud instance with TLS-only connections allowed. I have generated the certs via the Google dashboard, and downloaded the 'client' .pem file and the 'key' .pem file to the client machine.
I used these certs to configure Postico (a GUI utility) on the client, and it can now access the cloud server using TLS.  I need some way to configure these file with PostgresClientKit.
The 'ConnectionConfiguration' class has a .ssl property, but it's only true/false, with no other documented way of specifying there certs.

Here's my understanding of the current situation:  If the .ssl property of ConnectionConfiguration is true, then PostgresClientKit attempts to create a TLS connection to the Postgres server.  It will not authenticate the server certificate (so the connection will be encrypted, but there is the potential for a MITM attack as pointed out in #17, for example if DNS resolution were compromised).  It will also accept a self-signed server certificate.  And it will not send a client certificate (which would allow the server to authenticate, at the TLS level, the client).
From what you wrote, it sounds like your server requires authentication by a client certificate.  That would be why PostgresClientKit can't connect.
Does your scenario require client authentication at the TLS level?  (To be clear, this is distinct from Postgres user authentication, for example by Credential.cleartextPassword in PostgresClientKit.)
If not, you could change your server's configuration to not require an incoming connection to provide a client certificate.  With that change, PostgresClientKit should create a TLS connection.
But, if your scenario does require TLS client authentication, please look at my comment on #17.  This enhancement would allow the consumer of the PostClientKit library to have more control over SSL configuration.

Thanks for the responses. It's valuable information to know. My interim solution has been to download Google's Cloud SQL Proxy daemon. This little guy runs from the command line or launchd, and connects over TLS to the cloud instance. It then allows local apps to connect via <IP_ADDRESS>:5432. This solves lots of issues, but puts more things to break between the app and the instance. However, no complaints so far, and every local app connects transparently without issue.

v1.1.0 adds an sslServiceConfiguration property to ConnectionConfiguration, which allows you to set the client certificate to be used.  Please refer to the BlueSSLService documentation for information on how to do this.  (BlueSSLService is the SSL library used by PostgresClientKit.)

Thanks for the update! I'll give v1.1.0 a try. Hopefully that will cure my issue.


First note in every MIDI file is skipped

The first note (or chord sometimes) in a song is always skipped by the program. Editing the .mid file manually to add a single random note causes it to skip that note and play the rest of the song normally.

You're right, was a bad comparison...  Should have been
        // Find the next note to play
        size_t nextIndex = 0;
        for (; nextIndex < songData.size(); nextIndex++) {
            if (songData[nextIndex].timestamp >= currentProgress) {
                break;
            }
        }

@KevAquila you forgot to close this issue, it appears to be resolved in commit: https://github.com/KevAquila/WEBFISHING-Guitar-Player/commit/61ff4fc30685da511555aea89180c07482a2b682


enrich the build scripts: support deploy local on mac

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

Note to use python 3.9 on Mac to test the basic functionality.
Including some networkx test cases as well.

@lidongze0629 Could you please add a test case, which run some GAE apps, as well as networkx apps on local environment?


Mob sounds independent of mob location respective to player

Describe the Bug
I'm unsure of how to really describe this, but I feel like the title is pretty accurate. Essentially, the sounds of all of the mobs always sound like they're directly next to the player, irrespective of whether they truly are or not. This makes for both a very loud and disorienting experience when exploring.
I recognize the video isn't really the best example, as you can hear blazes, but I think it gets my point across.
Reproduction Steps

Wait for mobs to make noise(?)

Expected Result
Sounds to be effected by mob position relative to the player
Screenshots and Videos
https://user-images.githubusercontent.com/116246905/205516476-c8de10b4-305d-41c0-a654-8220b8bf5550.mp4
Operating System
Windows 10
Mod Version
2.5.1
Minecraft Version
1.19
Forge Version
43.1.52
Other Mods
No response
Additional Context
No response

Believe I've found the cause for this - the sound files may be in stereo rather than mono. I'll try converting them myself to see if that fixes it, and if it does I'll upload them here.

There's a lot of sound files and I think this is probably the issue too. I have a script that will convert them all for me that we can run to fix this, so don't worry about converting them all yourself! If you want to test one or two to guarantee this is the issue, that would be great! But don't worry about doing them all as I can do that in a matter of seconds.


Fix errors in examples

Description
This fixes some small issues in the sample code.
Tests

No tests were harmed in the making of this PR. (Just doc changes that fix sample code analyzer issues).

test-exempt: only affects comments


Better config error

The previous error doesn't indicate how to fix.

Codecov Report

Merging #517 (a110ea9) into main (d348c10) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #517   +/-   ##
=======================================
  Coverage   20.71%   20.71%
=======================================
  Files          49       49
  Lines        5890     5890
=======================================
  Hits         1220     1220
  Misses       4445     4445
  Partials      225      225

Impacted Files
Coverage Δ

config/config.go
26.72% <0.00%> (ø)


ColorConverter is missing...

Hello,
I noticed that ColorConverter, that is part of the System.Windows.Media, is missing in Avalon.Media
here is the definition from Microsoft documentation
https://docs.microsoft.com/en-us/dotnet/api/system.windows.media.colorconverter.convertfromstring?view=net-5.0
This is planned to be added or I can PR this feature?
Paulo Aboim Pinto

I guess Color.Parse is an equivalent in this case?

Yes .. it is ...
when we have this code in C# WPF code, now we have to refactor all those lines ...

now we have to refactor all those lines ...

You can add your own ColorConverter class that would call Color.Parse


Support for "ComparisonChain"

Hi, great job, sir! :)
I often include Guava in my projects, but I always end up using like 1% of it (mostly collections and predicates). The problem on Android is it's huge methods footprint, which I guess your lib is trying to solve.
I have only one request. Could you add class ComparisonChain? It has almost none dependencies, shouldn't be a problem :)

Hey @rafakob thanks for the input here. I do wanna carefully pick what to add to the library since It might loose its main feature which is to stay tiny :). (I do not wanna end up with a big library)
Could you provide more context/information or use cases? Specially for android.

You meant use cases of other parts of Guava or just that ComparisonChain?
I like to use it with Ordering class, eg:
private List<User> orderData (List <User>list) {
    return new Ordering<User>() {
        @Override
        public int compare(User left, User right) {
            return ComparisonChain.start()
                    .compare(left.age(), right.age())
                    .compare(left.getLastname(), right.getLastname())
                    .compare(left.getFirstname(), right.getFirstname())
                    .result();
        }
    }.sortedCopy(list);
}

@rafakob +1

Will keep this in mind and label it as a feature request.


User_info() Retrive Email and Phone number

Get the email and phone number from IG account if available in the contact menu (near Follow and DM section of a profile)
Thanks!

Implemented in 1.7.1


Using regexes in alias names

Hey there,
I would like to request the option to use regexes for alias names. This could make matching window names easier/more future proof for updates of said window names.
An example:
Currently gimp has the window name gimp-2.10, now if gimp has an update to v2.11 I will need to update my config file. I would rather like to define my alias as a regex, so that I can use gimp-\d\.\d\d or ^gimp.
I foresee a few breaking changes/issues with this, being:

Previous aliases such as "Org.gnome.Nautilus" need to be changed to "Org\.gnome\.Nautilus", as the . otherwise means any character, but it will still work.
Plainly using substring will also match against somethingsubstring-something, whilst it did not before. This should be changed to ^substring$ to obtain the current behavior.

Hi again, wanted to let you know I've made an initial implementation for alias regex matching.
the syntax is a little different from what you proposed due to string escapes, for for example the gimp pattern would look like this in the config:
"Gimp-\\d\\.\\d\\d" = "Gimp"

I decided against having regex as a toggle, since the substring matching works 99% of the time. I'll have a notice in the the new draft if this gets merged.
Bear in mind that this update builds on the 1.4.0 branch which has changed no_names option behavior, and introduce a new option called no_icon_names which behaves like the previous no_names (sorry about the confusion here but I thoughtlessly based the rexex impl on that branch)
try out the regex branch if you got the time. Thanks.

This is added in the latest version v2.0.0

Sorry, I have been too busy with school to actually be able to read your message and write some code, but thank you so much for implementing it!


Reduce string allocation on Sampler access

ForwardGlobals::create_bind_group creates a BindGroupEntry Sampler resource by fetching it from the asset cache. This incurs a string allocation with a ~0.5ms per call overhead on the web.

Is this a specific problem with just samplers, or should we consider remaking the asset cache to not use a stringly typed, non type-safe api (with everything being boxed, and different types that have the same debug impl will collide)?
If you use newtypes in the cache that end up having a debug-impl that only prints the inner type, or any other identical debug impl, the asset cache will cause it to collide with the inner type in the cache, and panic when downcasting.

Oh I see where it is from now, and I know how that system works. There is a self in that function that we can stuff the sampler into


defaultTest property gets deleted from .yaml file when it is edited through the web ui

https://promptfoo.dev/docs/configuration/guide/#default-test-cases
when I try to add this property to the config file in the UI via the "edit yaml" button, it just disappears after clicking "save"

Thanks for catching, fixed in #629

Thanks for a quick fix! I wanted to try multi-attempt evaluations, but seems like "evaluateOptions" also disappears from the config if it is edited in the UI  https://promptfoo.dev/docs/configuration/reference#config

added this one too :+1:

Change is released in 0.51.0

Hi, I checked out the updated version and these options are indeed in the UI, but have no effect and are ignored during the evalutation.

Hi, I checked out the updated version and these options are indeed in the UI, but have no effect and are ignored during the evalutation.

#829


Gdrive is amazing and just works

Hi,
I'd just wanted to say thank you, gdrive is amazing and just works (file upload, Linux), it's extremely simple to setup, I went from discovering it to use it in less than 5 minutes.
Thanks!

totally an issue

no really as someone who has a couple repos this is really kinda annoying

:)


correct comment

TransactionalMessageBridge#addRemoveTagInTransactionOp has a error comment

Coverage increased (+0.1%) to 50.622% when pulling ca9afd0a43397b479001a48d5f01b4ba0a1f8c16 on rushsky518:develop into 44569e4df6620fff61e43d782815f93a97e748ea on apache:develop.


Showing differences to create the more generic BM DHO compared to single DHO

DO NOT MERGE

Converting to draft to prevent any accidental merging


Generalise particle.xi etc for FieldSet with multiple grid resolutions

Currently, a Particle has only one xi, yi and zi, based on its grid index in the Fieldset.U grid. However, in principle some Fields could have other grids than the one for U. There is currently no way to define these different xi etc.
This omission significantly slows down Parcels in JIT mode, since index finding takes a long time if it constantly has to be done on different grids

Fixed in #241


Doom sync/build consumes all system resources

What did you expect to happen?
I should be able to put the process in background with Ctrl-Z and resume when I go to bed because the system is unusable while it is "Natively compiling 1820 files..."
I suggest you either make it use just one CPU per default, or at least make it suspendable. Please excuse the tone but it's truly infuriating to have my workstation disabled for 1+ hours. Thank you
What actually happened?
System un-responsive for 1+ hour
Describe your attempts to resolve the issue
Ctrl+Z
Steps to reproduce
doom build
System Information
https://pastebin.com/ytdbbRun

You can change how many cores are used by setting the NUMBER_OF_PROCESSES envvar. e.g.
NUMBER_OF_PROCESSORS=1 doom sync

Or by throwing this into ~/.doom.d/init.el:
(setq native-comp-async-jobs-number 1)

I'm working on a rewrite of the CLI as part of #4273, which will introduce some switches to control it in the future. As such as I'll consider this a duplicate of that issue and close this one.