No possibility to check already installed tools

What version?
0.17.0
Which command?
No response
What happened?
Whether it's proto bin, proto list or any other command, there's always a need to provide either id of a package or both id and version. I'd love to have some command such as just proto list to see what tools I have already installed.
Any logs?
No response
Operating system?
Windows
Architecture?
x64

Will add in the next release.

Added proto tools in v0.18.


Stream to usb directly

Hi everyone,
I’ve seen there’s an interesting project called serialwled and since I have lots of interference in my network I wanted to know if there’s a way that I can plug the esp32 directly to the tv usb instead of using the network.
Thanks!

Yes, you can do this with HyperSerial from AWAWA see: https://github.com/awawa-dev/HyperSerialESP32.
Depending on which ESP you use, you may need to install the necessary driver on your LG. Drivers for each TV architecture can be found on this page: https://github.com/throwaway96/webos-kernel-drivers

I'm using a Lightberry, which is basically an Arduino Leonardo, directly connected to the TV USB port.
Works like a charm with 218 APA102 LEDs.
Lightberry is flashed with code from the Hyperion github, using Fastled.
No any extra drivers needed.


[scss] timing functions in variables break syntax highlighting

VSCode Version: 1.5.2

The snippet:
// OTHER

$ease: cubic-bezier(0.4, 0, 0.2, 1);
$ease-out: cubic-bezier(0, 0, 0.2, 1);
$ease-in: cubic-bezier(0.4, 0, 1, 1);

IMO, also related to:

Most of the issues reported got fixed in the meantime. I created https://github.com/atom/language-sass/issues/196 for the remaining issue with fractional numbers.


Update AWS Go SDK to 1.31.3

What this PR does / why we need it:
Updates AWS Go SDK to 1.31.3 to support Route53 in AWS China Region.
See: https://aws.amazon.com/about-aws/whats-new/2020/05/amazon-route-53-is-now-available-in-AWS-china-region/
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Updates AWS Go SDK to 1.31.3 to support Route53 in AWS China Region

/ok-to-test

/kind cleanup

/cherry-pick release-0.15

/retest

Given this rolls a dependency forward by 7 minor revisions, as well as quite a few other dependencies as part of that, I don't think this is a good candidate for backporting (it's not a bugfix, we never supported AWS china in older releases so we aren't patching broken functionality).
Thanks for the PR 😄
/lgtm
/approve
/milestone v0.16


{bp-15120} libc/chdir: chdir/fchdir should not depend on environment variables

Summary
This PR will still allow basic shell operations such as cd/ls/pwd to be used even when the environment is disabled.
Impact
RELEASE
Testing
CI

[Experimental Bot, please feedback here]
No, this PR summary does not meet the NuttX requirements.  It's missing crucial information.
Here's why:

Insufficient Detail in Summary: While it states the what, it lacks the why and how.  Why is this change necessary? Was there a bug? Is it a performance improvement?  How does it achieve this functionality? What code was modified?
Missing Issue References:  Are there related issues in either the NuttX or NuttX-apps repositories?  These should be linked.
Incomplete Impact Assessment:  "RELEASE" is not sufficient.  Each impact category needs a YES/NO answer and a description if YES.  For example, even if the user impact is minimal, you still need to state "Impact on user: NO".  Consider the impact on the build system, hardware, documentation, security, and compatibility.
Insufficient Testing Information: "CI" is not enough. While CI testing is important,  you need to provide specifics about your local testing. What host did you build on? What target hardware or simulator did you test on?  The template specifically requests OS, CPU, compiler, architecture, board, and configuration details.  Critically, the "before" and "after" log sections are empty.  You must demonstrate that the change works and include evidence.

