fix: remove colors from the the result to fix the local test

After looking into the issue, this is the only way to run the tests on our local.
In GitHub CI, the result comes without colors, but in our local, it's come with the colors, so I added a function to remove the colors only on our local because it does not exist already on the GitHub CI

Codecov Report

Merging #225 (3505319) into main (6af7859) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #225   +/-   ##
=======================================
  Coverage   57.04%   57.04%
=======================================
  Files          43       43
  Lines         610      610
  Branches      110      110
=======================================
  Hits          348      348
  Misses        247      247
  Partials       15       15

@jasonkuhrt Also, I worked in your comments here to go with your directive  https://github.com/prisma/nexus-prisma/pull/204/files#r855419395

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

npm package (@next-major dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


freebsd: not ok 213

Version:  git-v1.x  2018-1130
Platform:freebsd (GhostBSD 18.10)

# uname -a
FreeBSD damon.ghostbsd.home 12.0-CURRENT FreeBSD 12.0-CURRENT #0 7ad4fada6(HEAD): Thu Aug  2 11:37:15 EDT 2018     root@hydra:/usr/obj/root/jenkins/workspace/TrueOS_-_Stable_-_Complete/TrueOS_Stable/amd64.amd64/sys/GENERIC  amd64

ok 212 - spawn_reads_child_path
not ok 213 - spawn_setgid_fails
# exit code 6
# Output from process `spawn_setgid_fails`:
# Assertion failed in test/test-spawn.c on line 1446: r == UV_EPERM
ok 214 - spawn_setuid_fails

Can you find out what the value of r is?
Are you running the test as root? It'll try to setgid/setuid to user nobody if you do.

@bnoordhuis r is 0

Well, that's interesting. Do ambient privileges work differently on freebsd? The test tries to spawn a process with gid=0 and expects that to fail because it shouldn't have sufficient privileges to do so.
Is your user account (or user 'nobody' if you run as root) part of the root group?

That's expected. The tests aren't really designed to be run as root and you shouldn't, it's potentially dangerous.

@bnoordhuis is it safe to close this based on your last comment?

Thanks. I don't see anything wrong in the excerpt; the setgroups() call is expected to fail when running as non-root. Can you post the full output?

The problem, I figure, is that the call to setgid() succeeds, however the unit test (indirectly) assumes it will fail...
I looked a bit more into the GhostBSD vm and I believe the problem is related to how GhostBSD's installer configures the initial user account, that user's primary group is set to wheel (0); take a look at the output from id(1):
uid=1002(ptlomholt) gid=0(wheel) groups=0(wheel),5(operator)
The GhostBSD installer does not provide options to configure this differently so it must have been an informed decision from the GhostBSD maintainers to set it up this, rather unusual, way! Typically users have a separate primary group, then 'wheel' membership is established as a secondary group membership.
Not sure if there is way to rewrite the test to accommodate this situation, maybe the existing trick to switch to nobody can be used - although I am sure if this is permitted for non-root users. Another possibility is to mock the implementation of setgid() but that's a quite different approach from how I have seen other libuv tests. Let me also ask in the GhostBSD forums why the user is set up like this...

Thanks Poul, that's useful information. Can you or @damon-kwok try this patch?
diff --git a/test/test-spawn.c b/test/test-spawn.c
index 05c76f614..819abc803 100644
--- a/test/test-spawn.c
+++ b/test/test-spawn.c
@@ -1427,8 +1427,7 @@ TEST_IMPL(spawn_setgid_fails) {
   int r;

   /* if root, become nobody. */
-  uv_uid_t uid = getuid();
-  if (uid == 0) {
+  if (0 == getuid() || 0 == getgid()) {
     struct passwd* pw;
     pw = getpwnam("nobody");
     ASSERT(pw != NULL);

If that doesn't help, I suppose the workaround is to skip the test when gid == 0 and uid != 0.

@bnoordhuis, that didn't work - I believe due to lack of permissions to become 'nobody':
not ok 1 - spawn_setgid_fails
# exit code 6
# Output from process `spawn_setgid_fails`:
# Assertion failed in /home/ptlomholt/dev/libuv/src/test/test-spawn.c on line 1434: 0 == setgid(pw->pw_gid)

How about instead switching to a different (than 0) gid in this situation? The following patch made the test succeed on this platform (had to shuffle the code around a bit) :
diff --git a/test/test-spawn.c b/test/test-spawn.c
index 05c76f61..b2d546d0 100644
--- a/test/test-spawn.c
+++ b/test/test-spawn.c
@@ -1426,24 +1426,29 @@ TEST_IMPL(spawn_setuid_fails) {
 TEST_IMPL(spawn_setgid_fails) {
   int r;

+  init_process_options("spawn_helper1", fail_cb);
+
+  options.flags |= UV_PROCESS_SETGID;
+#if defined(__MVS__)
+  options.gid = -1;
+#else
+  options.gid = 0;
+#endif
+
   /* if root, become nobody. */
-  uv_uid_t uid = getuid();
-  if (uid == 0) {
+  if (0 == getuid()) {
     struct passwd* pw;
     pw = getpwnam("nobody");
     ASSERT(pw != NULL);
     ASSERT(0 == setgid(pw->pw_gid));
     ASSERT(0 == setuid(pw->pw_uid));
+  } else if (0 == getgid()) {
+    /* otherwise, if gid is already 0, switch to some other gid */
+    struct passwd *pw = getpwnam("nobody");
+    ASSERT(pw != NULL);
+    options.gid = pw->pw_gid;
   }

-  init_process_options("spawn_helper1", fail_cb);
-
-  options.flags |= UV_PROCESS_SETGID;
-#if defined(__MVS__)
-  options.gid = -1;
-#else
-  options.gid = 0;
-#endif

   r = uv_spawn(uv_default_loop(), &process, &options);
 #if defined(__CYGWIN__) || defined(__MVS__)


Orthogonal init for conv as well

I think our convolutions are bias-free. Lemme double check though.

hmm now that i look at this, should we maybe do that check for linear layers as well? just to not error out if we ever use non-biased linear layers


Can't create more of 5 pool in one instance due to systemd

Sorry about Advertising
I need contributors for my project. If you would like to help the project is located here.
https://github.com/poKe-Websites/poKe-CMS
Thank You :)

Closing this out since it would actually break the functionality of the action


Update index.tsx

Typos

Coverage remained the same at 69.048% when pulling 1fa4f1c0e6676f559fb2c7d0ec29be7e15dee219 on mathiasbc:patch-1 into cd0b54f32c54702969dd44c36fffb0def7c1163f on scniro:master.

Coverage remained the same at 69.048% when pulling 1fa4f1c0e6676f559fb2c7d0ec29be7e15dee219 on mathiasbc:patch-1 into cd0b54f32c54702969dd44c36fffb0def7c1163f on scniro:master.

@mathiasbc thanks for this and you interest in the project! However, I'm not going to merge this in simply because these warnings methods are all temporary and I intend to remove them shortly. In fact, that will be my very next contribution on this (a littler overdue now.) Again, thank you, it's just these are getting ripped out and that small typo isn't the end of the word this very moment.


Create db schema

Generate the database design

Users, Celebrities, Activities, and Reviews all complete
Pending Orders table and associations.


HDDS-10686. Bump npm packages only for security

What changes were proposed in this pull request?

Upgrade npm dependencies only for security fixes.
Disable major version upgrades for Maven dependencies.

https://issues.apache.org/jira/browse/HDDS-10686
How was this patch tested?
N/A

LGTM, but just to confirm, dependabot will still open PRs for pnpm security updates based on the global repository settings, right? Is there a way to confirm this since I don't think we have access to the settings page for the repo? I see these but they aren't pull requests.

I think so.  Dependabot had been creating version bump PRs for security before HDDS-9171 added dependabot.yml.

I just checked the .asf.yaml docs and dependabot alerts (security udpates) are on by default and updates (version updates) are off by default. So this PR should give us the desired behavior.

https://github.com/apache/infrastructure-asfyaml?tab=readme-ov-file#dependabot-alerts-and-updates
https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=127405038#Git.asf.yamlfeatures-DependabotAlertsandUpdates


Resolve the 'Size' conflict between dart:ffi and dart.ui

Fixed this in bitsdojo_window: ^0.1.2.
Thank you for trying to help. Appreciate it.


Adding a background image

Hi there and thanks for the theme, really enjoyed it so far!
I have custimized it a bit for a conference page we are building (https://stevenputtemans.github.io/AMV2018/). So far so good, but now I would love to have the first block, where the text spawns in, to contain a 50% opague background image.
I tried adding style="background-image: url(../img/background.jpg) to the
 container, but that does not seem to do the trick. Any ideas on what I can try next?

Hey Steven!
Site looks great. I love what you've done with it!
I tried style="background-image: url(fullpath.jpg);background-size:cover;" in the same
 and it worked for me. You might need to check your path. Let me know if you still have an issue.
The 50% opaque part is possible if you add a pseudo element with 50% opacity, but if you're not comfortable doing that on your own I'd suggest that it would be easier to make the image itself 50% faded instead. That can be done in programs like GIMP or Photoshop, or with an online tool like Canva.

Hi @vickylai , after breaking my head for hours it seems that I found the same solution as you suggested: https://stevenputtemans.github.io/AMV2018/ and it works flawlesly.
Will test with the opague now, but not sure if the site really needs it

I think it looks fantastic. Well done!

Hi @StevenPuttemans, really like your conference site, is that possible to share the code behind?

Hi @mingsu you can basically find my original HUGO code at https://github.com/StevenPuttemans/own_website_hugo_code. Feel free to pick out any items you like! Still a lot of todo items on my list, but haven't found the time as post-doc to invest more.

Ow you mean the conference website 🤦♂ I will upload it tomorrow at work ...

The code can be found at https://github.com/StevenPuttemans/amv_hugo_code
Had some issues with pc and forgot about this, but here you go! Feel free to reuse anything there.


Esisalama computer school add

@fidele5 Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Browser - Takes 3 -4 minutes for getting responses.

Steps to reproduce the problem:

Create an agent with Browser tool.
2.Ask to do the following in 1 prompt

Go to cnn.com. Click on entertainment link. Get the latest news .

It took 3 -4 minutes for the response to be received.

Debug logs - calls (87).json

I had this chat which took about 1.4 minutes to respond.

go to world link and get the latest news

Debug logs - calls (88).json

@sangee2004 getting the content of a page seems reasonably fast now.
For example, the Get Page Contents call triggered by the prompt "Get the content of the hackernews home page" takes ~5 seconds to finish on https://test.otto8.ai.
Could you test out the individual tool calls to see how quickly they respond?
It may just be that prompts requiring complex navigation that involve a sequence of multiple tool calls takes longer, but I'm seeing the individual calls respond quickly now.

cc @cjellick


all: prepare for 0.12 branch

This bumps the minimum Go version so we can start using generics (sparingly/experimentally), deprecates some functions that were missed previously and bumps the test version. It also deals with enormous amounts of churn due to the changed in gofmt.
Please take a look.

Codecov Report

Merging #1823 (5366406) into master (c5022ba) will increase coverage by 0.01%.
The diff coverage is 100.00%.

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

@@            Coverage Diff             @@
##           master    #1823      +/-   ##
==========================================
+ Coverage   73.48%   73.49%   +0.01%
==========================================
  Files         528      526       -2
  Lines       62015    62003      -12
==========================================
- Hits        45573    45572       -1
+ Misses      13878    13867      -11
  Partials     2564     2564

Impacted Files
Coverage Δ

blas/blas32/blas32.go
0.00% <ø> (ø)

blas/blas64/blas64.go
72.50% <ø> (ø)

blas/cblas128/cblas128.go
77.31% <ø> (ø)

blas/cblas64/cblas64.go
0.00% <ø> (ø)

blas/gonum/dgemm.go
70.73% <ø> (ø)

blas/gonum/level1cmplx128.go
64.94% <ø> (ø)

blas/gonum/level1cmplx64.go
0.00% <ø> (ø)

blas/gonum/level1float32.go
0.00% <ø> (ø)

blas/gonum/level1float32_dsdot.go
0.00% <ø> (ø)

blas/gonum/level1float32_sdot.go
0.00% <ø> (ø)

... and 257 more

Help us with your feedback. Take ten seconds to tell us how you rate us.


Add warning about env vars defined via others

Why:
Variables in the env map cannot be defined in terms of other variables in the map, which may surprise those used to regular shell environment variables.
What's being changed:
Add note to documentation.

@FlashSheridan
Thanks so much for opening a PR! I'll get this triaged for review :zap:

This makes sense.  Thanks!


📃 [ENHANCEMENT] -

Issue Verification

[X] I have verified that this is a new documentation issue.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Documentation Area📚
The MySQL section under database
Issue Description✍
Most of the descriptions use the phrase "Guide/instructions on how to", this is a little repetitive to read in every description. Grammatically reformatting a few of them can make the repository more appealing to read and go through.
Relevant Links🔗
No response
Additional Context
No response
Code of Conduct

[X] I follow the CONTRIBUTING GUIDELINE of this project.

Can you elaborate more?


recapttcha doesn't work with more than 0 digits

When I change the length of the Captcha it disappears when I browse to the login page.
With 0 option an image is shown asking about if I am a bot.
What could be happening?
Regards

When I change the length of the Captcha

Where did you find it (length of the Captcha)?
There is a "limit" option only.
"Limit" is a number of authentication failures
that are allowed before the recaptcha will be shown.

Oh!!! Thanks for the explanation, I hadn't understood well when I installed the plugin.
Regards


Function sample

https://github.com/firebase/functions-samples/blob/9530537630c8be1ae752589d9f95763601852547/quickstarts/uppercase-firestore/functions/index.js#L19-L67

@devdev606 was this opened by accident? Is it ok if I close this?


Added DateTimeStyleConversion to JsonConfiguration

Prerequisites

[x] I have written a descriptive pull-request title
[x] I have verified that there are no overlapping pull-requests open
[x] I have verified made sure that I am following the Nancy code style guidelines
[x] I have provided test coverage for your change (where applicable)

Description
Addresses #2417

I hear @khellang says we should assume UTC. IMO I think we should leave it assume local.
Microsoft has already dug a hole by having DateTime.Now in .NET, and GETDATE() in SQL Server, and having to be explicit with DateTime.UtcNow / GETUTCDATE()
It's kinda the .NET way to assume local, but peoples web/database servers should always be configured to UTC anyway.

The configuration to me seems messy, but I don't have any better suggestions...

If everyone disagrees with me on AssumeLocal then merge away. OVERRULE ME! DO IT, I KNOW YOU WANT TO!


fix: removed .idea folder

Closes #12

Coverage report

St.:grey_question:
Category
Percentage
Covered / Total

🟢
Statements
100%
62/62

🟢
Branches
100%
11/11

🟢
Functions
100%
1/1

🟢
Lines
100%
62/62

Test suite run success
6 tests passing in 1 suite.
Report generated by 🧪jest coverage report action from 32bf9460bd405ec1b190ce20821649144e8be984


Career Paths - Each path should have individual URL/page

From the readability and SEO perspective, it's good to have individual pages with unique URLs for each subsection.
Currently, Career paths has all the paths on one page. Ideally, each subsection should have it's own individual URL/page.
The URLs should look something like this _http://softdevpk.com/career-paths/android-development_

We already have the individual pages with their respective links.


Security issues in latest postgres image

When we scan the postgres latest image we found critical and HIGH security issues in those images.
Attaching the latest image security issue report.
postgres:latest (debian 12.6)
=============================
Total: 145 (UNKNOWN: 0, LOW: 101, MEDIUM: 32, HIGH: 11, CRITICAL: 1)

And we found more critical issue in gosu library
usr/local/bin/gosu (gobinary)
=============================
Total: 52 (UNKNOWN: 0, LOW: 1, MEDIUM: 18, HIGH: 30, CRITICAL: 3)

We also found same report in dockerhub as well

See also https://github.com/tianon/gosu/blob/master/SECURITY.md


[KW-search] Indices on parents array

Description
Fixes https://github.com/dust-tt/dust/issues/9251
The PR will be updated with explain analyze queries to show the indices perform as expected. If they do not, they will be rolled back.
Related important context: https://dust4ai.slack.com/archives/C050SM8NSPK/p1733837120557819
Risks
Creating indices could slow down production. Mitigated by 1. running at low peak (lunch or evening) 2. using CONCURRENTLY:
CREATE INDEX CONCURRENTLY is designed to be low-impact:

Doesn't block writes/reads on the table
Takes ~2-3x longer than regular CREATE INDEX
Can be cancelled safely with pg_cancel_backend() or pg_terminate_backend()

Main impact is on disk I/O and some CPU
Two caveats:

Requires ~2x disk space during creation
Can't run inside a transaction block

Deploy

run index creation on prodbox
post explain analyze queries here
either rollback, or deploy core

327 => our Notion
parent id used => the Engineering page
Conclusion: index works for parents[2] case
dust-api=> EXPLAIN ANALYZE SELECT * FROM data_sources_nodes WHERE data_source = 327 AND parents[2] = 'e0f834b5-be5a-4356-9baa-f76e9c41adf2';
                                                                          QUERY PLAN
---------------------------------------------------------------------------------------------------------------------------------------------------------------
 Index Scan using idx_data_sources_nodes_parents_second on data_sources_nodes  (cost=0.56..47.88 rows=11 width=358) (actual time=0.648..0.710 rows=74 loops=1)
   Index Cond: ((data_source = 327) AND (parents[2] = 'e0f834b5-be5a-4356-9baa-f76e9c41adf2'::text))
 Planning Time: 0.603 ms
 Execution Time: 0.771 ms
(4 rows)

327 => our notion still
Conclusion: index works for single-parent array case
dust-api=> EXPLAIN ANALYZE SELECT * FROM data_sources_nodes WHERE data_source = 327 AND array_length(parents, 1) = 1;
                                                                            QUERY PLAN
------------------------------------------------------------------------------------------------------------------------------------------------------------------
 Index Scan using idx_data_sources_nodes_parents_single on data_sources_nodes  (cost=0.43..4462.94 rows=11 width=358) (actual time=0.022..0.182 rows=243 loops=1)
   Index Cond: ((data_source = 327) AND ((array_length(parents, 1) = 1) = true))
 Planning Time: 0.945 ms
 Execution Time: 0.243 ms
(4 rows)


VarDiff with low retargetTime throw System.NullReferenceException

Updating VarDiff if there is no shares submitted will throw Exception. I suggest we should consider updating diff at least one share even with low retargetTime and high start Diff.
[2017-09-24 13:47:00.4122] [E] [dash] Error handling request: mining.submit System.NullReferenceException: Object reference not set to an instance of an object.
   at MiningCore.VarDiff.VarDiffManager.Update(VarDiffContext ctx, Double difficulty, Double networkDifficulty) in /root/miningcore/src/MiningCore/VarDiff/VarDiffManager.cs:line 64
   at MiningCore.Mining.PoolBase`1.UpdateVarDiff(StratumClient`1 client, Double networkDifficulty) in /root/miningcore/src/MiningCore/Mining/PoolBase.cs:line 187
   at MiningCore.Blockchain.Bitcoin.BitcoinPoolBase`2.<OnSubmitAsync>d__6.MoveNext() in /root/miningcore/src/MiningCore/Blockchain/Bitcoin/BitcoinPoolBase.cs:line 148
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at MiningCore.Blockchain.Bitcoin.BitcoinPoolBase`2.<OnRequestAsync>d__10.MoveNext() in /root/miningcore/src/MiningCore/Blockchain/Bitcoin/BitcoinPoolBase.cs:line 281
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at MiningCore.Stratum.StratumServer`1.<>c__DisplayClass11_0.<<OnClientConnected>b__2>d.MoveNext() in /root/miningcore/src/MiningCore/Stratum/StratumServer.cs:line 160    at MiningCore.VarDiff.VarDiffManager.Update(VarDiffContext ctx, Double difficulty, Double networkDifficulty) in /root/miningcore/src/MiningCore/VarDiff/VarDiffManager.cs:line 64
   at MiningCore.Mining.PoolBase`1.UpdateVarDiff(StratumClient`1 client, Double networkDifficulty) in /root/miningcore/src/MiningCore/Mining/PoolBase.cs:line 187
   at MiningCore.Blockchain.Bitcoin.BitcoinPoolBase`2.<OnSubmitAsync>d__6.MoveNext() in /root/miningcore/src/MiningCore/Blockchain/Bitcoin/BitcoinPoolBase.cs:line 148
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at MiningCore.Blockchain.Bitcoin.BitcoinPoolBase`2.<OnRequestAsync>d__10.MoveNext() in /root/miningcore/src/MiningCore/Blockchain/Bitcoin/BitcoinPoolBase.cs:line 281
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at MiningCore.Stratum.StratumServer`1.<>c__DisplayClass11_0.<<OnClientConnected>b__2>d.MoveNext() in /root/miningcore/src/MiningCore/Stratum/StratumServer.cs:line 160

If you want to fix that one along with the other vardiff related bug I can assign it to you.

Yes, please assign it to me. Thanks.
I think the out-of-band vardiff updates is a good decision. But I would prefer to do it when idle checking.
somethings like: 60s(retargetTime) no shares submitted. Update VarDiff. add this 60s to the calculation for average Adjust the diff. wait for new share.
But I would suggest short retargetTime like 5s for pool farm.


Counter example in leptos

I was interested to see how Crux works with UI frameworks that are based on fine-grained reactivity. This is a basic example implemented in Leptos that we can evolve/work with.

I tweaked the Yew example a little (e.g. extracting out client-side HTTP and SSE capability implementations — which are obvs identical anyway between Yew and Leptos) and what remained is remarkably similar (at least in terms of complexity and quantity of code).
I think we can probs merge this as a basic example. What would be interesting though is something a little more complex that effectively splits the view model down the component hierarchy. I can imagine another Render capability that could carry more granular view model updates with it (e.g. instead of just asking us to call core.view() and getting it all at once), but that is fun to save for later.

I can imagine another Render capability that could carry more granular view model updates with it (e.g. instead of just asking us to call core.view() and getting it all at once), but that is fun to save for later.

@StuartHarris I've been thinking about this too. But I haven't run into a situation yet where core.view() is too slow 😄

I also took the opportunity to update the book to add a new page for the Leptos example, and update the Yew and NextJS pages to point to new minimal working code in the hello_world example. Apologies that it's flooded the PR with spam.


Contribution back witness helpers

We have a library that provides a number of helpers to facilitate the creation of transactions that spend (witness and normal) outputs, whether it creates the witness to spend a P2WPKH Output
or help you sign transactions.
Please note that we have planned improvements for the existing code (e.g. use the new Amount types).
Do you think that porting this code to rust-bitcoin/rust-bitcoin would bring any value? Would there be a more appropriate project under the rust-bitcoin umbrella that could benefit from this?

Hmm, @apoelstra has been working on rust-miniscript for the last year that would probably make it a lot safer to create tx witnesses and be compatible with more script types.
The idea is that we would use rust-miniscript to finalize a PSBT that has all the signatures. Those signatures could be added to the PSBT through various means. rust-miniscript can also help figure out signatures from which keys are relevant for a PSBT input.
So I think a signing flow would involve something like this:

create tx you want to create (define outputs and fund with owned inputs)
create a psbt from this tx and provide the previous txs or outputs
for each of the psbt inputs, go over all the relevant keys and generate a signature if the private keys is available; add the signatures to the PSBT
finalize the PSBT and turn it into a signed tx

Steps 3 (the part to get the relevant keys) and 4 are currently not possible but will be with rust-miniscript.

Yes, rust-miniscript can do this. I really do not want one-off templates in rust-bitcoin.

rust-miniscript it is, we'll give a try. Thanks!


feature(terraform): init terraform module

This PR creates a terraform module the creates the Forwarder lambda and allows the user to pass a list of log group names to create the needed Subscription filters.

hey @pecigonzalo regarding your comment previously:
Code wise/module wise, I dont think we should be doing the log-group subscriptions from Terraform. I think I mentioned the same in CF. Given we have Lambdas that can do the subscription backfill and dynamic addition of new log-groups, I don't think static resources (tf and cf) are suitable for handling dynamic resources like log-groups.

The problem with Subscriber/Unsubscriber module is that it created resources that Clouformation is not aware of. I wanted to avoid this pattern with terraform, and make it easier to find all the created subscription logs and easily maintain a list of log group names. I agree with you there should be a better way to handle the dynamic log groups. Wondering if current setup + listener only would fit, wdyt?

The problem with Subscriber/Unsubscriber module is that it created resources that Clouformation is not aware of. I wanted to avoid this pattern with terraform, and make it easier to find all the created subscription logs and easily maintain a list of log group names. I agree with you there should be a better way to handle the dynamic log groups. Wondering if current setup + listener only would fit, wdyt?

I understand the reasoning. The challenge is that the dynamic nature of resource like cloudwatch log groups does not fit well with Terraform rather static and stateful nature. If AWS supported a single global "subscription config" for all CWLG (CloudWatch Log Group) I would certainly prefer to configure this via Terraform.
For example, as a "Cloud Engineer", as soon as a new CWLG is created I want to ensure its logs are propagated the logging pipeline.
If this is done via a static terraform config for which I have to populate names, I have to either preemptively create the group and subscription or do it after it was created and I notice my logs are missing.
Furthermore, this is a challenge because its likely that the code that provisions the resource that creates the CWLG and the place in which I configure this are different.
I think the idea of having a CWLG controller, that takes care of configuring them all (kind of what we did with the lambda, and something I've implemented myself at other places) is more maintainable, fit for porpouse and works well in most environments.
The way I generally configure this is with a single controller lambda that has multiple triggers:

CWLG creation event -> Subscribe and set retention
Periodic -> Check all CWLG and reconcile any drift
Manual event -> Check a named CWLG and reconcile

So instead of having two ways of doing this, one that is a bit more subpar than the other (terraform), I would advocate for a single recommended approach. Particularly when anyone needing to configure this via Terraform only needs 1 or 2 resources at most, so its not something for which we add a ton of value in our module or they might get more value out of the settings of the module they might using already (be that a dedicated CWLG module or a Lambda or RDS module that already contains some CWLG settings).

@SollyzDev Added the missing suggestion (inline the policy) https://github.com/axiomhq/axiom-cloudwatch-forwarder/pull/88 along some minor style fixes. Feel free to merge, otherwise LGTM.

@SollyzDev inline_policy in the role is deprecated, I would suggest reverting those 2 commits and either merging the linked PR or doing it that way.

@SollyzDev Added the missing suggestion (inline the policy) #88 along some minor style fixes. Feel free to merge, otherwise LGTM.

I enabled auto-merge, waiting for reviewers approval.


template partials and include

Hi Carlton,
so, this bit me today:

I have 2 views that use very similar templates
both those views inherit a behaviour that tells them to render their partial if the request is htmx (ie: add #part tot he template name)
the partial bit is part of what's similar between the 2 templates, and I moved all that into an {% include %}
my views started failing to render the partial

This makes sense, the partial is now registered on the included template (parser.orgin).
The situation went from
A.html -> defines partial A.html#part
B.html -> defines partial B.html#part
to
A.html -> includes C.html -> defines partial C.html#part
B.html -> includes C.html -> defines partial C.html#part
A.html#part nor B.html#part are defined
I'm not sure this is a bug... it's not really... but it's not obvious either. If it's not a bug I guess it's a limitation. Maybe there should be something in the docs? I'm guessing this is likely to happen to other people doing some refactoring at some point.

Hey @nanuxbe — Thanks for the report.
Yes, that's totally expected behaviour. A.html#part no longer exists right, so it would be some going for the template loader to load it. (PRs always looked at, of course. 😉)
There is a similar thing with parent templates. To use a partial base.html#sidebar you'd use include from the child template, even when extending base.html, if that makes sense.
Let me ponder explaining all that. (PRs always looked at, of course. 😉)


Edit pass complete.

Edit pass complete for https://github.com/ajford76/sp-dev-docs/blob/adfor_spcna/docs/design/key-web-part-examples.md

@ajford76 I completed an edit pass and merged my changes with your branch.


Draw shapes with  LineLayer  to represent fields

Hello,
A customer contacted me because he would like to use maps that allow him to display and locate "private areas" (like plots for a farmer) through layers drawn on the map.
The expected result looks like this.
I've seen that MapBox allows to draw shapes on their map, like in this sample.
So I would like to know if it's possible to "import" the areas through shapes that have been created through an admin web app with your plugin (by using LineLayer for example). Or is there another approach that would be better?

Hi,
Yes, it's possible.
Please check out latest version and samples.
Cheers.


Fix exists false when field is explicitly defined

Overview
When users explicitly defined a text index's fields, and used $exists
with false, we tag false when constructing the query. This led to a
function clause for indexable_fields since we did not account for it.
This fix patches that up, but note that we don't care about the false
value itself since we only care about fields.
Testing recommendations
New unit tests added to test for this particular operator
GitHub issue number
This PR is the issue.
Checklist

[X] Code is written and works correctly;
[X] Changes are covered by tests;
[ ] Documentation reflects the changes;

@willholley : It's worth noting that the behavior is going to be different for because of how we perceive $exists "should" work. If we indexed everything, and we queried {"foo": {"$exists": false}}, then we would return  all the documents in the db that did not have "foo" as a field in the document. However, if you explicitly indexed "foo", then you would get 0 results back because you only indexed the subset of documents that contained "foo" as a field. The large (all_docs)superset is not part of the index, and therefore would not be returned. So _all_docs, explicit json index, index everything, and explicit text index would all return different results.

@tonysun83 if a user specifies {"$exists": false} explicitly, we can never use an index to fulfil that predicate (you're correct that this was addressed in https://github.com/apache/couchdb/commit/503f3e34afa0a68e91258ca864e70ab18e6584e2). Can you provide an example where you think we'll trip up (or ideally, a failing test)?
As a general rule, I think we should consider it a bug whenever there is variation in results due to indexes being present, the exception being index-specific operators such as $text. The expectation is that a user will start with no indexes (so queries will default to _all_docs) and add indexes when necessary to improve performance or when sorting is required. Adding an index to an existing application should never change the results of queries already being used.
Another thing to note, for awareness, is that an explicit partial index could be used to optimise an {"$exists": false} selector but these require users to opt-in - the query planner will never automatically select them.

@willholley : I've always been champion of making the results the same across json and text indexes. If $exists: false is a special case, perhaps we default to using _all_docs as well even with a text index? No other  way to do that unless the user already indexed everything with another text index (but we have no way of knowing that).

@willholley : Can you remind me of the corner cases where index-everything text indexes would return different results than _all_docs? If I recall it had to do with a field having a null value and querying for $ne. If discrepancies are only query specific, then I would advocate using index-everything for "exists": false simply for the performance gain. I'll think a bit more, but index-everything should index every single document.

@tonysun83 "index everything" does sound like a special case where "$exists":false could be fulfilled by the text index. Text indexes in general will behave differently to _all_docs or JSON indexes when range queries are used.
For example { "name": {"$gt": 0}} against a text index would only match documents where the value of "name" is an integer, whereas against _all_docs it will match documents where "name" is a string / object / array as well. This extends to null handling - text indexes do not include null values in range query results whereas other index types do.
I'd really like to address the difference in range handling between the index types (adding an index should never change the results of a query) but I think that needs a separate discussion / PR as it will be a breaking change to align them.

@willholley : should be the same across all index types now. This is for $exists: false. we might have to check out $ne operators as well.

Fixed the asserts.
The test you referenced is meant to return the same results as:
https://github.com/apache/couchdb/blob/master/src/mango/test/03-operator-test.py#L148


Question

So I want to help u with this project
Can u pls give me write access?
I'm really interested and I have many ideas with stuff like the fs io...

Hi DiamondCoder1000,
Thanks for your interest in this repository!
Since I will not be updating this repository, I think it would be better if you fork it and work on it under your own profile.
I would of course like to help you in the future in case you have any technical doubts, but giving you write access is pointless, since I'm not contributing anymore.
So it'll be better for you to fork this code, and start fresh on your own, since there will be no contributions from my side.


Create a navigation link based on annotation value

What would you like to be added:
Create a navigation link based on annotation value. Something similar to this feature on Spinnaker:
https://spinnaker.io/guides/user/kubernetes-v2/annotations-ui/
Why is this needed:
This feature will allow easy navigation to runbook pages for cronjobs and services. Currently, the link (just a value of an annotation) is cut in the detail page as it too long and the only option to get the link is to click on the edit button and copy it.
Environment you are Lens application on:

Kubernetes distribution: [EKS, AKS, GKE, Rancher, ...]
Desktop OS:

Would this be fixed by #1400?


define CachedResource::toJSON to prevent storing and to localStorage

We encountered a bug where an instance of a CachedResource that was instantiated from reading out of the cache had $promise and $httpPromise properties that were later getting serialized to localStorage. In an offline scenario, reading from the cache would then return an empty object because the real deferred objects would get blown away by the improperly serialized promises.
This PR omits these properties before writing to them to localStorage.

LGTM :clap:


Add Arbitrary instances to types.

I don't know how to proceed with BigInt, it doesn't have an instance of Arbitrary but we can use a variation of
instance Arbitrary BigInt where
  arbitrary = fromSomeIntTypeWithInstance <$> arbitrary

Then a lot of the new instances can be auto derived by :
deriving newtype Arbitrary SomeWrapper

I don't know the right place to put such an instance.
Apart from it, I have doubts in the following types :

redeemerHash

Serialization Address

CurrencySymbol

NonAdaAsset (depends on CurrencySymbol)

Value (depends on NonAdaAsset)

ValueSpentBalances (depends on Value)

OutputConstraint (depends on Value)

TxConstraints (depends on OutputConstraint)

UnattachedUnbalancedTx

MkUnbalancedTxError

PlutusScript

MintingPolicy (depends on PlutusScript)

Validator (depends on PlutusScript)

StakeValidator (depends on PlutusScript)

ScriptHash

MintingPolicyHash (depends on ScriptHash)

ValidatorHash (depends on ScriptHash)

StakeValidatorHash (depends on ScriptHash)

BlockHeaderHash

ChainTip (depends on BlockHeaderHash)

Tip (depends on ChainTip)

TransactionMetadatum

GeneralTransactionMetadata (depends on TransactionMetadatum)

TxConstraint

TypedTxOut (depends on Cardano.Types.TransactionOutput)

TypedTxOutRef (depends on TypedTxOut)

TypeCheckError (depends on Address and PlutusData)

TypedValidator (depends on Validator, MintingPolicy)

Cardano.Types.Transaction.Transaction

UnbalancedTx (depends on Cardano.Types.Transaction.Transaction)

Most of them are related to create arbitrary Datum or Redeemer or create an arbitrary script for validator or mint or create arbitrary transaction. The ones whose counterparts in Plutus I checked, don't have an  Arbitrary instance.
So, I think those must be left without Arbitrary instances unless we have a good reason.

Regarding types that don't have instances in Plutus, it's not worth the effort I think (especially for values that should point to some existing on-chain state, like hashes). Let's skip them.

Hi @Luis-omega ,
I've found some resources for you that we used on the Haskell side of things to provide sensible Arbitrary instances:

plutus-extra https://github.com/Liqwid-Labs/plutus-extra/blob/master/quickcheck-plutus-instances/src/Test/QuickCheck/Plutus/Instances.hs
purescript-bridge https://github.com/mlabs-haskell/purescript-bridge/blob/master/roundtrip/RoundTrip/ArbitraryLedger.hs

Could I suggest that you start with tests first, migrating any unit test cases to use quickcheck, rather than going module by module? For instance I'd start with https://github.com/Plutonomicon/cardano-transaction-lib/blob/master/test/Data.purs but there's plenty of test modules that are obviously a candidate for using quickcheck.
Another reason that I'd suggest starting with tests is that writing Arbitrary instances often lead to unintended recursion and the best way to know whether they actually work is to use them in a test immediately.
Regardless of the route you wish to take, I'd suggest at least to split the effort across smaller PRs, each containing an Arbitrary instance accompanied by a test that uses is.
(P.S. Are you aware of any code coverage tools for Purs/Js? That would be amazing to have)

Since this way of doing it looks much more complete, I would like to do it, but I will probably have to open a new ticket or modify the current one #466. Also, if the task is too big maybe it should be spread over multiple tickets. I think this is a good thing worth having in the test suit, but it maybe takes some time, so,  I would like to ask @klntsky  and @ngua  If I should be spending too much time in it first.

Sorry, I think I misunderstood the comment before, yeah, porting as many tests as I can to use QuickChec is a good idea, I going to do it asap.

I'm closing this branch to follow the tests aproach.

Reopening as Arbitrary instances may be useful for ToData/FromData roundtrip

We want to lower the number of open prs, this pr is just too old and I'm focusing on ODC right now.


pushover LICENSE

LOL.
Join us now and share the software;
You'll be free, hackers, use BSD.
Join us now and share the software;
You'll be free, hackers, you'll be free.

From the man the myth the legend himself


no additional information when clicking on "i" button in scan results

DLAU version 1.2 - no pop-up information is showing when clicking on the information button/hotspot.
Earlier versions had a description of the field although for CCB Users the Person Document description was showing (v1.1.4)

This is tracked as DNEXT-23771 and will be addressed in the next version 1.2.1


Clamp manual yaw target when system is in yaw rate limit

In this log the yaw tracking is really good:
http://logs.uaventure.com/view/xM8RPz26NeFtmcR8Cnm5RL#A_YawRate_PLOT
But the system response felt sluggish. This is most likely because a yaw setpoint was built up as the system was hitting the 60 deg/s yaw rate limit and so the quad continued to turn after the yaw set point.

I just flew https://github.com/PX4/Firmware/tree/yaw_man_limmit and it appears to have fixed the yaw overshoot behaviour i was seeing.
http://logs.uaventure.com/view/beciMrBZ8JVhDU9Z2vKDmQ

Nice! Note that you might want to calibrate your ESCs or correct your system's CG: http://logs.uaventure.com/view/beciMrBZ8JVhDU9Z2vKDmQ#Actuators_PLOT

Instructions: http://px4.io/docs/esc-calibration/

It has been calibrated, but I have a sick esc.


SPARKC-272: Bump Spark to 1.6-SNAPSHOT and add Apache Snapshot repository to resolvers

Build against Scala 2.11 finished with success

Build against Scala 2.10 finished with failure

Build against Scala 2.10 finished with success

+1


All options should have an "Off" key

For consistency, all options should have an "Off" key so one can say Ligatures=Off and turn off all ligatures (equivalent to Ligatures={NoRequired,NoCommon,NoContextual}) and so on.

Again sorry for the slow reply!! This is happening in the next release. (With current interface Ligatures=ResetAll.) Actually… maybe this isn't quite what you asked for. ResetAll removes all +onum AND -onum etc. etc. Do you think a global "off" still makes sense?

I think this can be closed now.


Compatibilty with inkpad3  (from tea and pocketbook) : the image is too hight by 25 pixels

The profile is basically the same as the  kobo aura one but need a slid up of 25pixel for all  images
I've tested by modifying the style.css file in OEBTS/text of an epub
@page {
margin: 0;
}
body {
display: block;
margin: 0;
margin-top:-25px;
padding: 0;
}
And it works fine !
Maby it can be corrected here :
(in comic2ebook.py line 403 I can add "margin-top:-25px" just for inkpad3)
f.writelines(["@page {\n",
"margin: 0;\n",
"}\n",
"body {\n",
"display: block;\n",
"margin: 0;\n",
"padding: 0;\n",
"}\n"])

Thanks but I don't plan to implement a profile for InkPad3.


Can't transfer repository owned by organization

Gitea version (or commit ref): 1.1.2
Git version: 2.7.4
Operating system: Ubuntu 14.04.5 LTS
Database:

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[X] No
[ ] Not relevant

Log gist:

2017/07/24 15:31:03 [...ters/repo/setting.go:277 SettingsPost()] [E] TransferOwnership: removeOrgRepo: database table is locked
2017/07/24 15:31:03 [D] Template: status/500
Description
Repositories can't be transferred out of an organization. I can transfer from my individual account into an org, but I can't transfer it back. I also can't transfer org to org. The error is happening here. I have another instance of Gitea where the same thing happens. I attempted to reproduce it on the try.gitea.io but couldn't get it to happen.
One instance was running Gitea 1.1.0 and I upgraded to 1.1.2 to see if it fixed it, but it didn't. The other instance was already at 1.1.2 before I tried.

I think it should be fixed in master

You are correct. I built from master and the problem is fixed. I'll go ahead an close this. We'll just use our built version until an official release is made. I should've looked harder through the commits. Thank you.


tx produces more unlocked (1000000) than it consumes (0)

I am following the official guide named “Deploying Subnet with EVM Based Blockchain using AvalancheJS”

I am using the same genesis.json file with updated data as indicated in the guide.
The problem occurs at the step in which we add a validator to our subnet. We simply try to run the file named addSubnetValidator.js.

The screenshot of the error message:

The error message:
/Users/usr/node_modules/avalanche/dist/common/jrpcapi.js:64
                    throw new Error(resp.data.error.message);
                          ^

Error: couldn't issue tx: flow check failed: tx produces more unlocked "BUuypiq2wyuLMvyhzFXcPyxPMCgSp7eeDohhQRqTChoBjKziC" (1000000) than it consumes (0)

Please provide detail on which tutorial you were following.

I followed this documentation: https://docs.avax.network/subnets/create-a-evm-blockchain-on-subnet-with-avalanchejs
Which is also the same as this repository’s readme file.

Please provide detail on which tutorial you were following.

Hi Yulin, any updates?

Hi @kmlcgn. We are looking into it. Will keep you posted. Thanks.

Hello @rajranjan0608, are there any updates on this? I am also having this issue. There seems to be something wrong with the guide mentioned above or the AvalancheJS library itself.

Hi @kmlcgn and @canatakan. I can confirm there's a bug in AvalancheJS. This PR will resolve the issue: https://github.com/ava-labs/avalanchejs/pull/664

Hi @rajranjan0608, thank you!

The issue has been fixed now with the latest AvalancheJS release. Please update your AvalancheJS version to use v3.15.5. Thanks for raising this issue 👍


postgresql: investigate if this service support migration check

What should we add to our content collections?
Based on the discussions on #723 postgresql may also support migration check. Perform the migration check such as explained in the article and see if it applies to postgresql service.
Is this something you would work on yourself?
[ ] I plan to open a pull request for this

This does not seem to be relevant or should be done for just one more service. If we want to have a standard, then this should include all services. Therefore, I am closing this issue.


WARN:amqp::channel: basic_publish(mandatory=true) failures are not handled and result in blocked channels!

I am keep getting this message when trigger the on_row_change function

it comes from the amqp lib that is used and it's not a error, it's a warning
https://github.com/Antti/rust-amqp/blob/master/src/channel.rs#L352
you see it only when you set the logging mode to debug.
it does not mean an error happened, it means, if an error would happen, the lib will not handle the error, and that is ok because
here we check for an error
https://github.com/subzerocloud/pg-amqp-bridge/blob/master/src/lib.rs#L99
and try to resend the message


#219 add support to use a resource profile for sandbox creation

Fixes #219 and #211

Awesome, thanks Tobias


Describe where to find a RISC-V toolchain

Triggered by the question at #1365.

Oh, wait a sec. This is described in great detail in the verification section. I'll change this to point there.

If we want to go down that route we need to give more information on a "matching" toolchain.
And probably add something that the lowrisc toolchains are really unsupported internal ones, nothing people should rely on.

And probably add something that the lowrisc toolchains are really unsupported internal ones, nothing people should rely on.

Well, fair enough, but we don't support anything else at the moment :-) I'll just change this to point at another part of our existing documentation. Maybe that, in turn, could be tidied up in a further PR.


Tab bar tap to return to main page

UIKit TabView should fix it, tap to return to root doesn't seem supported yet by SwiftUI TabView.

Set to release on the App Store as version 2.7.3.

@MadeiraAlexandre Do you think is fine to add a scrollToTop if a tap in the TabBarItem is performed and the TabPage is the same (as the item).
I checked but I’m not really into SwiftUI and I can’t find an easy implementation of that feature.

@MadeiraAlexandre Do you think is fine to add a scrollToTop if a tap in the TabBarItem is performed and the TabPage is the same (as the item).
I checked but I’m not really into SwiftUI and I can’t find an easy implementation of that feature.

Yes, it is fine. I've done it for Kitsune for MyAnimeList (on the Season and Profile page, on the others tabs I'll do it in a future update).
You can control it by using a ScrollViewReader and use a binding to trigger the scroll to the top.
I've done something like this on the computed property:
if seasonPath.isEmpty {     scrollSeasonToTop.toggle() } else {     seasonPath = .init() }
I pass this bool binding to the SeasonView and it can trigger the scrolling to the top, usually using a hidden circle on the top of the view like this one:
Circle().fill(.clear).frame(width: 0, height: 0).hidden().id("top_view").onChange(of: scrollToTop) { withAnimation { proxy.scrollTo("top_view", anchor: .top) } }
On tabs with search field, you could even scroll to the top and then trigger the search field (like Apple does on Apple Music app).
On Cronica I've set it to open the search field only on the Search page, but you could adapt it to make it work with scrolling too and even bring the same behavior to Watchlist tab. (The file used is TabBarView.swift).
I'll reopen this issue as a reminder to do it in a future update.

Thanks @MadeiraAlexandre , that's awesome!
Yes, I saw what you've done in the SearchView to trigger the search-on-tap with the Boolean var.

There's some big changes to how TabView operates on iOS and iPadOS 18. Some of this changes could also reduce code for the macOS side.
Some changes will be likely require on the iPadOS side.
New documentation:
https://developer.apple.com/wwdc24/10147
https://developer.apple.com/documentation/SwiftUI/Enhancing-your-app-content-with-tab-navigation


feat: support for detached working trees

Added config.worktrees.
Array of tables with the keys 'gitdir' and 'toplevel'.
If attaching normally fails, then each entry in the table is attempted.
Example:
  worktrees = {
    {
      toplevel = vim.env.HOME,
      gitdir = vim.env.HOME .. '/projects/dotfiles/.git'
    }
  }

Resolves #397

This seems to work now. In a normal git repo as well as with something from my dotfiles signs show up correctly and I can stage changes.

Great! Thanks for working on this. This is a very much appreciated feature by probably everyone using a bare repo for dotfiles or other purposes. (And miles better than GIT_DIR=$HOME/.dotfiles GIT_WORK_TREE=$HOME nvim)


drivers: entropy: nrf5: Use nrf_rng hal for registers w sideeffects

A few registers with sideeffects were not using the HAL yet.
Fixed.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #8546   +/-   ##
=======================================
  Coverage   63.92%   63.92%
=======================================
  Files         427      427
  Lines       41100    41100
  Branches     6939     6939
=======================================
  Hits        26275    26275
  Misses      11678    11678
  Partials     3147     3147

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 d317a0e...d93f90a. Read the comment docs.


Support RTL text

In order for Hebrew and Arabic to display correctly, we need to figure out on an element-by-element basis whether we should have dir="auto" or dir="rtl" set. My approach: automatically set the easy, harmless ones:
  isRTL: (s) ->
    # Hebrew is 0x0590 - 0x05FF, which is adjacent to Arabic at 0x0600 - 0x06FF
    /[\u0590-\u06FF]/.test s

  applyRTLIfNeeded: ->
    return unless me.get('preferredLanguage') in ['he', 'ar']
    @$('[data-i18n]').each (i, el) =>
      return unless @isRTL(el.innerHTML)
      el.dir = 'rtl'
      $(el).parentsUntil('table, form, noscript, div:not([class~="rtl-allowed"]):not([class~="form"]):not([class~="form-group"]):not([class~="form-group"]), [dir="ltr"]').attr('dir', 'rtl')
      $(el).parents('div.form').attr('dir', 'rtl')

This goes through all the elements that we translate with the data-i18n approach and sets dir="rtl" if they now actually contain Hebrew or Arabic text, plus sets the same thing on any "safe" parent elements. (Usually we stop at a div that we haven't explicitly marked as .rtl-allowed or that is a form.)
Then I also manually tagged a lot of things throughout the code with dir="auto" and .rtl-allowed and fixed up the styles in various places as needed.
I did a pass through main screens of student, teacher, and home player views, using our current level of Hebrew translation support to spot check things. I know I must have missed many places, but this should be a good start.

Coverage decreased (-0.01%) to 62.512% when pulling bbb4228cb471b6972a65f560d3c18bb4db9a6200 on rtl into 0fba4c7660ba1ffbaf49f0be65068df795d31dda on master.

Coverage remained the same at 62.512% when pulling 4ed11d1b024cfe1d449c6e3e7de5de5b4ea7a109 on rtl into a872f8a51a4b7df89a2c013e187bf5bef7101df0 on master.

Coverage remained the same at 62.512% when pulling 4ed11d1b024cfe1d449c6e3e7de5de5b4ea7a109 on rtl into a872f8a51a4b7df89a2c013e187bf5bef7101df0 on master.


How can I get the corrent stacktrace with the sourcemap files?

I have tried to get the correct stacktrace with the sourcemap files via the sourcemap library, but I got the wrong result. Is there something wrong in the sourcemap files? Because I got the corrent result with files generated by uglify-es.

And I also have tried to do that with source-map-support, but could't get the correct result.

I have the same issue, maybe it's a matter of nodejs, when I use node21, I can get the corrent column number in stacktrace, but node14 can't and I don't know why.


Add dot custom mark in vim mode

Add tick-dot and backtick-dot functionality to vim mode.

Thanks! Merged as bb0e1c3f


Validation message not present for Direction widget

On blank directions text box and clicking on 'Get Directions', observe that validation message is not generated after migrating to 3.10 API.
Also observed that sometimes clearing the text in search text box using double click event is not functional, mostly occurs after re-routing.

This app has been replaced by the new Transportation 511.  No action required at this time.


Correct spelling mistake in the case example

This PR fixes a small typo on the example detailing the case function.

Thanks!


Use Different Sources for Quotes, Lyrics, Books & More

Is your feature request related to a problem? Please describe.
Add more sources for races to come from instead of the singular source now.
Describe the solution you'd like
Using quotes from quoteable is pretty cool - but what if the user had an option to use different sources?
Describe alternatives you've considered
N/A
Additional context
Maybe this could be a dropdown? Or buttons that send different fetch requests? Or maybe just send one fetch request when the page is loaded, then return different random quotes from different random sources?

For me, for later:
Use

make quoteable more complex
adviceslip

More Research*

genius lyrics api
Movie / Show Specific Quotes
quotes.rest

Maybe

[ ] programming quotes
[ ] News?


FE: Basic map interface

Basic map page interface. Does not need the heatmap immediately. This should make the map itself appear and have dummy buttons for changing the views and opening the menu.
Imagery and positioning etc for buttons should be selected purely by CSS as possible to allow for an alternate view for computers.

[ ] Loads a satellite map image via Google Maps API
[ ] Dummy buttons for view changes and the menu

Graphically, looks nice and meets the requirements! I can't speak to the code as I don't know JS.
I only have 1 comment and 1 question, neither of which should prevent this from moving forward.

In the settings pop-up in the lower right, it has 3 default views, a similar sized blank space, then log-out. What is the blank space for?
The settings pop-up in the lower right feels slow as it goes through the animation.

I found a bug. It seems like if you go directly to the dashboard, then use the "Logout" button in the settings, then login, then click settings again, the menu doesn't inflate.

The blank space is this "I also added a temporary spacer to the menu so hopefully it looks a little less cramped overall?"
The menu can not inflate if there's a javascript error earlier, so check the developer console if that happens again. I'll make the method a little better to hopefully avoid that, though. I can speed up the animation at the same time.

alright, i'll take a look tonight

Added #29 addressing those.

Here's a screenshot of the bug. The weird black rectangle is the inflated menu.

I totally put all the comments about the inflation but in #29 on accident.

Spoke too soon, was able to reproduce the bug once again. No issues present in the console, besides a warning about google maps api:
util.js:211 Google Maps API warning: NoApiKeys https://developers.google.com/maps/documentation/javascript/error-messages#no-api-keys

Which branch did you find that on? Shouldn't happen on #29 's

The ui bug or the api warning? Because I found both on #29

The UI bug. API warning doesn't matter. I haven't been able to reproduce it though and there's really no reason it should happen on that branch. It happens when it doesn't know what the final height and width are, which is entirely unhelpful for why it might not know. There shouldn't be any reason why it both wouldn't know and still appear as 0px x 10px on that branch.

@logicplace I fixed the UI issue on #29, added a commit.

I have re-checked this branch after the changes and now seem to get several errors.

If I open the login page through my browser via drag-and-drop:
a - Log in
b - Next page displays "Oops! Something went wrong" with the following error in the console:
Google Maps API error: MissingKeyMapError https://developers.google.com/maps/documentation/javascript/error-messages#missing-key-map-error_.ab @ js?callback=readyMap:35

If I launch the login through an IDE (PyCharm currently)
a - Log in
b - Map loads
There is no map overlay. Also, the 'gear' settings icon is similar to the screen shot above.

Apparently Google changed their requirements yesterday which probably accounts for these errors.
Google requirements
Where does that leave this branch?

I vote we go ahead and merge this - Eric's item should fix the map issue, and I'm pretty sure his code piggybacks off this implementation. (Unless my understanding is incorrect.)

Wait! It's already merged. :facepalm: I'm going to close this ticket, then.


"utils: error: expected one CNI driver service, but found: 0" logs constantly in network-manager logs.

Fresh install of rancher-server version - v1.6.11-rc9
Set up with 3 hosts.
Have few containers deployed.
Network-manager logs shows these errors constantly:
11/10/2017 10:55:30 AMtime="2017-11-10T18:55:30Z" level=error msg="utils: error: expected one CNI driver service, but found: 0"
11/10/2017 10:55:45 AMtime="2017-11-10T18:55:45Z" level=error msg="utils: error: expected one CNI driver service, but found: 0"
11/10/2017 10:56:01 AMtime="2017-11-10T18:56:01Z" level=error msg="utils: error: expected one CNI driver service, but found: 0"
11/10/2017 10:56:16 AMtime="2017-11-10T18:56:16Z" level=error msg="utils: error: expected one CNI driver service, but found: 0"
11/10/2017 10:56:31 AMtime="2017-11-10T18:56:31Z" level=error msg="utils: error: expected one CNI driver service, but found: 0"
11/10/2017 10:56:46 AMtime="2017-11-10T18:56:46Z" level=error msg="utils: error: expected one CNI driver service, but found: 0"
11/10/2017 10:57:01 AMtime="2017-11-10T18:57:01Z" level=error msg="utils: error: expected one CNI driver service, but found: 0"

Fresh install of rancher-server version - v1.6.11-rc9 with network-manager:0.7.16 does not show these errors in logs.


Implemented test verification

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5  👈
#4
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CatChen and the rest of your teammates on  Graphite


Will Not Build on GHC 8 with Hackage Dependencies

Will not build on current Haskell Platform version with stricter dependencies on Hackage.

Builds successfully with stricter version ranges with Haskell Platform 8.0.1 MINIMAL. Please use the minimal version (which is currently also the recommended one on the Haskell Platform website for Windows) rather than the full version.


move_topic_to_stream: Migrate modal to dialog_widget.

I noticed a bug here, where if I just clicj Save changes button without changing anything, the error is shown (which is correct) but the spinner is visible on the button indefinitely and the button is disabled. We should hide the spinner and re-enable the button.

@sahil839 Fixed

I tested and it works fine. Code also looks good.

This seems to have "Save changes" as the button; where does that string come from?  Micromodal?  I think we want to use "Confirm" for this widget as was done in main; I'll change it here, but I'd like us to avoid using any strings that don't appear in git grep in the codebase, so we should override this Micromodal default more broadly.
(More generally, please flag UI changes to strings like this!  I spend a lot of time reviewing this doing QA to try to identify unintentional differences like that, and it helps a lot if one can pre-think about that before testing)

I opened #20209 and #20210 for follow-up bugs I noticed while reviewing this.
I don't think either are new in this PR, so I'm merging this after the button label change mentioned above.  Thanks for doing this migration!

Oh, I guess it is our code; not sure why I missed it in the grep:
static/js/dialog_widget.js:    const html_submit_button = conf.html_submit_button || $t_html({defaultMessage: "Save changes"});

That's probably fine; it might be nicer to have a conceptual flag for which to use rather than passing the string as an override, but that'd be a small API cleanup.


Missing binding to size in Modal component

Detailed description

Describe in detail the issue you're having.

The Modal component is missing binding to the size property.
https://www.carbondesignsystem.com/components/modal/style#sizes

Is this a feature request (new component, new icon), a bug, or a general issue?

This is a bug.

Is this issue related to a specific component?

The modal component.

What did you expect to happen? What happened instead? What would you like to see changed?

Setting the size should reflect change on modal size.

What version of the Carbon Design System are you using?

Version 3.32.1.

These specs are brand new (not a bug) - parent issue for modal updates can be found here: #994


Feature request : Optionnal global searchForm

{!! $grid->renderSearchForm() !!}
is not always wanted, but ATM, we cannot enable/disable it per grid.

Noted. Looking into this

I've added a fix;
Try creating your grid with;
class UsersController extends Controller
{
    /**
     * Display a listing of the resource.
     *
     * @param UsersGridInterface $usersGrid
     * @param Request $request
     * @return \Illuminate\Http\Response
     */
    public function index(UsersGridInterface $usersGrid, Request $request)
    {
        // the 'query' argument needs to be an instance of the eloquent query builder
        // you can load relationships at this point
        return $usersGrid
                    ->create(['query' => User::query(), 'request' => $request])
                    ->withoutSearchForm()
                    ->renderOn('welcome'); // render the grid on the welcome view
    }
}

Or simply just override the property $shouldRenderSearchForm on any grid, and set it to true/false

If youre using 2.0.x-dev, running composer update should get you this fix.

I tried 2.0.x-dev, but the fix is not grabbed ?

Aah, forgot to merge that fix to master. Try again in a few minutes...

Works fine !


first commit support for dcu

thanks @zhangxiao-stack , I send an invitation for write permission on this project. You can push your code to a branch inside this project and update the submodule commit of tf_community in #790 for e2e test.


Design The Api Tree Menu Component【2000GB】

Reqiurement

Support filter
Support add
Support listener

modify the route query param when click menu

don't use @circlon/angular-tree-component. because  it is a little slow while expand a big node( eg. a node have 2000 children).

note the tree is empty array.should have different preview

should display the loading status while get the menu data


using nginx as backend with reverse proxy example h2 or otherwise

@Duslia video of problem
https://tempclip.com/en/JVJWcrdlTUmKXRM/watch
using nginx as backend with reverse proxy example
100x multiple reloads for 404 pages
https://www.digitalocean.com/community/tutorials/how-to-create-a-self-signed-ssl-certificate-for-nginx-in-ubuntu-22-04
pls try with nginx as port 88 as in the script below
/etc/nginx/sites-enabled/default
server {

        listen 88 ssl http2 default_server;
        listen [::]:88 ssl http2 default_server;

        include snippets/self-signed.conf;
        include snippets/ssl-params.conf;
        root /var/www/html;
        index index.html index.htm index.nginx-debian.html;

        server_name _;

        location / {
                try_files $uri $uri/ =404;
        }

}

error message will only show up / appear after 1 minute or more.
2022/12/21 08:41:26.277858 client.go:1840: [Info] http2: Transport readFrame error on conn 0xc0001eda00: (*errors.errorString) EOF
2022/12/21 08:41:27.151528 client.go:1840: [Info] http2: Transport readFrame error on conn 0xc000310ea0: (*errors.errorString) EOF
2022/12/21 08:41:27.764772 client.go:1840: [Info] http2: Transport readFrame error on conn 0xc0003111e0: (*errors.errorString) EOF

main.go
package main

import (
        "context"
        "crypto/tls"
        "fmt"
        "log"
        "time"

        "github.com/cloudwego/hertz/pkg/app"
        "github.com/cloudwego/hertz/pkg/app/client"
        "github.com/cloudwego/hertz/pkg/app/server"
        "github.com/cloudwego/hertz/pkg/network/standard"
        "github.com/hertz-contrib/http2/config"
        "github.com/hertz-contrib/http2/factory"

        "github.com/hertz-contrib/reverseproxy"
)

func main() {

        cfg := &tls.Config{
                MinVersion:       tls.VersionTLS12,
                CurvePreferences: []tls.CurveID{tls.X25519, tls.CurveP256},
                CipherSuites: []uint16{
                        tls.TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305,
                        tls.TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384,
                        tls.TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256,
                },
        }

        cert, err := tls.LoadX509KeyPair("server.crt", "server.key")
        if err != nil {
                fmt.Println(err.Error())
        }
        cfg.Certificates = append(cfg.Certificates, cert)
        h := server.New(server.WithHostPorts(":443"), server.WithALPN(true), server.WithTLS(cfg))

        h.AddProtocol("h2", factory.NewServerFactory(
                config.WithReadTimeout(time.Minute),
                config.WithDisableKeepAlive(false)))
        cfg.NextProtos = append(cfg.NextProtos, "h2")

        h.Use(func(cc context.Context, ctx *app.RequestContext) {
                ProxyHandlerHertz(cc, ctx)
        })
        h.Spin()
}

func ProxyHandlerHertz(cc context.Context, ctx *app.RequestContext) {
        rp, _ := reverseproxy.NewSingleHostReverseProxy("https://<IP_ADDRESS>:88/")

        h2client, _ := client.NewClient()
        h2client.SetClientFactory(factory.NewClientFactory(
                config.WithDialer(standard.NewDialer()),
                config.WithTLSConfig(&tls.Config{InsecureSkipVerify: true}),
        ))
        rp.SetClient(h2client)
        log.Printf("\nctx = %s",ctx)
        rp.ServeHTTP(cc, ctx)
}

Can you click here?

Besides, you can add two logs like this

And take a look at the log output when blank pages appear

i've decided to not use reverse proxy feature with nginx as backend now coz of this issue.
i'm writing a custom reverse proxy backend not using http. so maybe i wont test this for now.
however, u shld look into this issue with nginx, it will definitely not work.
if the "github-actions" can extend 1 mth, i will test when i have time and report, else pls help make sure this issue can be resolved.

@Duslia i've decided to not use reverse proxy feature with nginx as backend now coz of this issue. i'm writing a custom reverse proxy backend not using http. so maybe i wont test this for now. however, u shld look into this issue with nginx, it will definitely not work.
if the "github-actions" can extend 1 mth, i will test when i have time and report, else pls help make sure this issue can be resolved.

Follow the template of the issue and the issue will be normal.💕

@Duslia i've decided to not use reverse proxy feature with nginx as backend now coz of this issue. i'm writing a custom reverse proxy backend not using http. so maybe i wont test this for now. however, u shld look into this issue with nginx, it will definitely not work.
if the "github-actions" can extend 1 mth, i will test when i have time and report, else pls help make sure this issue can be resolved.

I remember that you can copy response to server response instead of reusing it. You can copy the reverse proxy code and copy the response to the server, like this:

I think you can solve this bug by doing this.

@Duslia totally clueless about what u mentioned.
can u just help with adding the code inside here?
i'm too used to fasthttp way so no idea what u mean really.
also, i hope not to do anything that may break upgrade changes.
pls provide the code by expanding below.
i'm sure i wont be the one to ask this question when hertz becomes popular in future.
package main

import (
        "context"
        "crypto/tls"
        "fmt"
        "log"
        "time"

        "github.com/cloudwego/hertz/pkg/app"
        "github.com/cloudwego/hertz/pkg/app/client"
        "github.com/cloudwego/hertz/pkg/app/server"
        "github.com/cloudwego/hertz/pkg/network/standard"
        "github.com/hertz-contrib/http2/config"
        "github.com/hertz-contrib/http2/factory"

        "github.com/hertz-contrib/reverseproxy"
)

func main() {

        cfg := &tls.Config{
                MinVersion:       tls.VersionTLS12,
                CurvePreferences: []tls.CurveID{tls.X25519, tls.CurveP256},
                CipherSuites: []uint16{
                        tls.TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305,
                        tls.TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384,
                        tls.TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256,
                },
        }

        cert, err := tls.LoadX509KeyPair("server.crt", "server.key")
        if err != nil {
                fmt.Println(err.Error())
        }
        cfg.Certificates = append(cfg.Certificates, cert)
        h := server.New(server.WithHostPorts(":443"), server.WithALPN(true), server.WithTLS(cfg))

        h.AddProtocol("h2", factory.NewServerFactory(
                config.WithReadTimeout(time.Minute),
                config.WithDisableKeepAlive(false)))
        cfg.NextProtos = append(cfg.NextProtos, "h2")

        h.Use(func(cc context.Context, ctx *app.RequestContext) {
                ProxyHandlerHertz(cc, ctx)
        })
        h.Spin()
}

func ProxyHandlerHertz(cc context.Context, ctx *app.RequestContext) {
        rp, _ := reverseproxy.NewSingleHostReverseProxy("https://<IP_ADDRESS>:88/")

        h2client, _ := client.NewClient()
        h2client.SetClientFactory(factory.NewClientFactory(
                config.WithDialer(standard.NewDialer()),
                config.WithTLSConfig(&tls.Config{InsecureSkipVerify: true}),
        ))
        rp.SetClient(h2client)
        log.Printf("\nctx = %s",ctx)
        rp.ServeHTTP(cc, ctx)
}

@Duslia can u help this out? i'm curious what's the best way to do this... seems there will be memory copying.
pls provide the code just to verify if it will work according to what u mean.

No. I wouldn't help you write the code.


Run javadoc without linting

This PR relates to issue https://github.com/jbee/silk/issues/83 -- via a hand-crafted javadoc call that actually generates the API documentation and also shows where those errors and warnings are coming from.

We can have a look at the errors/warnings later. Currently I want to focus on the transition both with the build and to next version 19.1. If both are more stable I make a sweep cleaning javadoc issues.

Okay, I'll add a .jitpack.yml configuration file next. With that the deploy process is covered and provides a proof-of-concept basis for deploying to Maven Central later.


Is it possible to downgrade requirements?

Is it possible to run your  library on Android Studio 2.3.3? My gradle settings are:
In project level build.gradle
classpath 'com.android.tools.build:gradle:2.3.3'

In app level build.gradle
compileSdkVersion 25
buildToolsVersion "25.0.3"

and in gradle-wrapper.properties
distributionUrl=https\://services.gradle.org/distributions/gradle-3.3-all.zip

I can't compile project with above settings:

and I can't upgrade gradle plugin to 4.1 staying in Android Studio 2.3.3

@isabsent Can I know why can't you update to 4.1?

I can upgrade to 4.1 but  would not like to do it right now, because I would loose some opportunities I have in 3.3 and have not in 4.1.

@isabsent I'm afraid you can do that right now.
But, I have a good news for you. If you are using kotlin, you can use the new QuickPermissions-Kotlin that works out of the box without any gradle plugins and works with instant run.


Update Java app examples to use new PKCS8 flag

Fixes #4071.
Summary of changes:

Update Hibernate & JDBC sample apps to use auto-generated key name

Update instructions to remove the openssl key-munging step

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/8e7272435584626d923805bbc3aba4bd2c5978e8/
Edited pages:

v19.1/build-a-java-app-with-cockroachdb-hibernate.md
v19.1/build-a-java-app-with-cockroachdb.md
v19.1/create-security-certificates.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/6e302556d18ca4b7d36e06947b150647cb792968/
Edited pages:

v19.1/build-a-java-app-with-cockroachdb-hibernate.md
v19.1/build-a-java-app-with-cockroachdb.md
v19.1/create-security-certificates.md


Opening any doc display it with incorrect theming

Godot version
006915b4824a91c0e0fc95ac6aa2f70908a0a027
System information
Windows 11
Issue description

Steps to reproduce
Open any doc item in the editor help
Minimal reproduction project
No response

CC @AaronRecord

Good catch, I'll try and look into tomorrow

Works fine on Ubuntu 22.04 LTS

I can reproduce the problem on my computer (also Win 11, although I'm guessing that's unrelated?)

Okay so this is weird, I can't reproduce with a debug build compiled with msvc, but I can reproduce with the release builds from the CI checks (both on commit cb62d3193)

If I switch themes in the editor settings, everything goes back to normal, I think it's a regression related to themes or related notifications. It may be regression introduced by https://github.com/godotengine/godot/pull/62846 ?

If I switch themes in the editor settings, everything goes back to normal, I think it's a regression related to themes or related notifications. It may be regression introduced by #62846 ?

It is a regression from #62846

If I switch themes in the editor settings, everything goes back to normal, I think it's a regression related to themes or related notifications. It may be regression introduced by #62846 ?

It is a regression from #62846

Don't know if you want a new issue for it, but #62846 causes an infinite loop segfault when I try to open one of my current projects. I haven't been able to pin down exactly what's causing it, but EditorLog is asked to log an error, add_image fails due to the theme_cache not being properly initialised, which leads to a new error log, and so on. I added a couple of if (icon != nullptr) in around add_image EditorLog to avoid the loop, but instead it got stuck in some other loop instead..


Save numberOfRequests per stub even if --mock is disabled

Right now, unless you run mb with the --mock parameter, the "# Requests" column isn't populated in the imposters UI page, because it relies on capturing each request. Adding a numberOfRequests would allow us to show the correct number without adding a memory leak.

Is this exposed anywhere externally? I can't find it in the json returned when querying an imposter.

Hi Colin,
Yes, it's at the root level of the imposter. What version are you using?
See sample output below:
MacBook-Pro-105:mountebank:% curl -d'{"protocol":"http","port":3000}'
http://localhost:2525/imposters
{
"protocol": "http",
"port": 3000,

"numberOfRequests": 0*,
"requests": [],
"stubs": [],
"_links": {
"self": {
"href": "http://localhost:2525/imposters/3000"
}
}
}%

MacBook-Pro-105:mountebank:% curl http://localhost:3000/
MacBook-Pro-105:mountebank:% curl http://localhost:3000/
MacBook-Pro-105:mountebank:% curl http://localhost:2525/imposters/3000
{
"protocol": "http",
"port": 3000,

"numberOfRequests": 2*,
"requests": [],
"stubs": [],
"_links": {
"self": {
"href": "http://localhost:2525/imposters/3000"
}
}
}%

On Fri, Feb 15, 2019 at 1:32 PM Colin Schoen<EMAIL_ADDRESS>wrote:

Is this exposed anywhere externally? I can't find it in the json returned
when querying an imposter.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bbyars/mountebank/issues/170#issuecomment-464171782,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAESv2omavZg3XREJ5-Q51DJcHJ_C3lHks5vNwtngaJpZM4LBc_7
.

That is number of requests per imposter rather than per stub. I was under the impression this ticket was tracking number of requests for each individual stub.

Ah sorry, the issue title was confusing. The original issue was to fix a
problem in the UI when the —mock flag wasn’t passed.
Do you need it at the stub level even without —mock?
On Fri, Feb 15, 2019 at 6:17 PM Colin Schoen<EMAIL_ADDRESS>wrote:

That is number of requests per imposter rather than per stub. I was under
the impression this ticket was tracking number of requests for each
individual stub.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bbyars/mountebank/issues/170#issuecomment-464258307,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAESvyITXShAgTiYHVAUyyyTZ5qlmcqwks5vN04egaJpZM4LBc_7
.

--
Sent from Gmail Mobile


Mobile Search

Hi, first of all  thanks for the great package
I notice in mobile there's a little error, search results appear behind, wouldn't be better to include the search field inside the sidebar? like in the sample image, I uploading also a image notes some things I notice in the mobile version, thanks for everything

or maybe this is a better idea, and just include extra links and versions on the sidebar in the bottom, just an idea

Thank you @nam-co for sharing the bug
Now solved by @Maen1 and attached to a new release v1.3.5 💃

Thanks @Maen1 and @saleem-hadad


⚠ BCR Bot has degraded performance

In eb8980c, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 5719 ms

Resolved: BCR Bot performance has improved in ded928d.


update mysql driver from 5.1.9 to 8.0.16 communication link failure

Accepted github dependabot PR to the latest MySQL JDBC driver has error.  Reverting for now, research later.
org.apache.ibatis.exceptions.PersistenceException:
### Error querying database.  Cause: org.springframework.jdbc.CannotGetJdbcConnectionException: Failed to obtain JDBC Connection; nested exception is com.mysql.cj.jdbc.exceptions.CommunicationsException: Communications link f
ailure

The last packet sent successfully to the server was 0 milliseconds ago. The driver has not received any packets from the server.
### The error may exist in page1.xml
### The error may involve com.mycompany.dao.Page1Dao.getPages1ByCustomer
### The error occurred while executing a query
### Cause: org.springframework.jdbc.CannotGetJdbcConnectionException: Failed to obtain JDBC Connection; nested exception is com.mysql.cj.jdbc.exceptions.CommunicationsException: Communications link failure

The last packet sent successfully to the server was 0 milliseconds ago. The driver has not received any packets from the server.
        at org.apache.ibatis.exceptions.ExceptionFactory.wrapException(ExceptionFactory.java:26)
        at org.apache.ibatis.session.defaults.DefaultSqlSession.selectList(DefaultSqlSession.java:111)
        at org.apache.ibatis.session.defaults.DefaultSqlSession.selectList(DefaultSqlSession.java:102)
        at org.apache.ibatis.binding.MapperMethod.executeForMany(MapperMethod.java:119)
        at org.apache.ibatis.binding.MapperMethod.execute(MapperMethod.java:63)
        at org.apache.ibatis.binding.MapperProxy.invoke(MapperProxy.java:52)

Tried a couple things, same error.
a. change driver in fortress.properties:
database.driver=com.mysql.jdbc.Driver to:  com.mysql.cj.jdbc.Driver
b. update dependencies:

org.mybatis
mybatis
3.5.5 to:  3.2.7

mysql
mysql-connector-java
5.1.9 to: 8.0.16

org.mybatis
mybatis-spring
2.0.5 to: 1.2.2

resolved


[KYUUBI #1102] Add test cases for error and exception request

Why are the changes needed?

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #1103 (fdd0d57) into master (9dfbd2b) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #1103      +/-   ##
============================================
- Coverage     78.80%   78.80%   -0.01%
  Complexity       80       80
============================================
  Files           175      175
  Lines          6451     6453       +2
  Branches        764      764
============================================
+ Hits           5084     5085       +1
- Misses          923      925       +2
+ Partials        444      443       -1

Impacted Files
Coverage Δ

.../apache/kyuubi/server/api/v1/ApiRootResource.scala
40.00% <0.00%> (-26.67%)
:arrow_down:

...pache/kyuubi/sql/KyuubiQueryStagePreparation.scala
81.37% <0.00%> (+0.98%)
:arrow_up:

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 9dfbd2b...fdd0d57. Read the comment docs.

@yaooqinn added some exception requests verify and refactor, please review.

thanks, merged to master


Use BlockTime rather than Time

Moving to faster block times may mean that BlockTime could include fractional seconds in the future.
Since using BlockTime versus Time is still logically equivalent we may as well already be using BlockTime instead..
So why this PR? The purpose is to make the future PoA v2 PR smaller by addressing trivial (non-logic-changing) updates separately.
There will be future PRs involving mining timestamp (and the use of BlockTime instead of Time) such as an updated https://github.com/stratisproject/StratisFullNode/pull/310.

Looks good, is this backward compatible with the current code? It appears that way... just want you to confirm.
Yes, that is the intent.


set the metadata_expire option on yum systems

Fixes https://github.com/keybase/keybase-issues/issues/2816.
r? @cjb

These imported chromium scripts support some more obscure RPM-based package managers (yast and urpmi), but I don't know if they support this option, and my preference is not to touch them.

I don't think this will fix existing installs, because the cronjob sets repo_add_once=false in /etc/default/keybase when it runs, which prevents either the cronjob or the postinst hook from calling the install_yum routine. I don't plan to touch this logic either.

Tested that installing a package with these changes applies correctly to /etc/yum.repos.d/keybase.repo, but only after deleting /etc/default/keybase.


Github Action build not signing AppX package

Describe the bug
In the automatic build of this project, the AppX package was not properly signed, which caused the .appx format installation file downloaded in Release to fail to install due to certificate issues.
To Reproduce
Steps to reproduce the behavior:

Download .appx file from release
Double-click it
Windows warns that it is an untrusted application and therefore cannot be installed

Expected behavior
It should be able to be installed normally, just like the version in exe or msi format.
Screenshots

Additional context
After checking the Github Action build process for this project, I located the problem here
https://github.com/RocketChat/Rocket.Chat.Electron/blob/1b6ba4974b826c88b4948900e77c488c2deb9087/workspaces/desktop-release-action/src/windows.ts#L11
It does not properly configure the environment variables required for digital signatures, like
https://github.com/RocketChat/Rocket.Chat.Electron/blob/1b6ba4974b826c88b4948900e77c488c2deb9087/workspaces/desktop-release-action/src/windows.ts#L6-L9
You can check the Github Action log to check this issue
https://github.com/RocketChat/Rocket.Chat.Electron/runs/3539521579?check_suite_focus=true#step:10:53

Please test if the problem persists with our latest release. I will be closing the issue but if there is any problem yet, we will open it again.

In the current latest release (3.8.7), this problem still exists.
https://github.com/RocketChat/Rocket.Chat.Electron/releases/download/3.8.7/rocketchat-3.8.7-win-x64.appx
https://github.com/RocketChat/Rocket.Chat.Electron/runs/6713607085?check_suite_focus=true#step:10:61

It still exists with 3.9.14.
This app package’s publisher certificate could not be verified. Contact your system administrator or the app developer to obtain a new app package with verified certificates. The root certificate and all immediate certificates of the signature in the app package must be verified (0x800B010A)

It still exists with 3.9.14.
This app package’s publisher certificate could not be verified. Contact your system administrator or the app developer to obtain a new app package with verified certificates. The root certificate and all immediate certificates of the signature in the app package must be verified (0x800B010A)

@cmouse we are using the .appx installers to publish on Microsoft Store. They don't complain about that, what could be different that you need that Microsoft store didn't asked?

I have no idea, I was using the asset from your release page. I have pretty vanilla windows 11 with no special things, and I get that message when I tried to install the appx package.

I have no idea, I was using the asset from your release page. I have pretty vanilla windows 11 with no special things, and I get that message when I tried to install the appx package.

But why you dont use MSI or EXE installer? APPX is just for distribution as far as I know.

If it is not intended to be used should this be mentioned somewhere?

Sorry, where it is mentioned? We generate it to send to Microsoft Store.

You publish it in your release page. Nothing says one should not use it, so I wonder if you could add a note about this that it is not supposed to be used.
I feel like you feel somehow offended, it's not my intention to do so. If you feel this is not useful discussion, feel free to close this issue.

You publish it in your release page. Nothing says one should not use it, so I wonder if you could add a note about this that it is not supposed to be used.
I feel like you feel somehow offended, it's not my intention to do so. If you feel this is not useful discussion, feel free to close this issue.

No no, not offended at all. I want to understand what we can do to improve this. We publish all the packages that we use, but people usually use what installer they need. I can take a look if we can sign the appx packages, but they are mainly generated to be published by Microsoft Store.


Android -  When user tapped on the eye icon in password field the keyboard changes @Puneet-here

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:

Sign out > Enter Email > Continue
Focus on password field
Check the keyboard it has a numbers row at the top.
Now tap on the eye, the keyboard will change and you won't see the number row.

Expected Result:
The keyboard shouldn't change, it should have the number row.
Actual Result:
The keyboard changes.
Workaround:
Unknown
Platform:

Where is this issue occurring?

Android

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/173586659-f0fcdfa5-abc3-4ef2-bece-35d815f372c6.mp4
Expensify/Expensify Issue URL:
Issue reported by: @Puneet-here
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1652896569021399
View all open jobs on GitHub

Checking for volunteers to triage, as I do not have an Android device

@cristipaval was able to reproduce it! Thanks so much!

PROPOSAL
Problem
This issue can be reproduced on virtual device. When we tap on the eye, the keyboard suggestions show.
The secureTextEntry props of TextInput is true in PasswordForm component, inputType will set to InputType.TYPE_TEXT_VARIATION_PASSWORD on native side.
When we tap on the eye, the secureTextEntry props of TextInput change to false, and inputType unset with InputType.TYPE_TEXT_VARIATION_PASSWORD on native side. It means keyboardType change to default.
Solution
When we change secureTextEntry to false, also set keyboardType props of TextInput to visible-password. Since this value works on Android only, we need to do some modification.
1.Copy src/components/TextInput/index.native.js to src/components/TextInput/index.ios.js, keep iOS unchanged.
2.Add getKeyboardType props to BaseTextInput in src/components/TextInput/index.native.js
https://github.com/Expensify/App/blob/5b0cd481c87e89772f7d21fdbad6bf815b495b15/src/components/TextInput/index.native.js#L7-L15
+ getKeyboardType={(passwordHidden) => props.secureTextEntry && !passwordHidden ? 'visible-password' : props.keyboardType}

3.Rename src/components/TextInput/index.native.js to src/components/TextInput/index.android.js, so the changes only apply to Android.
4.Add keyboardType to src/components/TextInput/BaseTextInput.js.
https://github.com/Expensify/App/blob/5b0cd481c87e89772f7d21fdbad6bf815b495b15/src/components/TextInput/BaseTextInput.js#L195-L201
+ const keyboardType = this.props.getKeyboardType ? this.props.getKeyboardType(this.state.passwordHidden) : this.props.keyboardType;

https://github.com/Expensify/App/blob/5b0cd481c87e89772f7d21fdbad6bf815b495b15/src/components/TextInput/BaseTextInput.js#L272-L274
+ keyboardType={keyboardType}

5.When we tap on the eye, the keyboard will automatically capitalize, we don't want that, set autoCapitalize to none in PasswordForm to fix it.
https://github.com/Expensify/App/blob/5b0cd481c87e89772f7d21fdbad6bf815b495b15/src/pages/signin/PasswordForm.js#L139
+ autoCapitalize="none"

I got confirmation from @parasharrajat that it is OK to work on this one due to some of the confusing status of your other issues.
The proposal above looks OK to me 👍

@stephanieelliott I think the next step here is to hire them in Upwork?

Sorry for the wait, I've been out at a conference!
Upwork job is here: https://www.upwork.com/jobs/~01e107e5f916abe7ab
@b1tjoy please apply for the job in Upwork

Looks like I got assigned after the proposal was accepted. Will be reviewing the PR.

@b1tjoy Do you have any updates for me on this one?

@mananjadhav I will post pull request soon, in less than 12 hours.

Hey @mananjadhav and @b1tjoy can you grab the job on Upwork when you get a chance? https://www.upwork.com/jobs/~01e107e5f916abe7ab

PR submitted, this is just waiting on internal and C+ review!

Reassigning this one as I am going OOO, thanks @kadiealexander!

The PR https://github.com/Expensify/App/pull/9593 has been successfully merged.
My upwork proposal status was declined by client, should I do anything about it?

@b1tjoy could you please link your Upwork profile here so I can take a look at your Upwork proposal and see what happened?

@kadiealexander Thank you! My upwork profile https://www.upwork.com/freelancers/~015f3ce165fe53a62f

Thanks @b1tjoy! I'm not sure what happened there sorry! I've created a new job post just for this here https://www.upwork.com/jobs/~01e1db0a026ba15120 and sent you a hire request.

Everyone is paid! Thanks team!


MUMUP-2252 Add UWRF Directory Search

Coverage remained the same at 15.784% when pulling 68e50e4e471f087cd8aa23a8a3232cab6b310dc1 on uw-master into d80dcdce3a23842be80fdb29e0ff0000d18972d9 on master.

We verified results when 0 state, error state, and too many state.
:+1:


Added explanation on abstract classes

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@m-s-yusuf Thanks for your contribution. This didn't seem to add that much to the explanation. Also, we shouldn't be using Latin expressions like i.e. - these articles should be as plain-English as possible.


[SPARK-19562][BUILD] Added exclude for dev/pr-deps to gitignore

What changes were proposed in this pull request?
Just adding a missing .gitignore entry.
How was this patch tested?
Entry added, now repo is not dirty anymore after running the build.

Can one of the admins verify this patch?

Test build #3574 has started for PR 16904 at commit 855f3e0.

Test build #3574 has finished for PR 16904 at commit 855f3e0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.


[next-react-query-tailwind] Lock dependency versions

Resolves #225

Let's close this as a dupe of #269 then as it will be resolved there.