Example of a better Summary:
This PR fixes a regression introduced in [commit hash or issue #] where basic shell commands (cd, ls, pwd) were unavailable when the user environment was disabled.  This change modifies the shell initialization logic in apps/examples/nsh/nsh_main.c to ensure these commands are always registered, regardless of environment settings. This resolves the issue by directly calling the built-in command handlers instead of relying on environment lookups.
This would then be followed by properly filled-out Impact and Testing sections.


Collapse all SubCells when it's root cell got collapse.

Hello,
When I expand Computers ,it shows computer1 and when expanding computer1 it shows notebook1.
Then collapsing computers and again expanding computer it expand as it's previous stage.
I want to collapse all the subcells. How can I do this?
Please help me.

Hi,
You have to set collapsesChildRowsWhenRowCollapses property of the RATreeView to YES

Closing. Feel free to reopen if necessary.


How to use with Meteor?

Hi guys,
I'm using Meteor and deploy to Ububtu 14 via mupx which also using docker and try to hook N|Solid in but I'm going no where. (yes I'm noob)
Any hint?

Hi @katopz
The main issue is that Meteor is built using Node.js version 0.10.x and N|Solid extends the latest version of Node.js. Right now using Node.js version 4.2.4.
Let us know if you have more questions about this topic.

Hi @cronopio
Let's say if someone magically can make a use of 4.x versions
What's my next step?
Thanks

Hey @katopz,
That sounds really cool! Unfortunately, the NodeSource Node.js Docker Images were not designed with this use case in mind.
While I would be interested in hearing about your success with using/modifying these images for Mupx, these were designed to be deployed directly on the Docker Engine. This may better be posed as a question for the Meteor or Mupx teams in using third-party Docker images with their tooling.


[Feature-16396][storage-plugin] Tencent Cloud COS Storage Plugin

Purpose of the pull request
This pull request adds a Storage Plugin to the DS Resource Center to support Tencent Cloud COS and aims to close #16396 .

#16396

Brief change log

Add dolphinscheduler-storage-cos in dolphinscheduler-storage-plugin
Add qcloud cos_api to pom.xml files

Verify this pull request

Pull Request Notice
Pull Request Notice
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

we should remove bcprov-jdk15on-1.69

Since we don't have any tencent cos support for integration testing. I will -1 on this until we can find a way to test against this in out UT, IT and E2E.

If COS  is merged, I will maintain the test related functions


Add XML resource

Add XML resource
Fix #371
Support XML resource as a source,condition,target
Required

[ ] https://github.com/TomWright/dasel/issues/175
[ ] Documentation

Test
To test this pull request, you can run the following commands:
  cp pkg/plugins/xml
  go test .

At this stage source and condition are working as expected but additional work is still needed for the target stage. Considering that I am mainly interested by the target at this time, I am keeping this pull request open.

Closing in favor of https://github.com/updatecli/updatecli/pull/823


Clr-signpost inside of clr-modal doesn't fit in the screen

Describe the bug
When a clr-signpost is being used in a clr-modal, the signpost overflows the screen and it’s bottom part cannot be seen.
How to reproduce
https://stackblitz.com/edit/clarity-modal-with-signpost

Steps to reproduce the behavior:

Have a component that opens up a clr-modal
Have a clr-signpost in that modal with long content
Observe that the bottom of the signpost is outside the visible part of the screen

Expected behavior
The whole signpost to be in the visible part of the screen. In the case when the content of the signpost is too big, it can be scrollable.
Versions

Angular: 8
Node: 12
Clarity: 2.4

Device:

Type: MacBook Pro
OS: macOS 10.15.7
Browser chrome
Version 86

An app could workaround this by adding CSS to make the content of the signpost scrollable, but from a UX perspective this is quite a large signpost which makes me wonder why it needs to be so large? Perhaps another pattern is better, like expanding content inline?

The smart popovers story, which would be a good solution candidate for this issue, was recently closed as something we prefer to implement in Clarity Core and avoid duplication of effort. https://github.com/vmware/clarity/issues/1757
Based on this and the proposed workaround, I'm going to close this one.


fix(block): make inner aware of title positions

Previously, when computing the inner rendering area of a block, all titles were assumed to be positioned at the top. If some or all titles were at the bottom, this caused various incorrect computations of the inner area, depending on which borders were set; no border at the bottom would lead to an area that's too big, no border at the top but at the bottom and all titles at the bottom would lead to an area that's too small.
Now, when computing the inner area, the positions of the titles are taken into account.
fix #658

add a test case where you cover the issue in #658

Good suggestion. I have amended the PR accordingly. Let me know if anything can be improved. :relaxed:

I have added an explanation of the bug and the fix to both the description of this PR and the commit message. Let me know should brevity be lacking, or if the level of detail could be improved upon (in either direction).
Also, I have reworked the part that @joshka had a nit about, without accepting the suggestion. If you prefer to have your suggestion exactly, I'll incorporate that instead.

Thanks for the shortened bug description as well as the improved code snippet. :+1: I've incorporated both and amended the PR accordingly.

Thanks for your contribution!


tls: support ed25519 for client cert digital signature

Seems that Envoy cannot verify client certificates signed with ed25519 in TLS1.3:
(from server hello)
Acceptable client certificate CA names
CN = localhost
Requested Signature Algorithms: ECDSA+SHA256:RSA-PSS+SHA256:RSA+SHA256:ECDSA+SHA384:RSA-PSS+SHA384:RSA+SHA384:RSA-PSS+SHA512:RSA+SHA512:RSA+SHA1
Shared Requested Signature Algorithms: ECDSA+SHA256:RSA-PSS+SHA256:RSA+SHA256:ECDSA+SHA384:RSA-PSS+SHA384:RSA+SHA384:RSA-PSS+SHA512:RSA+SHA512
Peer signature type: Ed25519
Server Temp Key: X25519, 253 bits

Here is the envoy debug log when testing with a ed25519 client cert.
[2023-02-24 15:02:14.384][11395773][debug][conn_handler] [source/extensions/listener_managers/listener_manager/active_tcp_listener.cc:147] [C0] new connection from <IP_ADDRESS>:50183
[2023-02-24 15:02:14.392][11395773][debug][connection] [source/extensions/transport_sockets/tls/ssl_socket.cc:233] [C0] remote address:<IP_ADDRESS>:50183,TLS error: 268435648:SSL routines:OPENSSL_internal:PEER_DID_NOT_RETURN_A_CERTIFICATE
[2023-02-24 15:02:14.392][11395773][debug][connection] [source/common/network/connection_impl.cc:656] [C0] remote close
[2023-02-24 15:02:14.392][11395773][debug][connection] [source/common/network/connection_impl.cc:250] [C0] closing socket: 0
[2023-02-24 15:02:14.392][11395773][debug][connection] [source/extensions/transport_sockets/tls/ssl_socket.cc:233] [C0] remote address:<IP_ADDRESS>:50183,TLS error: 268435648:SSL routines:OPENSSL_internal:PEER_DID_NOT_RETURN_A_CERTIFICATE
[2023-02-24 15:02:14.392][11395773][debug][conn_handler] [source/extensions/listener_managers/listener_manager/active_stream_listener_base.cc:120] [C0] adding to cleanup list

looking at  https://bugs.chromium.org/p/boringssl/issues/detail?id=530&q=Ed25519&can=1
->
https://github.com/grpc/grpc/issues/24252#issuecomment-909916002
->
https://github.com/google/boringssl/blob/master/ssl/extensions.cc#L437
->
https://github.com/google/boringssl/blob/master/ssl/extensions.cc#L482
it seems we might just need to configure  hs->config->verify_sigalgs
is that somewhere in envoy's code?

Good find. It's not called anywhere. Not sure if this needs to be another
knob in xDS.
On Fri, Mar 10, 2023 at 10:06 AM Laurent Demailly @.***>
wrote:

looking at
https://bugs.chromium.org/p/boringssl/issues/detail?id=530&q=Ed25519&can=1
->
grpc/grpc#24252 (comment)
https://github.com/grpc/grpc/issues/24252#issuecomment-909916002
->
https://github.com/google/boringssl/blob/master/ssl/extensions.cc#L437
->
https://github.com/google/boringssl/blob/master/ssl/extensions.cc#L482
it seems we might just need to configure hs->config->verify_sigalgs
is that somewhere in envoy's code?
—
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoy/issues/25727#issuecomment-1464183230,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACIYRRSYB6HJMLA5X7FSCLTW3NUSVANCNFSM6AAAAAAVEYQ2RE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Alain confirmed on a side build (forcing the constant in the defaults, pending actual config option) that it works so yes, it seems "just" a matter of making that list configurable (or tbh, I wouldn't mind either that boringssl included with envoy would just get patched)

here are some details ... I struggled a bit with  the envoy/boringssl integration. I first compiled bssl with cmake/ninja on main but envoy has 2 boringssl patches that got in the way. so i used the  main-with-bazel branch to create a boringssl.tar.gz archive.
The patch is simply ...
`✔ XF4P4GJTCF ~/src/envoy/boringssl-git remotes/origin/main-with-bazel > git diff
diff --git a/ssl/extensions.cc b/ssl/extensions.cc
index ba9236035..73f4820e4 100644
--- a/ssl/extensions.cc
+++ b/ssl/extensions.cc
@@ -436,6 +436,7 @@ bool tls1_check_group_id(const SSL_HANDSHAKE *hs, uint16_t group_id) {
// algorithms for verifying.
static const uint16_t kVerifySignatureAlgorithms[] = {
// List our preferred algorithms first.

SSL_SIGN_ED25519,
SSL_SIGN_ECDSA_SECP256R1_SHA256,
SSL_SIGN_RSA_PSS_RSAE_SHA256,
SSL_SIGN_RSA_PKCS1_SHA256,`

and in envoy  i referred to it as a file in bazel/repository_locations.bzl...
`diff --git a/bazel/repository_locations.bzl b/bazel/repository_locations.bzl
index 93d4e7fbbc..310a352710 100644
--- a/bazel/repository_locations.bzl
+++ b/bazel/repository_locations.bzl
@@ -109,10 +109,10 @@ REPOSITORY_LOCATIONS_SPEC = dict(
# 3. Find a commit in BoringSSL's "master-with-bazel" branch that merges <boringssl_revision>.
#
# chromium-105.0.5195.37 (linux/beta)

   version = "098695591f3a2665fccef83a3732ecfc99acdcdd",

   sha256 = "e141448cf6f686b6e9695f6b6459293fd602c8d51efe118a83106752cf7e1280",

   strip_prefix = "boringssl-{version}",

   urls = ["https://github.com/google/boringssl/archive/{version}.tar.gz"],

   version = "",

   sha256 = "e1934a2dc400eb3a6f33344d9bd35812e013b695a2734bed77a663170417ef9d",

   strip_prefix = "boringssl",

   urls = ["file:///Users/adazzi/src/envoy/boringssl.tar.gz"],`


Image vulnerability scanning updates have broken the Validate docker build

Checked for duplicates
Yes - I've already checked
🐛 Describe the bug
Validate builds have been failing since the docker vulnerability updates.
🕵 Expected behavior
I expected the build to work
📜 To Reproduce

Select one of the more recent build failures from https://github.com/NASA-PDS/validate/actions/workflows/unstable-cicd.yaml

🖥 Environment Info
GitHub Action env
📚 Version of Software Used
latest snapshot
⚙ Engineering Details
NOTE: This is blocking tagging of the repo. We can merge rollback of changes if we cannot find a solution quickly.
🎉 Integration & Test
No response

Whew! Wow, today I learned there's a reason the GitHub Action is called docker/build-push-action and not a separate build-action and push-action.
Going down the rabbit hole, I've learned that there's a difference between a Docker image and a Docker manifest file. When you do
docker image build --tag whatever .

you're building an image and loading it into your local Docker daemon. To do the same thing with docker buildx, you'd do
docker buildx build --load --tag whatever .

But you can't do that with multiplatform images:
$ docker buildx build --load --tag whatever --platform linux/arm64,linux/amd64 .
ERROR: docker exporter does not currently support exporting manifest lists

The difference is that there's really no such a thing as a "multiplatform image"; an image is always for a single platform. When you use --platform with two platforms linux/arm64,linux/amd64, you're creating three things:

An image for linux/arm64
An image for linux/amd64
A manifest list that points to the two images

The local Docker daemon has no idea what to do with "manifest lists" so it rejects it. A container registry does understand manifest lists, so the only thing you can do with a "multiplatform image" is push it:
docker buildx build --push --tag whatever --platform linux/arm64,linux/amd64 .  # Bye --load, hi --push!

Since GitHub Actions inherently supports multiplatform images, it's docker/build-push-action. You're expected to build and push to a container registry, not try any funny business by leaving the "image" in the local Docker daemon.
But we want to leave the "image" in the local Docker daemon so we can use the Grype scanning tool; that's why in the yaml we said:
uses: docker/build-push-action@v6
with:
    load: true
    push: false

I had assumed this would work fine with multiplatform images. Sadly, it does not.
What to do? Well there are several alternatives.
Use containerd
On Docker Desktop, you can use an enhancement to the local Docker daemon called containerd. Just check this box:

It has built-in support for local multiplatform images (which again are just multiple platform-specific images plus a manifest list).
The problem is that in GitHub Actions, containerd support is not yet official. You can make it happen with:
steps:
-
    name: Use containerd
    env:
        DOCKER_CLI_EXPERIMENTAL: enabled
    uses: crazy-max/ghaction-setup-docker@v2
    with:
        version: v24.0.6
        daemon-config: |
              {
                    "features": {
                      "containerd-snapshotter": true
                    }
              }

# rest of the workflow

This sets up Docker Community Edition (such as what Docker Desktop is based on) as the context for the workflow. The problem, though: do you trust the software from someone named @crazy-max? Don't answer that! After all, I'm @nutjob4life
Make a Single Platform Image for Scanning
Rather than build a multiplatform image from the onset, build a single platform image, scan it—and if it passes, then build-and-push a multiplatform image.
The problem with this is that sometimes issues are found specifically with dependencies on a specific platform. For example, an unbounded loop vulnerability might exist only on the arm64 version of libwhatever.so but not in the amd64. If you scan only the amd64 version of the image, you'd miss the problem in final published multiplatform image.
In this case, you can use the matrix feature of the GitHub Actions workflow to do the scans. However, this makes the stable-cicd.yaml and unstable-cicd.yaml files exceedingly complex.
Just Use Dependabot for Now
The work that Grype does with image scanning overlaps with what Dependabot does anyway. And Dependabot does a really nice job. Maybe we should just wait until containerd becomes mainstream.

@jordanpadams I think we can safely close this, right?

Closing


Add custom install location argument to installer

📝 Provide a description of the new feature
It'd be great if PowerToys installer executable provides an option to specify install path as a commandline argument. This is useful for package managers like Scoop which installs to user's home directory.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

Good request.  https://github.com/microsoft/PowerToys/wiki/Installer-arguments-for-exe does not currently support this

@enricogior should we mark this "help wanted" or something you want your team to take on?

@crutkas
it would be better to implement this ourselves.

@enricogior can I work on this?
We can add a new arguments to the bootstrapper --install_folder and pass the path to the INSTALLFOLDER property of the MSI.

@davidegiacometti
yes sure, thanks!

Fixed with 0.29, released today. https://github.com/microsoft/PowerToys/releases/tag/v0.29.0


ffmpeg call fails with Invalid argument

I'm using aaxtomp3 for a few years now and recently updated my Arch Linux's packages as well as aaxtomp3. Suddenly, aaxtomp3 fails for me with the following error:
$ AAXtoMP3 -a -s -t . SomeBook.aax

----Decoding---Some Book----deadbeef--
Source: SomeBook.aax
Total length: 13:16:05
[NULL @ 0x5604b5d28600] Requested output format 'm4a' is not a suitable output format
./Some Author/Some Book//Some Book.m4a: Invalid argument

I found one post on SO that contains a somewhat similar looking error. Looks like ffmpeg gets an invalid input argument (like the error says...).
Note: I also discovered that the variable output_directory is defined with a trailing /, but file paths are always defined by adding another / to it, like output_file="${output_directory}/${currentFileNameScheme}.${extension}". I removed the trailing / from the definitions of output_directory in my script but this wasn't the problem, as expected. I'd still suggest to remove it, because the only places where the variable is used, it should always work without the trailing / and it would be cleaner this way.

If you use -e:m4a instead of -a, is the output file in the expected format? (Extension and encoding)

I used -e:m4b instead, which works! Didn't check with -e:m4a. Is this intended behavior?


[BUG] flytekit complains invalid flyte config_file_path specified while the intention is not to have a config file

Describe the bug
When no flytekit.config is packed in an image, flytekit complains about Invalid flyte config_file_path </path/to/flytekit.config> specified. upon executing a task.
It could be users intentionally not wanting to use a flytekit.conf, so the warning is a bit more scary than it could be.
Expected behavior
More mild and less seemingly harmful warning message is logged.
Flyte component

[ ] Overall
[ ] Flyte Setup and Installation scripts
[ ] Flyte Documentation
[ ] Flyte communication (slack/email etc)
[ ] FlytePropeller
[ ] FlyteIDL (Flyte specification language)
[x] Flytekit (Python SDK)
[ ] FlyteAdmin (Control Plane service)
[ ] FlytePlugins
[ ] DataCatalog
[ ] FlyteStdlib (common libraries)
[ ] FlyteConsole (UI)
[ ] Other

To Reproduce

Create and package a workflow without providing a flytekit.conf file
Run the workflow and examine the logs

Screenshots
N/A
Environment
Flyte component

[ ] Sandbox (local or on one machine)
[ ] Cloud hosted

[ ] AWS
[ ] GCP
[ ] Azure

[ ] Baremetal
[ ] Other

Any environment
Additional context
No.

@honnix thank you for this change. Also, we are trying to get rid of the flytekit.conf at runtime completely

@honnix thank you for this change. Also, we are trying to get rid of the flytekit.conf at runtime completely

Oh I totally forgot about this issue. I think it can be closed. Sorry about that.

Oh I totally forgot about this issue. I think it can be closed. Sorry about that.


The xbox one developer mode activation app

The Xbox one developer mode activation app looks good but I haven't used it yet I'm looking forward to using it and seeing what I can come up with,  I'm not rich so I will see if I can afford to pay the fee for it. I really hope that I can afford it,,it looks so awesome I want to make lots of programs and stuff also to be able to help others and fix my Xbox one while I'm doing it first thing I want to do is check out grand theft auto V and see what is going on and why I don't get any trucks from armor guards and why when you invest in the stock market and it only allows for one added stock and why it disappears pretty much immediately also would like to see if I can fix pc's too. If I don't get it this fortnight I'll have to wait till next fortnight until I can afford to pay for it ,,I'd like make things that would probably blow your mind cause I've been playing around with sketchup and blender and unity,,unity is the one that they use for kerbal space program game and a heap of other games as well ,I'm not very good at blender and unity but I'm excellent with sketchup I can draw a 3D anything I tell a lie I'm not very good at doing people and animals they always come out wrong but I can do craft and machines really good if this page was big enough I would post a few pictures of my efforts on sketchup ,,, I really enjoy messing around with stuff I enjoy seeing how it all works and how to manage and operate said things like the DMA app (developer mode activation app) I can't wait until I can start messing around with it and see what I can do to change my Xbox ones attitude towards the game grand theft auto V I wonder what scripting language it uses I hope it's something easy like C# or C/# plus ,you can do those on a note pad on any computer or pc I hope I don't have to do reports on it and send them to wherever because I don't like doing reports constantly it wears me out,I just want to mess around for a while these days and learn at my own pace I don't think that what I've talked about here is asking for too much ,just one little program to help me get so stuff done on my laptop and Xbox one mainly debug the games that I play. So in summary all I'm asking for is the developer mode activation app and nothing else.I just want my game to run right gta (grand theft auto V) also I would love for it to stop signing me out while I play games on the Xbox one it only happens when I'm online when it signs me out but that's not the worst thing that happens to me,I play KSP(Kerbal space program) on my laptop but the game has gotten to big for my little pc and freezes up and stalls for like 1 to 5.5 hours at a time sometimes it crashes all of a sudden the game is about 11 gigabytes which isn't that big but it's rough on the 4 gigabyte ram and the 1.7 Ghz processor which in comparison to other pc's of today it's pretty small and weak . It takes so long for KSP to start up first off it takes 3 to 4 minutes just to load up the screen from a white screen to game screen takes about 30 to 40 minutes to load up the start menu takes a further 16 minutes to load up from the game saves menu to the main game itself where you start off in what is called the kerbal space center, where there is a heap of selections starting with the vab or the vehicle assembly building then you have the mission building the recruitment building and the sph or the space plane hanger and then there's the most important one which is the TS Or the Tracking station where all your other missions are just sitting there waiting for you to come back and complete the stuff that you started before ,you can run more than one mission at a time you can go outside the space ship walk around or fly around in low gravity with your jet pack or if you take a car to another planet or moon you can drive anywhere you want there's no size limit to how big you can build or how many missions you can have all depends on how big is your pc and if you got cool graphics card you can turn up the visuals for kerbal space program and it does look really good when you turn it up on the settings for clouds and dirt and words on the menu screen would be a lot clearer and easier to read you can also see further than if you don't turn on full graphics. If you're no good at math then this is not the game for you ,,,,,,,,,,it makes you think cause you don't get prebuilt craft in this game or not very often ,,the only prebuilt craft that you get is in the vehicle assembly building and the space plane hanger where you can try your luck at flying the some of the worst built craft in the game or you can try your hand at improving them but be careful that you don't screw it up worse than it was in the first place there's cheats if you feel the need for that kind of thing,and there's lots and lots of mods or add-ons everybody should be familiar with add-ons by now,,, there's literally hundreds of mods to change and to choose from every single one of them different from next you can take pictures and video of your efforts put them on Facebook or something like that it's not too bad I tell you these things cause it's an amazing game time taking genius game not impossible to play unless you have a craptop like me and then it makes it hard to play KSP I've played a lot of games in my life but kerbal space program is the one that sucked me in it's just such a wonderful game. I always forget my lights on my craft but always remember to put landing legs on it,,, satellite dishes everywhere tons of fuels made by a amateur and designed by a novice what could go wrong......

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

ID: cc064e49-5f24-ff80-d511-7f1947fc39da
Version Independent ID: b8ad47c2-8264-c232-167f-b9ee69d6dce1
Content: Xbox One Developer Mode activation - UWP app developer
Content Source: windows-apps-src/xbox-apps/devkit-activation.md
Product: windows
GitHub Login: @mtoepke
Microsoft Alias: scotmi

I want to be a developer but I have no idea what I'm doing

Thanks for your feedback, @awabast.


KeyError: 'request'

Python 3.9.1 (default, Dec 13 2020, 11:55:53)
from proxy_requests import ProxyRequests
url = "https://api.ipify.org/"
r = ProxyRequests(url)
print(r)

--------------------------------------------------
KeyError         Traceback (most recent call last)
<ipython-input-9-c6b8a6a9416b> in <module>
----> 1 print(r)

~/.local/lib/python3.9/site-packages/proxy_requests/proxy_requests.py in __str__(self)
    202
    203     def __str__(self):
--> 204         return str(self.rdata['request'])
    205
    206

KeyError: 'request'

Yes I can confirm this. Going to close this as it's the same issues as #20


Generate alerts

Add alert for forked topic
Update base mailer style
Update conversation/comment email
Add alert mailer and emails

Pull Request Test Coverage Report for Build 233

22 of 52 (42.31%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.1%) to 90.233%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app/mailers/comment_mailer.rb
0
1
0.0%

app/mailers/conversation_mailer.rb
0
1
0.0%

app/models/alert.rb
12
13
92.31%

app/mailers/alert_mailer.rb
0
27
0.0%

Totals

Change from base Build 231:
-1.1%

Covered Lines:
1626

Relevant Lines:
1802

💛  - Coveralls


Should set pltsql_implicit_transactions = false as default

Description
We have wrongly set default for pltsql_implicit_transactions = true for
Postgres 16 to pass the explicit value check.

Why it happened

define babelfishpg_tsql.ansi_defaults will set init value pltsql_implicit_transactions = true since the feature ansi_defaults.
When PG_INIT variable, ansi_defaults will be defined as true.
So during PG_INIT init value pltsql_implicit_transactions will be changed to true, then changed to false in define babelfishpg_tsql.implicit_transactions.
PG 16 add extra check of the DefineCustomBoolVariable function to make sure the init value is the same as the default value, so it crashed the babel extension creating.
Our last attempt to fix this issue rudely set pltsql_implicit_transactions = true at first place to bypass the problem and it'll cause other potential problems during runtime.

Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 71.704%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
39366

Relevant Lines:
54901

💛  - Coveralls


Delete should respect ETag

Currently you can delete an entity without an ETag

This is also part of #16

fixed in #29


For each

Passed all of the tests with the exception of test 7. It says challenge 7 was skipped. And though my array is identical to the result, it fails.

All of the challenges pass! Each challenge is worth .83 points!


How to get the ID value of the content created in the transaction

Your Question
How to get the ID value of the content created in the transaction
The document you expected this should be explained
github.com/jinzhu/gorm v1.9.11
Expected answer

not a valid gorm question


Another fail with Dockerfile build

Having a Dockerfile
FROM lukemathwalker/cargo-chef:latest-rust-1.58.0 as chef
WORKDIR /app

FROM chef AS planner
COPY . .
RUN cargo chef prepare --recipe-path recipe.json

...

It fails with
 => [planner 1/2] COPY . .                                                                                43.3s
 => ERROR [planner 2/2] RUN cargo chef prepare --recipe-path recipe.json                                                                                0.6s
------
 > [planner 2/2] RUN cargo chef prepare --recipe-path recipe.json:
#13 0.536 thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/cargo-chef-0.1.33/src/skeleton/read.rs:56:30
#13 0.536 note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Same happens again if it's installed manually on rust:1.58-bullseye

Link to the repository?

Closing due to the lack of a minimal reproducible example.


The ESP-NOW frame length (AEGHB-98)

In ESP-NOW documentation is described the frame format is as:
------------------------------------------------------------------------------------------------------------
| MAC Header | Category Code | Organization Identifier | Random Values | Vendor Specific Content |   FCS   |
------------------------------------------------------------------------------------------------------------
  24 bytes         1 byte              3 bytes               4 bytes             7~255 bytes        4 bytes

and the Vendor Specific Content:
-------------------------------------------------------------------------------
| Element ID | Length | Organization Identifier | Type | Version |    Body    |
-------------------------------------------------------------------------------
    1 byte     1 byte            3 bytes         1 byte   1 byte   0~250 bytes

I'd like to ask how is possible that the body length is 250 B, but if the Body is full, Vendor Specific Content is 257 B.
Is a problem in my thought or the specification is wrong?
Link for documentation: https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/network/esp_now.html
The Reddit discusion: https://www.reddit.com/r/esp32/comments/11ruo1w/the_espnow_frame_length/

@petrkucerak Thanks for your feedback, the Body length should be 0~248 bytes. We'll update it ASAP.

@petrkucerak The document has update, thanks again.


Add package lock to gitignore

This package lock file is autogenerated by npm v5+. Unless we adopt a policy to commit it and manually unlock/update versions, we should add it to our .gitignore.

I think we should just add it to the repo itself. But it should be added to our .npmignore as well if they're not already.

We don't have yarn.lock in our .npmignore, which is almost the same thing. And I think new versions of npm will ignore it anyway:

One key detail about package-lock.json is that it cannot be published, and it will be ignored if found in any place other than the toplevel package. It shares a format with npm-shrinkwrap.json, which is essentially the same file, but allows publication. This is not recommended unless deploying a CLI tool or otherwise using the publication process for producing production packages. [source]


NPD doesn't notice OOMs in newer Linux kernels

Sample OOM message log part from 4.19.112:
Memory cgroup out of memory: Kill process 4015 (python) score 1923 or sacrifice child
Killed process 4015 (python) total-vm:32812kB, anon-rss:23588kB, file-rss:1684kB, shmem-rss:0kB

Sample OOM message log part from 5.4.49:
Memory cgroup out of memory: Killed process 4249 (python) total-vm:32812kB, anon-rss:23856kB, file-rss:1700kB, shmem-rss:0kB, UID:0 pgtables:112kB oom_score_adj:994

NPD parses kernel log file on the fly looking for its parts matching the regexes defined in kernel-monitor-filelog.json. The one of interest here is this one:
{
	"type": "temporary",
	"reason": "OOMKilling",
	"pattern": "Kill process \\d+ (.+) score \\d+ or sacrifice child\\nKilled process \\d+ (.+) total-vm:\\d+kB, anon-rss:\\d+kB, file-rss:\\d+kB.*"
}

This explains why the NPD manages to report an OOM in the former case and doesn't notice the OOM in the latter case (Kill process \\d+ (.+) score \\d+ or sacrifice child\\n part is absent).
I see three possible options to fix this:

get rid of Kill process \\d+ (.+) score \\d+ or sacrifice child\\n part of the OOM regex,
make this part of the regex optional,
create an additional rule in kernel-monitor-filelog.json to cover the new OOM logging format as well (probably would require adapting the first rule's regex pattern as well).

Third approach seems most sensible if feasible.
/cc @jkaniuk
/assign

Looks like the aforementioned Kill process %d (%s) score %u or sacrifice child\n OOM message part was removed in https://github.com/torvalds/linux/commit/bbbe48029720d2c6b6733f78d02571a281511adb kernel commit (5.1 release candidate).
The meaning of Kill process [PID] ([PROCESS_NAME]) score [SCORE] or sacrifice child message is saying that we look into the child processes of the [PID] process and heuristically try to find a worse (didn't go deep into what "bad" exactly means here) candidate to be OOM killed. As stated in the kernel commit, this was duplicating some work done beforehand because the worst process was already earlier selected as the victim.
Because of that, I'm currently leaning towards completely erasing that prefix from the OOM message regex.


Documentation Update or Wrapper preset import bug (@BenchTweakGaming)

Prerequisites

[X] Verify that this is not a Windows issue;
[X] Refer to the system requirements;
[X] Refer to the How to use;
[X] I do not use a homebrew Windows image;
[X] I did not tweak Windows before that could cause system instability;
[x] If your issue concerns the Wrapper, please mention @BenchTweakGaming in issue

Steps to reproduce
Using the wrapper application and importing a preset:
If the preset is NOT in the default Sophia Script folder (where the default Sophia.ps1 script resides), it won't load and instead generate a message about a new version of Sophia Script being found.
Moving the same script to that default folder, will result in proper preset loading and expected behavior.
Steps to Reproduce:
Start Wrapper - load default preset from sophia folder - (optional: make changes) - export preset to folder outside of the default - load that preset.
You will then see message about new version and pressing No/Cancel - results with empty preset (yes results in expected browser launch behavior but wrapper still "broken").
Move that same preset to the original default folder and import it.
No issues.
Note:  If it helps, last time I used this was on a Win10 machine with Sophia v5.14.3 and Wrapper v2.5.7 - with no issues importing from different folder.
Windows Version
Win 11 Pro 22H2 (22621.1555)
Sophia Script version
6.4.4

You cannot move preset to any folder excet the folder where all other files are. Because preset tries to import module from $PSScriptRoot folder.

I'm working 6.5.0 already. :)


Add button to show/hide globe

This PR addresses Issue #95.

I added a button that updates a boolean state to show/hide the globe. I put it next to the location button so it is easily visible. I made the globe hidden by default because the weather app was crashing for me otherwise.

Hey @gibbonhug please check the conflicts that need to be resolved


Broken link on yew.rs

This is about:

[ ] A typo
[ ] Innaccurate/misleading documentation (e.g. technically incorrect advice)
[ ] Undocumented code
[ ] Outdated documentation
[x] Other

Problem
The https://yew.rs/docs/en/ website's logo links to a nonexistant page
It links to https://yew.rs/en which yields a 404 about firebase having no index.html in /en nor a 404.html in /
I use up to date Firefox if that makes a difference, although I doubt that.

This issue also exists on the other languages

Thanks for pointing this out!


DOC fix text on null hypothesis in quantile example

@m-maggi Do you want to review this one?

@lorentzenchr I am new to code reviews on github, do you see my comment? I see that with a „pending“ label.

At the top right, there is a green button "review changes". If you click, you can select "comment", "approve" and "request changes". I usually only use one of the first 2 choices.


implement DeletebyQuery

When deleting documents that are indexed using routing, it is necessary to use routing for the deletion as well. Since the DCP deletion event only provides the document key, workaround solutions are needed to find the routing value.
We can also perform the deletion using 'Delete by query'. Can we implement 'Delete by query'?"

Delete by query is mostly a search operation. We can give freedom, but this brings problems such as conflict/refresh to handle. Because of this problems, personally i did not want it on connector but if it is really necessary, i will follow.
I want to ask, why we cannot regenerate this routing key from event? for your routing problem.

We discuss internally with @enesyalinkaya and decited to forward with fetch and delete solution.
Because of that we give freedom to access elastic client from mapper to user with this implementation.


Philips TV Codes 26PFL560H

This is actually an information or feedback about the "smartir Philips TV"
Home Assistant version
0.109.3
SmartIR version
1.9.0
It  works with:
Philips Model 26PFL560H and the Philips IR codes work as follows:
On / Off
Volume up / Volume down
Input Selctor:
well, except that the names provided don't match the selectd input, but all can be selected and eventually you get where you want to (;-))
Thanks for the great work!

You can override the source names see here.
Thank you

Thank you,
I found this already.
Such a cool integration you did here!
I love it,
needs to become a standard integration for hasss.io (;-))
Cheers
Dietger


android 4.4  send file to gmail，to xxx has file path

when  i share a  file in android 4.4 ,i found this problem.

this file path was put in to : xxxx。
then，this is my code
$cordovaSocialSharing.share(
null,
null,
fileDirectory + "/" + document.name,
null,
sharesucc,
failuse);
what's wrong with me???

by the way,  Native email app has the same problem


[Discussion] Self Host?

It would be pretty awesome if fuse-box could be self hosted.
And by that I mean be able to bundle/build the fuse-box project.
Thoughts?

It's actually possible, and already happening when you run node test
But why would one want to have it?

If a project uses fuse-box as a dependency, being able to bundle that project
eg. nexe

But as you can probably guess I'm running into a problem with Config.
Would accept a PR that moved Config to the FuseBox instance level and shared it using the
WorkflowContext?

@calebboyd there are millions of references to the context, across the code.... You can try of course, but push to a different branch


[RFE] expose port/path on rancher agent to check its health

Enhancement request to allow for the ability to monitor rancher agent health/liveness. The use case being: having an agent in an autoscaling group, where if the agent stops responding, we want the host to be recycled.

I think this is an important feature. We need a way to monitor if the rancher/agent:latest container is up and running. If is not, we spin a new EC2 custom instance and destroy the old one.

I’ve seen instances where the Docker daemon dies too and prevents containers from properly being scheduled on a host. Having this in place to signal a new machine is needed would greatly help!

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


btn-sm rendering (line-height?) issue on Android Chrome

It seems like there is a line-height type rendering issue with text within the btn-sm buttons. The text (to my eye, at least) doesn't appear centered vertically within the button. This issue does not appear on any other button sizes, nor does it appear on desktop. First noticed on my own project but have noticed the same on the Bootstrap website itself. See image below from https://getbootstrap.com/docs/4.0/components/buttons/
OS: Android 8.0.0
Browser: Chrome 66.0.3359.158
Device: Moto G6 Plus

I could not find the source of the problem on the Bootstrap website using the inspector, however I suspect it is something to do with the native font stack, perhaps different base line heights across different operating systems.
In my own project the issue was down to a webfont I was using. I suspect the built-in line height of that webfont was irregular which caused the issue. I've ended up reverting to the native font stack for all but heading tags/classes.

I can see this issue too on a Nexus 7 (2013) with Android 7.0 and Chrome 67.


Add Model diagrams and description to DG

Modified UML class diagrams for the Model component, as well as the accompanying descriptions, to reflect the changes due to the split of Person into Tutor and Student.

Codecov Report

Merging #104 (54d00d7) into master (c2776da) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #104   +/-   ##
=========================================
  Coverage     75.64%   75.64%
  Complexity      582      582
=========================================
  Files            88       88
  Lines          1823     1823
  Branches        209      209
=========================================
  Hits           1379     1379
  Misses          392      392
  Partials         52       52

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 c2776da...54d00d7. Read the comment docs.


Dark Mode for Login page

This issue depends on #368 being completed.
When Dark Mode is on - the login page should look like this: https://mongodb.invisionapp.com/share/PCYZ6ZW7FR8#/screens/433591777_Log_In
Color key for light mode: https://mongodb.invisionapp.com/share/PCYZ6ZW7FR8#/screens/433591768_Color_Key_-Light_Mode
Color key for dark mode: https://mongodb.invisionapp.com/share/PCYZ6ZW7FR8#/screens/433591769_Color_Key-Dark_Mode
Component Color specs for light mode: https://mongodb.invisionapp.com/share/PCYZ6ZW7FR8#/screens/433591770_Component_Colors-Light_Mode
Component Color specs for dark mode: https://mongodb.invisionapp.com/share/PCYZ6ZW7FR8#/screens/433591771_Component_Colors-_Dark_Mode
Both dark and light modes should follow the color key and specs above (it is possible that the current light mode is not).
Please make sure to TEST your build on a device or emulator to verify everything has changed - you can get a login and realm-app-id with easy instructions at https://wildaid.github.io/contribute/sandbox.html

@o-fish-wildaid can I do this?

PR: https://github.com/WildAid/o-fish-android/pull/387

Hi @ardlank I've assigned the issue to you. Welcome to the project!


suggestion - improve documentation

It doesn't explain what to do, just what to put in the configuration. Once you have set the configuration, what do you have to do?
No error appears in the logs, but the remote does not appear as a device or entity.
On the other hand, how can I know the options for smartIR. In the config only this appears:
device_code:1066
controller_data:remote.orvibo_remote_xxxxxxxxxxxx

where do I get this data?

Good morning
I ask myself exactly the question! Is there a way to have a little more details on the configuration?


Build conductor server 2.28 with ES6, start up error

Hi All,
I am trying to build a conductor server in version 2.28 with es6, but there is a start error in my way, the error is "Caused by: java.lang.ClassNotFoundException: conductor.org.apache.logging.log4j.core.config.Configurator".
My build command in maxOS is ./gradlew clean configureRelocationShadowJar shadowJar -x test -Prelease.version=1.0.0
Is there something i missing?
By the way, I had done with the guideline in https://github.com/Netflix/conductor/tree/master/es6-persistence#build

Copying my response from Gitter:
Hey @DivineDm, any particular reason to use Conductor 1.28 over the latest available 2.x deployments?

@kishorebanala sorry, typo my bad! My forked conductor version is 2.28, also updated in gitter.  By the way, I had edited the build.gradle, and run with es6 successfully now, but I'm not sure what i did is the best practice

@DivineDm No worries. Can you provide more details about changes in build.gradle that has fixed your error please.

@kishorebanala I had rollback the build.gradle to conductor 2.26, and it works

@DivineDm Please feel free to reopen if you see this issue again.


SUSE dotnet 2.0.3 Description: Infinite recursion during resource lookup within System.Private.CoreLib.

SUSE dotnet 2.0.3 Description: Infinite recursion during resource lookup within System.Private.CoreLib. This may be a bug in System.Private.CoreLib, or potentially in certain extensibility points such as assembly resolve events or CultureInfo names. Resource name: ArgumentNull_Generic
https://github.com/dotnet/corefx/issues/30208

Issue moved to dotnet/coreclr #18446 via ZenHub

Peter, let me know if i can collect some trace and some core dump files.
Are the core dumps useful in this case? SLES 12 SP2 x86_64

Let's use the issue I created in the coreclr repo to dig deeper. Yes, getting a dump when the problem occurs would be helpful or a sample that repros the problem.


Optimize assertType function

When hierarchical object comes, typescript-json becomes slower than typescript-is.
Find the reason why and optimize it.

It's for safety. Therefore, would not.


Fix disabled state for debug toolbar icons (#149172)

This PR fixes #20941
This issue has been fixed in vscode, cherry-pick the commit to ADS, original PR: https://github.com/microsoft/vscode/pull/149172
before:

after:

In ADS Stable builds, opacity is 0.4

@cheenamalhotra yes, it was discussed in the original PR: https://github.com/microsoft/vscode/pull/149172

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.002%) to 41.963%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-bindings/src/common/azureFunctionsUtils.ts
1
80.65%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
28858

Relevant Lines:
63878

💛  - Coveralls


ENH: Add begin() and end() to FixedArray, support range-based for-loops

Added begin(), end(), cbegin(), cend(), iterator, and
const_iterator to itk::FixedArray. Included GTest unit tests.
This commit allows iteration over a FixedArray by means of a
C++11 range-based for-loop.

Note that itk::FixedArray already has its Begin() and End() member functions, which should also be fine, but they just don't get picked by range-based for-loops, because of the way they are spelled.

Great idea!
Could you add rbegin and rend as well?
https://github.com/InsightSoftwareConsortium/ITK/blob/d6b48deea2ee9e7e6f006defa6823a136be056e6/Modules/Core/Common/include/itkFixedArray.hxx#L149-L191

FYI, the amend that I just did only extended the unit tests for this feature.

@phcerdan

Great idea!

Thanks, Pablo  :-)

Could you add rbegin and rend as well?

rbegin and rend might also be interesting, indeed, but I would prefer to have it in a separate pull request. Is that OK to you? My aim for this pull request (#757) was just to add range-based for-loop support. By keeping the PR small, I hope that it can be reviewed and merged easily.

Sure! Approved. Sorry for the brevity (answering from the phone)


Link not work!

Hello,
I apologize for the inconvenience. I'm currently working on fixing the link, and as soon as it's finished, I will make sure to keep you informed. Additionally, you can download the file and use it locally once the link is operational.
Thank you :)

Hello,
I apologize for the inconvenience. I'm currently working on fixing the link, and as soon as it's finished, I will make sure to keep you informed. Additionally, you can download the file and use it locally once the link is operational.
Thank you :)

Hello,
I apologize for the inconvenience. I'm currently working on fixing the link, and as soon as it's finished, I will make sure to keep you informed. Additionally, you can download the file and use it locally once the link is operational.
Thank you :)

Can u please explane me how to run locally the entire project? thaks

@MarKaliGG I have recently deployed my application on AWS, and you can now access it :)


Transforming the CNOT-Dihedral class into an Operator class

What is the expected behavior?
Transforming the CNOT-Dihedral class into an Operator class, inherits from BaseOperator.
Add the following missing methods:

[x] tensor
[x] expand
[x] conjugate
[x] transpose
[x] adjoint
[x] is_cnotdihedral
[x] synthesis of circuits on more than 2 qubits (the total algorithm run-time and circuit size of the output is O(n^3).)

close by #463.


Additional AppsAndFeaturesEntry fields for ARP Matching

Description of the new feature / enhancement
In the AppsAndFeaturesEntry, there are additional fields which could be used for ARP Matching if they were added to the manifest and were populated by package maintainers. Any available ARP fields could be added to the manifest to improve ARP Matching
Proposed technical implementation details
AppsAndFeaturesEntries:
  - DisplayName: # This already exists
    Publisher: # This already exists
    DisplayVersion: # This already exists
    ProductCode: # This already exists
    UpgradeCode: # This already exists
    InstallerType: # This already exists
    HelpLink: # This is new
    UrlInfoAbout: # This is new
    URLUpdateInfo: # This is new
    EstimatedSize: # This is new

@Trenly how are these related to matching? When would you see a user wanting to see these?

@Trenly how are these related to matching? When would you see a user wanting to see these?

When a user forks an open source application and begins to distribute their own version, especially with nullsoft based applications, there are times that the Display Name, Product Code, etc are left unchanged, but items like the HelpLink automatically change to show which repository compiled the application. This has a low (but non-zero) chance of overlapping with existing package ARP entries unless the additional fields are considered.
It would also be helpful to write some of these fields, such as UrlInfoAbout and HelpLink when installing portable apps.
Finally, some apps published by NVIDIA all have the same ProductCode and version, but differ by their EstimatedSize entry.
(Also, just for the sake of completeness)

Cool. Thanks for the clarification. I'm going to have to move this out, but I wanted to understand the context. We're going to bump the schema version to match the client version for 1.4. I'll be updating issues accordingly.


Feature/60 fluent bit

Issue #, if available:
Description of changes:
Adds aws-for-fluent-bit as an add-on.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

@kcoleman731 make lint gives 20 warnings (semicolon). I will autofix, but this setting should be active in your workspace.


Transform javascript into typescript

This is definitely very long issue, but we have to move into something more clean and stable

Is this an open-source project I want to contribute to for my first time?

Yeah, our framework is definitely open source, MIT License

could you give me a brief introduction to your project like what are the dependencies etc

So you want me to write your framework into ts using react or just ts I want to contribute but without specific information I can't do anything

framework is back-end, based on node, written in javascript. This specific issue is about to write it on typescript. It's like couple of files need to be re-written. Dependencies you can see in package.json


Missing Registries.checkNameAvailability()

The ACR name must be unique and this method can help with that. The method is already implemented in the corresponding inner object.

done in v1.4


No more use the broken sync-read-txn heed feature

We recently found out that the read-sync-txn heed feature was invalid and must be removed from this crate. We were declaring it in milli/meilisearch but, fortunately, not sharing the RoTxns across threads 😮💨
This PR removes the sync-read-txn heed feature form the Cargo.toml file. I will fix this in heed v0.20.0 and will fill a RustSec advisory in the meantime.

It is failing due to the removal of the sync-read-txn feature (obviously), which wasn't only implementing RoTxn: Sync but also making it possible to have multiple RoTxn on the same thread. This is why I renamed the flag and I will be releasing a new version of heed v0.12.7, which introduces a new valid and safe read-txn-no-tls feature implementing RoTxn: Send and making it possible to have multiple RoTxn on the same thread.


Rust implementation

I just made the Rust lang implementation. You might want to check it
Anime4K-rs
Can I make a pull request to add it under "Projects that use Anime4K" on README.md ?

It is done!

Thanks!


Session cannot be re-loaded

When I have two concurrent requests to my site from the same session, the session keys/values
are not updated until the server responses are fully sent.  One request therefore can not use the
other request's session key/value updates in a timely manner.  In my case, I usually have to reload
the site.
Is there any reason you don't update the session keys/values in real-time (so the second connection
could see them right away, even though the first connection is still getting service)?  I'm guessing
this is a remnant of locking and might give the second connection more stability if it tries to read
a session value just as the first connection is writing it.  Anyway, it would be nice to have an option
to allow faster communication between requests from the same session.

There's very little that can be done to make session work deterministically for concurrent requests, you'd hit race conditions everywhere. Most requests are quick enough you've have a hard time telling that they were concurrent. Do you have very slow concurrent requests to the point where it's obvious? What kind of information are you trying to share between them?

Yes, it's very clear.  I try to keep the connection for a minute, to see who else connects within that minute,
and then tell the original connecter.  You already have race conditions if two people connect and
update the session and disconnect all with identical timing anyway, so the question is what's tolerable.
For my case, you could just update the session value immediately (so that other connections could
immediately see it) and the race condition just occurs sooner.  I still don't see a serious problem, but
imagine a problematic test case could be made...so I was just hoping for the option to allow either
mode of operation.

Session does not sound like the right infrastructure for this kind of problem. Have you tried using SignalR?

Yes, I use SignalR also, and I do have alternative methods, but simply letting the session update
earlier is the more straight-forward.  I'd be willing to pick through the code and just allow the option
somehow (even just for a private version if no one wants it...thus asking here).

It's not as easy as it looks. Let's start with the impossible...
If the session hasn't been established yet then the concurrent requests do not share a session cookie and have no means of being correlated. You need at least one request to complete and set the session cookie on the client before any concurrent requests could be made.
Once you have that cookie you need to look at the lifecycle of session on a request. The whole session is loaded from the cache the first time anyone tries to read from it on a request. Note a session is stored as a whole unit rather than on a per-entry basis. The session can be pre-emptively loaded by calling LoadAsync().  For efficiency and consistency purposes the session is not stored back to the cache until the request is complete. You can also-preempt this by calling CommitAsync(). Even if you do that the other request would have no way of detecting a change was committed without continually calling LoadAsync(). All of these extra operations would severely impede the performance of your cache and application.

Maybe I'm seeing a bug with LoadAsync since I am doing exactly as you suggest in your last paragraph:
The session has already been established.  Then, I make two concurrent requests, with one starting a
few seconds before the other.  In my simplified test case, the responses take a minute simply using Sleep.  Before the sleep, I increment a session variable and call "await CommitAsync".  After the sleep, I
call "await LoadAsync" and then check that session variable, but the increment of the slightly-later
request is never seen by the slightly-earlier request.  Is that behavior expected?

You're right, it can't be re-loaded even after committed. Nobody has ever tried to do that before.
https://github.com/aspnet/Session/blob/49e6920a7caa096997432240777af609e78acb9d/src/Microsoft.AspNetCore.Session/DistributedSession.cs#L203
Manually reloading it from the ISessionStore doesn't look practical as you wouldn't have access to the sessionKey.

This issue was moved to aspnet/Home#2394


Question: It is possible get Current Loged User In DbContext

Hello, I build blazor server multi tenant application and I wants to use HasQueryFilter in entity framework DbContext for predefined filter with TenantId. But I cannot get Current Loged User in DbContext. I inject to DB Context AuthenticationStateProvider but I get error GetAuthenticationStateAsync was called before SetAuthenticationState.
It is possible use HasQueryFilter depend on Current Logged User in DB Context

There you can fidn solution!
https://docs.microsoft.com/en-us/answers/questions/318230/blazor-server-it-is-possible-get-current-loged-use.html


Case sensitive?

My organization uses a URL with 'OWA' in all caps instead of 'owa'.  Should the filter be able to catch that?

Unfortunately, Chrome apparently doesn't support that ( https://stackoverflow.com/questions/16993372/chrome-extension-case-insensitive-content-script-matches ). I am no longer (fortunately!) in a position that requires me to access an OWA server, so if someone wants to take over as maintainer, please let me know.
I just modified the extension to also look for OWA, though, and you can install it as a developer extension if you need it!

Extension updates should be published in next few minutes!

It works for me now, thanks!

Excellent - thanks for testing & letting me know!!


chore(flakiness): update flakiness format

This patch:

updates Flakiness Dashboard format to define version per-build
and to pass COMMIT information
drops the README.md generation - we'll move on to a designated flakiness
dashboard viewer

NOTE: Prior to landing this commit, I'll migrate all the current flakiness data to the format.

I think it bothers me more than the indentation bothers you, but I don’t
feel super strongly about it.
On Tue, Aug 6, 2019 at 3:25 PM Andrey Lushnikov<EMAIL_ADDRESS>wrote:

@aslushnikov commented on this pull request.
In test/test.js
https://github.com/GoogleChrome/puppeteer/pull/4808#discussion_r311301042
:

@@ -111,6 +111,8 @@ new Reporter(testRunner, {
showSlowTests: process.env.CI ? 5 : 0,
});

-utils.initializeFlakinessDashboardIfNeeded(testRunner);
-testRunner.run();
+(async() => {
I tried that initially, but didn't like extra indentation. Does this
bother you strongly?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/GoogleChrome/puppeteer/pull/4808?email_source=notifications&email_token=ABDI62PXH5TIRUOQ3UDVWBLQDH27HA5CNFSM4IJP3352YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCAYQBUI#discussion_r311301042,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABDI62JTHHQJYXYXKGD6PTLQDH27HANCNFSM4IJP335Q
.

I'll land this now and will discuss the reason for your disturbance offline


Provide an option to remove bad animations

About this issue

Briefly describe the issue
How can the issue be reproduced / sample code

Details
MarkdownBasicText contains a very bad default animation, please provide an option to remove this animation
see code here
https://github.com/mikepenz/multiplatform-markdown-renderer/blob/develop/multiplatform-markdown-renderer/src/commonMain/kotlin/com/mikepenz/markdown/compose/elements/MarkdownText.kt#L131
Checklist

[x] Searched for similar issues
[x] Checked out the sample application
[x] Read the README
[x] Checked out the CHANGELOG
[x] Read the FAQ
[x] Checked out the MIGRATION GUIDE

Thank you for bringing this up. A PR to address this feature request would be greatly appreciated!

@mikepenz sorry to interrupt. I know this has been closed and implemented, I just can't get the animation disabled. I am trying with this code:
At the bottom, I add animate=false.
CompositionLocalProvider(LocalImageTransformer provides CustomImageTransformer()) {
                                Markdown(
...

class CustomImageTransformer : ImageTransformer {

    @Composable
    override fun transform(link: String): ImageData? {
        return null
    }

    override fun placeholderConfig(density: Density, containerSize: Size, intrinsicImageSize: Size): PlaceholderConfig {
        return PlaceholderConfig(
            size = with(density) {
                if (containerSize.isUnspecified) {
                    Size(180f, 180f)
                } else if (intrinsicImageSize.isUnspecified) {
                    Size(containerSize.width.toSp().value, 180f)
                } else {
                    val width = minOf(intrinsicImageSize.width, containerSize.width)
                    val height = if (intrinsicImageSize.width < containerSize.width) {
                        intrinsicImageSize.height
                    } else {
                        (intrinsicImageSize.height * containerSize.width) / intrinsicImageSize.width
                    }
                    Size(width.toSp().value, height.toSp().value)
                }
            },
            animate = false
        )
    }
}

Besides that, I find API-wise, linking MarkdownText animations to ImageTransformer is not very intuitive name-wise, but I suppose it is done this way since the images are displayed inline. Maybe some MarkdownAnimationProvider.

@Dima-369 can you please create a new issue for this, so it's not being lost.
While animation was only part of the image loading at this time, I agree it might be long term more fitting to make it its own config.

can you please create a new issue for this, so it's not being lost.

Sure, I created https://github.com/mikepenz/multiplatform-markdown-renderer/issues/244


uv run doesn't think the current environment is compatible when it is [rye virtual package issue?]

uv 0.1.35
platform: linux (x86_64)

Problem: uv run always creates a new environment even if current should be compatible, in certain projects.

I know that uv run is not an official feature yet, still it's fun to try to see how it works.
How to Reproduce

git clone -b 0.5.5 https://github.com/bluss/pyproject-local-kernel
cd pyproject-local-kernel/tests/server-client/client
Inspect pyproject.toml and requirements files to see they only depend on ipykernel and jinja2.
uv venv --python 3.11.7    # it's the python I happened to use
uv pip install -r requirements.lock
uv run -v python

Expected: It uses the environment ./.venv in the current directory
Actual behaviour: DEBUG Using Python 3.11.7 environment at .uv/.tmpvVFPMZ/bin/python
Note: Using python 3.12.2 instead has the same result.
Note: Using uv pip install -r pyproject.toml instead of the requirements.lock file has the same result.
Note: If I remove the dependencies in the pyproject.toml and reinstall, it still fails. So it might be about the client "package" defined by the pyproject.toml itself?  That might be the cause, according to rye this is not a package at all, because it's marked virtual and should not be installed. (I don't know how uv is supposed to know that or if there's a rye agnostic way to configure this..)
In general I wanted here that if there was a complete "rye sync"'ed environment, it should be used as is by uv run if possible.

Thanks for giving it a try. This seems weird, I'd imagined this would work. I'll look into this.

Is this still an issue?

This seems to be fixed. Uv has developed a lot since then, anyway. In this particular case it doesn't use an emphemeral venv anymore, so that's good.


Add support on RCS-ui for Domestic scheduled payment consent details

Description

[ ] Add support to present the domestic scheduled payment consent details on RCS user interface Here
[ ] Add mock for domestic scheduled payment consent details here

Acceptance criteria

[ ] The consent page mock render properly with the consent details mocked.
[ ] The consent details page is render properly with all components need it.
[ ] The consent details (response from RCS-backend) is populated properly in the consent details page with no errors.

Hey team! Please add your planning poker estimate with ZenHub @andra-racovita @mariantiris


a37208a (master) panics in release mode

[user@host:~/projects/rust-bitcoin-spv]$ RUST_BACKTRACE=1 ./target/release/bitcoin_spv --deb test-mainnet.db
2018-11-12 16:32:54 INFO  [bitcoin_spv::database] working with memory database
2018-11-12 16:32:54 INFO  [bitcoin_spv::node] loading headers from database...
2018-11-12 16:32:54 INFO  [bitcoin_spv::node] reading headers and filters ...
2018-11-12 16:32:54 INFO  [bitcoin_spv::node] read 0 headers from the database
2018-11-12 16:32:54 INFO  [bitcoin_spv::dns] reaching out for DNS seed...
2018-11-12 16:32:54 INFO  [bitcoin_spv::dns] received 184 DNS seeds
2018-11-12 16:32:54 INFO  [bitcoin_spv::p2p] trying outgoing connect to <IP_ADDRESS>:8333 peer=0
2018-11-12 16:32:55 INFO  [bitcoin_spv::p2p] client /Bitcoin ABC:0.16.1(EB8.0)/ height: 530361 peer=0
2018-11-12 16:32:55 INFO  [bitcoin_spv::p2p] handshake peer=0
2018-11-12 16:32:56 INFO  [bitcoin_spv::node] received 2000 headers new tip=00000000dfd5d65c9d8561b4b8f60a63018fe3933ecb131fb37f905f87da951a from peer=0
2018-11-12 16:32:56 INFO  [bitcoin_spv::node] received 2000 headers new tip=00000000922e2aa9e84a474350a3555f49f06061fd49df50a9352f156692a842 from peer=0
2018-11-12 16:32:57 INFO  [bitcoin_spv::node] received 2000 headers new tip=00000000dbbb79792303bdd1c6c4d7ab9c21bba0667213c2eca955e11230c5a5 from peer=0
2018-11-12 16:32:57 INFO  [bitcoin_spv::node] received 2000 headers new tip=0000000094fbacdffec05aea9847000522a258c269ae37a74a818afb96fc27d9 from peer=0
2018-11-12 16:32:57 INFO  [bitcoin_spv::node] received 2000 headers new tip=0000000099c744455f58e6c6e98b671e1bf7f37346bfd4cf5d0274ad8ee660cb from peer=0
2018-11-12 16:32:57 INFO  [bitcoin_spv::node] received 2000 headers new tip=0000000011d1d9f1af3e1d038cebba251f933102dbe181d46a7966191b3299ee from peer=0
2018-11-12 16:32:58 INFO  [bitcoin_spv::node] received 2000 headers new tip=000000002d9050318ec8112057423e30b9570b39998aacd00ca648216525fce3 from peer=0
2018-11-12 16:32:58 INFO  [bitcoin_spv::node] received 2000 headers new tip=00000000679a1ab3af6da03f13a0bc96d7215e65458b2d2edfa030b5b431e8b3 from peer=0
2018-11-12 16:32:58 INFO  [bitcoin_spv::node] received 2000 headers new tip=00000000f914f0d0692e56bd06565ac4de668251b6a29fe0535d1e0031cfd0de from peer=0
2018-11-12 16:32:59 INFO  [bitcoin_spv::node] received 2000 headers new tip=00000000770ebe897270ca5f6d539d8afb4ea4f4e757761a34ca82e17207d886 from peer=0
2018-11-12 16:32:59 INFO  [bitcoin_spv::node] received 2000 headers new tip=000000004625a14242beccb38c63a1f770a76ee5788764e6c0abd4129bbc1b9d from peer=0
2018-11-12 16:32:59 INFO  [bitcoin_spv::node] received 2000 headers new tip=00000000f04fccc81f37002707e9501a3f7bdcf25f65531f386a2da8af20122e from peer=0
2018-11-12 16:32:59 INFO  [bitcoin_spv::node] received 2000 headers new tip=000000006d6c151db6d4d67356d590a897a11cd7d8111ee989de6f2f548410bf from peer=0
2018-11-12 16:33:00 INFO  [bitcoin_spv::node] received 2000 headers new tip=00000000172c5ed49d7dfc29bf9a18a53fa2d050fa37aa210d6d4080fd0c7e67 from peer=0
2018-11-12 16:33:00 INFO  [bitcoin_spv::node] received 2000 headers new tip=00000000de1250dc2df5cf4d877e055f338d6ed1ab504d5b71c097cdccd00e13 from peer=0
2018-11-12 16:33:00 INFO  [bitcoin_spv::node] received 2000 headers new tip=00000000049172ba3ec1b673cf13e3d0049c1c07bb103ed3fa300e3833480055 from peer=0
2018-11-12 16:33:01 INFO  [bitcoin_spv::node] received 2000 headers new tip=00000000495968d19210d3be15bd24fdc19805a0ef15026b0bb4482b04a9da3c from peer=0
2018-11-12 16:33:01 INFO  [bitcoin_spv::node] received 2000 headers new tip=0000000080c3deea35dc3df90a5fbe5f27db52f5e01018ae7d62f8b454c71335 from peer=0
2018-11-12 16:33:01 INFO  [bitcoin_spv::node] received 2000 headers new tip=000000002dfebce284d1e08b6cf04452530891579b7377669865889498de8f3f from peer=0
2018-11-12 16:33:01 INFO  [bitcoin_spv::node] received 2000 headers new tip=00000000504d5fa0ad2cb90af16052a4eb2aea70fa1cba653b90a4583c5193e4 from peer=0
2018-11-12 16:33:02 INFO  [bitcoin_spv::node] received 2000 headers new tip=000000000f80c09687893406279f62da437a6a0b95b8dc096b30c10ce088fc64 from peer=0
2018-11-12 16:33:02 INFO  [bitcoin_spv::node] received 2000 headers new tip=000000000122898b31073a770a97cf599c00672fc8d6ae15652235862f8b76d8 from peer=0
thread '<unnamed>' panicked at 'slice index starts at 6 but ends at 0', libcore/slice/mod.rs:2340:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
stack backtrace:
   0: std::sys::unix::backtrace::tracing::imp::unwind_backtrace
             at libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
   1: std::sys_common::backtrace::_print
             at libstd/sys_common/backtrace.rs:71
   2: std::panicking::default_hook::{{closure}}
             at libstd/sys_common/backtrace.rs:59
             at libstd/panicking.rs:211
   3: std::panicking::default_hook
             at libstd/panicking.rs:227
   4: std::panicking::rust_panic_with_hook
             at libstd/panicking.rs:476
   5: std::panicking::continue_panic_fmt
             at libstd/panicking.rs:390
   6: rust_begin_unwind
             at libstd/panicking.rs:325
   7: core::panicking::panic_fmt
             at libcore/panicking.rs:77
   8: core::slice::slice_index_order_fail
             at libcore/slice/mod.rs:2340
   9: hammersbald::memtable::MemTable::get
  10: hammersbald::bitcoin_support::BitcoinAdapter::fetch_header
  11: bitcoin_spv::database::DBTX::insert_header
  12: bitcoin_spv::node::Node::process
  13: bitcoin_spv::p2p::P2P::event_processor
  14: bitcoin_spv::p2p::P2P::run
  15: <futures_util::future::poll_fn::PollFn<F> as futures_core::future::Future>::poll
  16: futures_executor::thread_pool::PoolState::work

No plan for RPC. Instead it implements traits defined by rust-lightning. In contrast to c-lightning the goal is to have a single process level 1 (SPV) and 2 (Lightning) stack.


Android mWeb - IOU - Final review modal is not correctly displayed

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Log in to new.expensify.com on mobile web on Android device
Tap on FAB
Tap on Request money
Set an amount and continue
On user search, click on any user with the keyboard open.

Expected Result:
Final review modal should be correctly displayed.
Actual Result:
Final review modal is not displayed correctly.
Workaround:
None needed, visual issue.
Platform:
Where is this issue occurring?
Web
iOS
Android
Desktop App
Mobile Web (Android only) ✔
Version Number: 1.0.76-2
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
Issue is only reproducible on Android mWeb. iOS mWeb is working as expected.

https://user-images.githubusercontent.com/44479856/125125091-69b6d380-e0f9-11eb-9a31-4f3955051b41.mp4
Expensify/Expensify Issue URL:
View all open jobs on Upwork

From @rushatgabhane https://expensify.slack.com/archives/C01GTK53T8Q/p1625777131052000

When requesting money, open the keyboard and tap on a user.

Triggered auto assignment to @Luke9389 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

I was not able to reproduce this on my Google Pixel 5. What device is being used to test this?

I was not able to reproduce this on my Google Pixel 5. What device is being used to test this?

@Luke9389 Just reproduced it on Google Pixel 3.

hmm, interesting. I wonder how we set the height for this view. Perhaps it's the different screen size. I believe the Pixel 5 has a larger screen...

hmm, interesting. I wonder how we set the height for this view. Perhaps it's the different screen size. I believe the Pixel 5 has a larger screen...

@Luke9389 Yeah, Pixel 5 has slightly taller screen.
To reproduce the issue, just increase the height of your keyboard.

I'll be out of office all week next week, so I'll be able to return to this after that.

@Luke9389 Whoops! This issue is 2 days overdue. Let's get this updated quick!

@Luke9389 Eep! 4 days overdue now. Issues have feelings too...

@Luke9389 8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it!

Just getting back to this after being gone a week.
@rushatgabhane are you wanting to submit a proposal for this?

Also @isagoico would you mind retesting this to verify it's still happening, and also let us know which device you're using?

@Luke9389 No, I won't be submitting a proposal for this issue.
Btw, still reproducible on Pixel 3.

OK, I think we can slap the External label on this thing to get a proposal then.

Triggered auto assignment to @stephanieelliott (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@stephanieelliott, @Luke9389 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

Leaving an update here to remove the Overdue label, waiting for Stephanie to post this on Upwork I think.

Posted to Upwork: https://www.upwork.com/jobs/~01d14b1c783c4c0a1f

Can't reproduce it on Pixel 3 as well. But padding is off between input and List.

Can't reproduce it on Pixel 3 as well. But padding is off between input and List.

Try increasing keyboard height.
The simulator keyboard is short.

Try increasing keyboard height.

How?

I tested it on Extra Tall but still not reproducable.

I can't replicate the issue either 😂
I believe it got fixed because of #4215

@parasharrajat idk what im doing different. But I can replicate the issue.
Pixel 3 physical device, mobile web.
https://user-images.githubusercontent.com/29673073/129075632-266fdf7c-0214-4982-bc1c-724dc93c1bd4.mp4

Yea, I still haven't been able to produce this either. Maybe it's a difference in the Android version? @rushatgabhane do you know if you have a system update available?

@Luke9389
Nope, the phone is up-to-date. (Android 11)
Btw, Isabella can produce it. So it isn't just me.

And just to verify, which browser and version are you using?

Chrome 92.0.4515.131 (up-to-date from play store)

OK cool, thx!

Btw, Isabella can produce it. So it isn't just me.

I totally believe it's happening! Just trying to nail down why its only reproducible for some of us :)

@stephanieelliott, @Luke9389 Eep! 4 days overdue now. Issues have feelings too...

Gonna move this to Weekly while we try and nail down reproduction.

Triggered auto assignment to @bfitzexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

I'm taking an extended OOO, re-applied the External label to get another CM to take this the rest of the way, thanks @bfitzexpensify!

I don't have an Android so can't test this one out. Are you still seeing it as a reproducible issue @rushatgabhane and @isagoico?

I don't have an Android so can't test this one out. Are you still seeing it as a reproducible issue @rushatgabhane and @isagoico?

Yup, reproducible on Pixel 3, and Samsung J3 (2016).
For staging and production.

OK I have found the cause of this issue
https://github.com/Expensify/App/blob/e5957e1ffff7e111a459b05643bac18ffab20619/src/components/IOUConfirmationList.js#L335.
I can't even see the user

This MaxHeight crops the top list.
No worries, I have found the solution too which is better than the previous code. It allows more flexibility.
First, a little talk about the UI we want:

we want the top list to be flexible enough so that it grows when # of participants grows.
We want the Comment section to be always visible at the bottom with the button.
When there are less # of participants then we want comment section to be placed on the top of the screen as much as it can.

Fix:
I will use the flexbox to fix this instead of js.

First, move the comment section out from the Scrollview.
Then we should do flexShrink:0 for Footer.
Then add {flexGrow: 1, flexShrink: 0, flexBasis: 92} to the Comment input. Here 92 = height+ paddingVertical.
Then Change the scrollview style to {flexGrow: 0, flexShrink: 1, flexBasis: 'auto'}.
Finally remove the maxHeight from OptionsList.

This fixes the issue.

Ok @parasharrajat, you've got the Green 🟢 Light to start your PR. Thanks 👍

Please assign this issue to me.

@parasharrajat, hired you on upwork — will pay on the usual 7 day schedule after deploy to prod

🚀 Deployed to production by @roryabraham in version: 1.0.88-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

Seems like botify has marked the wrong date. PR has deployed to production 8 days back and payment should be released today. According to the bot, payment should be released tomorrow.

Agreed. Paid you out via Upwork @parasharrajat and ended the contract. Closing this one out.


Update representation of print version

LGTM :-)

Published<EMAIL_ADDRESS>Thanks! 🐨


Updated README.md

Added that you link in the local library instead of add/install.

This is obsolete now that it's published to NPM

Replaced with #76


add git developer OAuth client Id and secret in header when calling github api

@fazlerabby07 it looks unsafe as headers will be exposed in browser and header contains secret values. is it the best way to do?

@sifatul fix the headers when we call github api. now we convert the github clinet and secret into base64 string.