Allow updating statuses in the translation UI

Description
In #3124, we render statuses based on the configured program, but attempting to save changes doesn't actually update the statuses in the database. This change achieves that.
As part of this, the ProgramTranslationForm needed to be refactored since there are a dynamic number of statuses for a program and we can't easily have a unique field name. I instead exposed this through using a DynamicForm (there's precedent of using this for API key CiviForm Admin UI) and wrapping up the field handling in that class so downstream code can just get a set of LocalizationUpdate.StatusUpdate objects.
Some notes:

We continue to not really do any validation in the form itself (outside of expecting fields to be present). This responsibility lies in ProgramService
For simplicity, if any of the program status is out of sync with the configured statuses, we fail the request and reload the translation UI with an error message. We could try and "patch" the statuses we could find, but that seems to add unnecessary complexity for the uncommon "stale tab" scenario.
LocalizedStrings previously had no way to clear translations for a given locale. I added that for this specific use case and left a TODO against cleanup bug to consolidate our other usages to support this.

Testing notes:
The added browser test is temporarily skipped until feature flags can be controlled in browser tests. Prior to disabling the test, I allowed CI to run with the test enabled, which was successful (https://github.com/civiform/civiform/runs/7812004315?check_suite_focus=true).
Additionally, I set application_status_tracking_enabled = false in application.dev.conf and application.test.conf to simulate prober environment state and confirmed that tests pased.
Release notes:
N/A
Checklist

[X] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >
[X] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Issue(s)
Fixes #2754; #3144

Codecov Report

Merging #3143 (b62320d) into main (ca0b9b7) will increase coverage by 0.29%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main    #3143      +/-   ##
============================================
+ Coverage     77.92%   78.21%   +0.29%
- Complexity     2185     2204      +19
============================================
  Files           277      279       +2
  Lines          8721     8810      +89
  Branches        591      596       +5
============================================
+ Hits           6796     6891      +95
+ Misses         1701     1696       -5
+ Partials        224      223       -1

Impacted Files
Coverage Δ

...lers/admin/AdminProgramTranslationsController.java
79.16% <100.00%> (+7.73%)
:arrow_up:

.../app/forms/translation/ProgramTranslationForm.java
100.00% <100.00%> (ø)

server/app/services/LocalizedStrings.java
90.90% <100.00%> (+0.52%)
:arrow_up:

...erver/app/services/program/LocalizationUpdate.java
100.00% <100.00%> (ø)

...p/services/program/OutOfDateStatusesException.java
100.00% <100.00%> (ø)

...erver/app/services/program/ProgramServiceImpl.java
89.43% <100.00%> (+1.05%)
:arrow_up:

server/app/services/program/StatusDefinitions.java
94.73% <100.00%> (ø)

server/app/services/program/ProgramDefinition.java
87.98% <0.00%> (+0.96%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

LGTM, just the discussion on a few threads


Fix Invalid endpoint error in Ch 02 lab

When attempting to create a boto3.client, the endpoint_url parameter expects "https://" as part of the URL.  The result is that the following error is thrown when executing step 4 of the notebook:
ValueError: Invalid endpoint: minio-api-projects-data.apps.ocp4.example.com
The value is grabbed from the AWS_S3_ENDPOINT environment variable.

I'm closing this because the accompanying coursebook instructions already include the https:// prefix when defining the endpoint in the data connection.


schedule() command doesn't defaulty take into account instruction map.

Environment

Qiskit Terra version: 0.25.0
Python version: 3.10
Operating system: Win11

What is happening?
schedule() command fails when the circuit includes 'measure' with the error message
schedule(circuit, backend)

qiskit.pulse.exceptions.PulseError: "Operation 'measure' exists, but is only defined for qubits [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26]."

The command works when specifying the default instruction map
schedule(qc, backend, backend.defaults().instruction_schedule_map, meas_map=backend.configuration().meas_map )
How can we reproduce the issue?
 provider = IBMProvider()
 qc = QuantumCircuit(1)
 qc.h(0)
 qc.measure_all()  # or qc.measure(0)
 # Schedule
 qctr = transpile(qc, backend_nisq, initial_layout=None, optimization_level=0)
 sch = schedule(qctr, backend)

What should happen?
default instruction and default measure maps should be defaultly taken into account without having to explicitly specify it in the schedule() command
Any suggestions?
No response

@patdenim ,
I am unable to reproduce the error as shown by you.
This is what I am getting instead!
Could you please confirm if you are getting the same, I would like to work on this issue :)
I have checked it, the same is the problem with qiskit verison 0.44.2 with qiskit-terra version 0.25.2.

@fvarchon , am I allowed to work on this issue?

@MozammilQ Yes for sure :)

@fvarchon , sir, please see if my efforts a good enough :), please let me know if any additional changes are required :)

Thanks @MozammilQ, you've got the point !! fails only when backends are V1.

@to24toro, @nkanazawa1989: was this fully closed by #10988 or is there more to do?

@jakelishman
I think we can close this issue and I will close it.


Wrong value for year

I have just seen this library and i have no benefit in defending it.I was actually checking the issues to know whether i would use it or not.When you click today , it doesn't change any value from the above neither the day nor the month nor the year until you actually select it from the calender.The today button just scrolls to today both in year and month , but as long as you haven't selected it .it wont show on the top.

This is not expected behavior to me.


How to remove -SkipCACheck or use other parameter?

Hi,
I have a question is about the following command in this MS Doc, "-SkipCACheck":
Invoke-Command -ComputerName $HybridEndpoint                -Credential $Credential
-Port 5986                -UseSSL
-ScriptBlock $Script                -ArgumentList "*"
-SessionOption (New-PSSessionOption -SkipCACheck)
Is there any other way to remove "-SkipCACheck" parameter or any other parameter I can use instead of "-SkipCACheck"?

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

ID: 24537450-303d-5978-29e6-1c1e429b63fa
Version Independent ID: e14d342a-8fe4-050d-989a-5e38281663f6
Content: Manage remote on-premises resources by using PowerShell functions
Content Source: articles/azure-functions/functions-hybrid-powershell.md
Service: azure-functions
GitHub Login: @eamono
Microsoft Alias: eamono

When I didn't set up "-SkipCACheck" parameter:
It will show the following error message:

@yansyunchen the error is because the doc walks through the steps using a self-signed certificate. If you are using a certificate that is generated from a Custom CA and it is setup as a trusted root, then you don't need the parameter anymore.

Hi, Pramod,
Thanks for your response. Is that possible set the self-signed in the function app as a trust CA?


Pattern Catching

Been discussed on the Discord before but I didn't see an issue
Prior art includes Swift catch and C# catch-when

It would be nice to flesh out a specific syntax proposal and/or specific features you want. Here are a few possibilities:
1. catch switch
@STRd6 has suggested the following in the past I believe:
try
  //...
catch switch
  <? RangeError
    // like a normal pattern-matching switch

One thing that's not clear with this syntax is how to get the error object. Should catch switch e bind e to the error? It's a little funny-looking (e is a left-hand side here, whereas usually the argument to switch is a right-hand side), but it could work.
Actually, the following form already works!
try
  //...
catch e switch e
  <? RangeError, <? ReferenceError
    //...
  {message: /bad/}
    //...
  else
    // other exception

So a very simple thing would be to drop the requirement of e here, and then we get all the power of pattern matching (including future features).
catch <pattern>
It would also be nice to avoid the word switch. Part of the problem here is that pattern-matching switch doesn't have a word before the cases, like case or when, so it's hard to infer the switch. But here's one possible syntax that might be good: put the pattern (instead of an identifier) right after catch.
try
  //...
catch <? RangeError, <? ReferenceError
  //...
catch {message: /bad/}
  //...
catch e
  // other exception

Multiple catch blocks seems to be the way of Swift, C#, and Python, so this seems reasonable.
catch...when
Alternatively, if we just wanted a limited form of pattern catching that supports, say, types only (like Python's try...except), we could do something like switch...when:
try
  //...
catch e
  when RangeError, ReferenceError
    //...
  when TypeError
    //...
  else
    // other exception

Plausibly we could do multiple of the above proposals, for different situations. I believe they are all mutually compatible. But it'd be nice to see some real examples to better judge.

But it'd be nice to see some real examples to better judge.

Most of my real-world examples are type checking (example 1, example 2, example 3). However, there are cases where I want to check other information about an error than just its type: example from a Civet project, example from a TS project. In both of these cases I still do a type check -- the former requires it so that TS is okay with me accessing the .message property, while the latter is semantically required -- but I also check something about the properties.


Additional opq fixes

Remaining ones look to be related to sret/root returning and whether boxing happens.

Codecov Report
Patch coverage: 75.00% and project coverage change: -0.03 :warning:

Comparison is base (5669c89) 77.69% compared to head (2b49653) 77.66%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #788      +/-   ##
==========================================
- Coverage   77.69%   77.66%   -0.03%
==========================================
  Files          18       18
  Lines        7401     7401
==========================================
- Hits         5750     5748       -2
- Misses       1651     1653       +2

Impacted Files
Coverage Δ

src/compiler.jl
78.27% <75.00%> (-0.02%)
:arrow_down:

... and 1 file with indirect coverage changes

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


Add a dummy "orders" page + proper text for login/resetPassword pages + better design for login/resetPassword pages

I also pushed the branch I worked on this weekend offline. It will have nearly the same wthout the react-router. I think both are ok and we could keep whatever one you preffer.

@Blightwidow merged yours and synced this one, there are few more changes remaining.
PTAL


Numeric keys in structs

When using CBOR based parsers, a common occurrence is that a struct such as
struct ProtocolElement {
    title: String,
    value: u16,
    unit: Option<String>,
}

needs to be deserialized from CBOR that, in diagnostic notation, looks like this:
{
    0: "My title",
    5: 23,
    7: "knots",
}

Right now, implementing this manually is rather complex for a task that's very common with CBOR (and possibly other serialization formats that lean towards numeric keys).
If derivation were allowed as #[serde(rename_numeric = 0)] , these structs would be trivial to use.
Often, CBOR uses numeric keys where  JSON uses string keys (while otherwise sharing the structure). It'd be tempting to suggest the same mechanism for #[serde(alias_numeric = 0)], but that would (even though barely) not allow using the same type for both CBOR and JSON serialization, because while it'd process CBOR and JSON correctly (tolerating JSON-style input on the CBOR side), this would not quite cut it on the serialization side (because we can't tell CBOR to use the alias where JSON uses the name), so I'd keep that aspect out of scope in the first iteration.

I'd like to see this as well. I'd be happy to work on a PR if this sounds like something you'd consider adding.
My two cents: if bikeshedding on the name, I'd rather see rename = 0, rather than a new rename_numeric = 0, since it's more consistent, and it doesn't really complicate parsing AFAICT (in both cases we're just parsing a literal). Happy to be corrected on this though


Failures on line 9

Hi Dave!  I have implemented your script but am unable to get it to work.  In short, when I enter the url for an API in a browser (chrome) I get an expected response.  Take your example: Return the timestamp of the current ISS location.
If I copy and paste the url in chrome ( http://api.open-notify.org/iss-now.json ) I can see a response, ( {"iss_position": {"longitude": "-155.7640", "latitude": "38.3718"}, "message": "success", "timestamp":<PHONE_NUMBER>}) ).
Unfortunately, when I enter the values in the Google sheet exactly as shown in the first example I see an error." Request failed for http://api.open-notify.org/iss-now.json,{{timestamp}} returned code 404 (line 9).
I found this error when I tried a URL with authentication.  I was able to get a response using chrome but when I entered it into the spreadsheet I received the following error message " Request failed for https://api.noodle.com/v1/project_stages?user=xxxxxxx%2540xxxxx.xxx&api_key=e567sakjhilqa9768932bdca007616472a4e6,{{timestamp}} returned code 401. Truncated server response: {"errorcode":1,"message":"The user was not found"} (use muteHttpExceptions option to examine full response) (line 9)."
Note I've changed all the details above so the url shown will not work except for your public example.
So in both instances, the error involves line 9.
Have I missed something?
Thank you,
Rich

Hi Rich, I think I see what is happening based on the error code.  I believe you are entering your formula like this:
=GETJSON("http://api.open-notify.org/iss-now.json,{{timestamp}}")
Instead, you need to ensure that both the URL and the second parameter (template) are both enclosed in their own double quotes like this:
=GETJSON("http://api.open-notify.org/iss-now.json","{{timestamp}}")
You should then be good to go.
Also, my script unfortunately does not currently support URLs with authentication.  The Google Sheets function I am able to leverage (UrlFetchApp) has limited support for authentication since it only supports "basic authentication" which is a weak form of authentication and covers relatively few URL authentication scenarios on the web.

Hi Dave, Nice code, JSON handling is cruelly missing from gsheets...
One addition, and a question.
API KEY authentication :
I was able to modify your code slightly to pass through an API Key and it worked. Like this:
`function JSONtoObject(url) {
var apiKey = "bb18a2c6-30cc-4244-915e-d5ad5bd1f020" ;
var url = encodeURI(url);
var response = UrlFetchApp.fetch(
url,
{
"headers":{
"api-auth":apiKey
}
}
)`
Templates
I am having difficulty make your template input work with this attached JSON example that uses some kind of nested objects or multiple dimensions.  When I use {{ac}} as template, I get entire rows of [object,object] pairs.
Sorry for the question, beginner here, any help appreciated.
https://drive.google.com/file/d/1ztfC3wFwuXFmRBYNJzO_pHt6Cvm7CPLk/view?usp=sharing

@mthrum sorry for the late reply, but I just updated the code to handle your use case which relates to nested arrays.  In your example, you can use the following template to access the postime value in the first record:
{{ac[0].postime}}
To access the postime value in the second record, use {{ac[1].postime}}, etc.


server: remove local rocksdb dsn.

Fix #1206
PTAL @siddontang @BusyJay
Note:
After this PR, we should update tikv startup script by removing -S raftkv. But you don't have to make changes if use a config file before. @iamxy @nolouch @shenli

Removing -S raftkv is not backward-compatible, we should keep it.
We can add a deprecated tag for the flag first.

Rest LGTM

LGTM

LGTM


DemoApp sample report throws error

The demo app sampleReport throws an error:
Response status code does not indicate success: 401 (Unauthorized).
Microsoft.IdentityModel.Clients.ActiveDirectory.AdalServiceException: Response status code does not indicate success: 401 (Unauthorized)
AdalServiceException: AADSTS7000222: The provided client secret keys are expired. Visit the Azure Portal to create new keys for your app, or consider using certificate credentials for added security: https://docs.microsoft.com/azure/active-directory/develop/active-directory-certificate-credentials
Trace ID: db6f3573-78bf-4fc8-8aca-0d2932597600
Correlation ID: 6c707ff9-51eb-4485-a592-e11045e976a9

Same issue:
Chrome Inspector for report XHR Get request:

This issue is fixed.


LOG4J2-2832: Updated strategy name to existing strategy

Instead of using a hard coded name the class name of the interface was used which
a) Increases visibility where the interface is referenced
b) Prevents that the message needs to be adjusted when the interface name changes

@rgoers is there anything todo which prevents this from merging?


Graceful shutdown of run does not work perfectly

🙂 Expected behavior
When I stop sunodo node using CTRL-C that I executed with sunodo run it should terminate gracefully, stop all containers and remove them.
🫠 Actual behavior
What is happening instead?
The containers are stopped and removed, but it is printing the alarming message below:
panic: close of closed channel

goroutine 528 [running]:
github.com/docker/compose/v2/pkg/compose.(*composeService).Up.func2.1.1()
        github.com/docker/compose/v2/pkg/compose/up.go:81 +0xa4
github.com/hashicorp/go-multierror.(*Group).Go.func1()
        github.com/hashicorp/go-multierror@v1.1.1/group.go:23 +0x60
created by github.com/hashicorp/go-multierror.(*Group).Go in goroutine 293
        github.com/hashicorp/go-multierror@v1.1.1/group.go:20 +0x84

It looks like docker compose is raising an exception, which causes the execution to exit with an error code 2.
    Error: Command failed with exit code 2: docker compose --file<EMAIL_ADDRESS>--file
<EMAIL_ADDRESS>--file
<EMAIL_ADDRESS>--file
<EMAIL_ADDRESS>--file
<EMAIL_ADDRESS>--file
<EMAIL_ADDRESS>--project-directory . --project-name af28c486 up --attach
    validator --attach prompt

🧪 Minimal test case
sunodo run
# CTRL-C to stop it

🌎 Environment
macOS Sonoma 14.2.1 (M1)
Docker Desktop 4.26.1 (131620) is currently the newest version available.
Docker Compose version v2.23.3-desktop.2
sunodo 0.10.3
✔ Possible solutions
I trying replacing the 2.23 version of docker compose plugin by the version release yesterday 2.24, thinking that the problem could be similar to this closed issue, but it did not work as well.

Just for the record, this issues doesn't happen on my environment:
❯ docker compose version
Docker Compose version v2.23.3
❯ docker version
Client: Docker Engine - Community
 Version:           24.0.7
 API version:       1.43
 Go version:        go1.20.10
 Git commit:        afdd53b
 Built:             Thu Oct 26 09:07:58 2023
 OS/Arch:           linux/amd64
 Context:           default

Server: Docker Engine - Community
 Engine:
  Version:          24.0.7
  API version:      1.43 (minimum version 1.12)
  Go version:       go1.20.10
  Git commit:       311b9ff
  Built:            Thu Oct 26 09:07:58 2023
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.6.26
  GitCommit:        3dd1e886e55dd695541fdcd67420c2888645a495
 runc:
  Version:          1.1.10
  GitCommit:        v1.1.10-0-g18a0cb0
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

I could reproduce the error on an linux/arm64 environment, maybe it's related to golang/arm64 and we should upstream this bug report.
[ec2-user@ip-10-0-63-164 my-ts-dapp]$ docker compose version
Docker Compose version v2.24.0
[ec2-user@ip-10-0-63-164 my-ts-dapp]$ docker version
Client:
 Version:           24.0.5
 API version:       1.43
 Go version:        go1.20.10
 Git commit:        ced0996
 Built:             Tue Nov 14 00:00:00 2023
 OS/Arch:           linux/arm64
 Context:           default

Server:
 Engine:
  Version:          24.0.5
  API version:      1.43 (minimum version 1.12)
  Go version:       go1.20.10
  Git commit:       a61e2b4
  Built:            Tue Nov 14 00:00:00 2023
  OS/Arch:          linux/arm64
  Experimental:     false
 containerd:
  Version:          1.7.2
  GitCommit:        0cae528dd6cb557f7201036e9f43420650207b58
 runc:
  Version:          1.1.7
  GitCommit:        f19387a6bec4944c770f7668ab51c4348d9c2f38
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0


frequent null value in column "item_id" violates not-null constraint exception

I am frequently seeing these exceptions for securitygroup, elb, and alb.
I have ran monkey db upgrade but still getting below exceptions. anyway, my setup is just 2-3 weeks old so it should not be required.
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/engine/base.py", line 1246, in _execute_context
    cursor, statement, parameters, context
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/engine/default.py", line 588, in do_execute
    cursor.execute(statement, parameters)
psycopg2.errors.NotNullViolation: null value in column "item_id" violates not-null constraint
DETAIL:  Failing row contains (138221, 10, Unknown Access, null, f, null, null, 2020-08-17 11:07:28.174966, null, null, null, null, null, null, null, f).

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/security_monkey/task_scheduler/tasks.py", line 343, in _audit_changes
    au.audit_objects()
  File "/usr/local/lib/python3.6/dist-packages/security_monkey/auditor.py", line 683, in audit_objects
    method(item)
  File "/usr/local/lib/python3.6/dist-packages/security_monkey/auditors/elb.py", line 184, in check_internet_scheme
    sub_issue_message=issue.issue, score=issue.score)
  File "/usr/local/lib/python3.6/dist-packages/security_monkey/auditor.py", line 1005, in link_to_support_item_issues
    issue = self.add_issue(link_score, issue_message, item)
  File "/usr/local/lib/python3.6/dist-packages/security_monkey/auditor.py", line 631, in add_issue
    self.override_scores = query.all()
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/orm/query.py", line 3233, in all
    return list(self)
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/orm/query.py", line 3388, in __iter__
    self.session._autoflush()
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/orm/session.py", line 1597, in _autoflush
    util.raise_from_cause(e)
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/util/compat.py", line 398, in raise_from_cause
    reraise(type(exception), exception, tb=exc_tb, cause=cause)
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/util/compat.py", line 153, in reraise
    raise value
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/orm/session.py", line 1586, in _autoflush
    self.flush()
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/orm/session.py", line 2479, in flush
    self._flush(objects)
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/orm/session.py", line 2617, in _flush
    transaction.rollback(_capture_exception=True)
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/util/langhelpers.py", line 68, in __exit__
    compat.reraise(exc_type, exc_value, exc_tb)
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/util/compat.py", line 153, in reraise
    raise value
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/orm/session.py", line 2577, in _flush
    flush_context.execute()
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/orm/unitofwork.py", line 422, in execute
    rec.execute(self)
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/orm/unitofwork.py", line 589, in execute
    uow,
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/orm/persistence.py", line 245, in save_obj
    insert,
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/orm/persistence.py", line 1137, in _emit_insert_statements
    statement, params
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/engine/base.py", line 982, in execute
    return meth(self, multiparams, params)
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/sql/elements.py", line 293, in _execute_on_connection
    return connection._execute_clauseelement(self, multiparams, params)
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/engine/base.py", line 1101, in _execute_clauseelement
    distilled_params,
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/engine/base.py", line 1250, in _execute_context
    e, statement, parameters, cursor, context
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/engine/base.py", line 1476, in _handle_dbapi_exception
    util.raise_from_cause(sqlalchemy_exception, exc_info)
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/util/compat.py", line 398, in raise_from_cause
    reraise(type(exception), exception, tb=exc_tb, cause=cause)
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/util/compat.py", line 152, in reraise
    raise value.with_traceback(tb)
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/engine/base.py", line 1246, in _execute_context
    cursor, statement, parameters, context
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/engine/default.py", line 588, in do_execute
    cursor.execute(statement, parameters)
sqlalchemy.exc.IntegrityError: (raised as a result of Query-invoked autoflush; consider using a session.no_autoflush block if this flush is occurring prematurely)
(psycopg2.errors.NotNullViolation) null value in column "item_id" violates not-null constraint
DETAIL:  Failing row contains (138221, 10, Unknown Access, null, f, null, null, 2020-08-17 11:07:28.174966, null, null, null, null, null, null, null, f).

[SQL: INSERT INTO itemaudit (score, issue, notes, action_instructions, background_info, origin, origin_summary, class_uuid, fixed, justified, justified_user_id, justification, justified_date, item_id, auditor_setting_id) VALUES (%(score)s, %(issue)s, %(notes)s, %(action_instructions)s, %(background_info)s, %(origin)s, %(origin_summary)s, %(class_uuid)s, %(fixed)s, %(justified)s, %(justified_user_id)s, %(justification)s, %(justified_date)s, %(item_id)s, %(auditor_setting_id)s) RETURNING itemaudit.id]
[parameters: {'score': 10, 'issue': 'Unknown Access', 'notes': None, 'action_instructions': None, 'background_info': None, 'origin': None, 'origin_summary': None, 'class_uuid': None, 'fixed': False, 'justified': False, 'justified_user_id': None, 'justification': None, 'justified_date': datetime.datetime(2020, 8, 17, 11, 7, 28, 174966), 'item_id': None, 'auditor_setting_id': None}]
(Background on this error at: http://sqlalche.me/e/gkpj)
ERROR:security_monkey:[X] Database error processing accounts Dev Account, cleaning up session.
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/engine/base.py", line 1246, in _execute_context
    cursor, statement, parameters, context
  File "/usr/local/lib/python3.6/dist-packages/sqlalchemy/engine/default.py", line 588, in do_execute
    cursor.execute(statement, parameters)
psycopg2.errors.NotNullViolation: null value in column "item_id" violates not-null constraint
DETAIL:  Failing row contains (138221, 10, Unknown Access, null, f, null, null, 2020-08-17 11:07:28.174966, null, null, null, null, null, null, null, f).

Are you using the latest version from the develop branch?

yes

I am using latest commit from develop branch.

Unfortunately, this one is very difficult to debug. You will need to run this in PyCharm and add a breakpoint to see what the state of the data is and why that's happening.


error: 1-2435-8325

I did everything was suppose but that erro shows on the start of my DSI Xl.
I used the Version 3.6.7
any suggestions?

Same issue, used latest version.

SAME


crashing when inject

when i inject the cheat or dll its crashing is there anything i have to change in the source
becuse i didn't miss with the files yeet

Yes change all client_panorama.dll to client.dll

@jon4dev yah it works thinks


the changelog for 6.8.0 should mention the end of support for node-sass

https://github.com/foundation/foundation-sites/commit/3a4eecb168273a509dfe9989977790edd25dbdb2 introduces usages of @use in the Sass files, which is not supported by node-sass (because it is not implemented by libsass but only by dart-sass).
While this move is fine (dart-sass is the modern implementation of Sass), I think it would deserve being highlighted explicitly in the readme.

Yep, and SASS versions support should be clearly stated in project description.
Foundation Sites 6.8.0 and Motion UI 2.0.4 don't support node-sass.
For example, "Zurb template" will fail to compile, because it requires the latest version of foundation-sites, but installs node-sass as well.
I tried to update foundation-sites and motion-ui in my boilerplate and didn't expect such breaking change in minor version.

@alexeych have you found a work around? I still regularly use the foundation-zurb-template, but now can't work out how to get it to install without loads issues (which I'm guessing are due to node-sass?).

@Currey either use sass instead of node-sass or downgrade foundation-sites to 6.7. That's the workaround.

foundation-zurb-template should really get an updated package.json. :)
I'm just keep it pinned < 6.8 for simplicity.
"foundation-sites": "~6.7.5",

@erutan updating the template is indeed a good move as tracked in #12588.
However, to resolve the current issue, I'm expected a maintainer to update the changelog for 6.8.0

Yeah, that's why I made a different issue. :)


Fix for #154

Q
A

Bug fix?
[x]

New feature?
[ ]

New sample?
[ ]

Related issues?
fixes #154

What's in this Pull Request?
fix for #154

Thanks @AJIXuMuK!


Bad screenshot type yaml configuration

Version(s) affected
1.1.4
Description
Screenshot type config parameter doesn't works.
How to reproduce
pwa:
    manifest:
        enabled: true
        screenshots:
            -
                src: 'screenshots/screenshot-desktop-1.png'
                platform: 'macos'
                label: 'Landing page'
                form_factor: 'wide'
                type: 'image/png'

Possible Solution
It seems that for now only works if you write format instead of type. Fix the related documentation here

Thanks.
Documentation is also aligned with this.


prevent hang on 2.9.3

seems that the benchmark is always hanging on 2.9.3

does not solve the issue


Welcome message

A message that the server sends every X time to the available players.
The message and the time can be configured by the admin in the game. We should make the message system to keep track of individual player and send the message only when the time has come.

Started some work on this i have auto-broadcasts going via an external file "broadcasts.ice"  on a timer


App crashes if i try to play song on notification bar

Describe the bug
when playing a music and i tring to play or next song in notification bar it shows me stooped.
To Reproduce
Steps to reproduce the behavior:

Download the app and open it
then play a music
then click back button and exit from app
then try to listen next song click on next button from notification bar

Expected behavior
I expected that the app would not crash when I click on next button from notification bar
Device:

OS: Android 8.1.0
appVersion <IP_ADDRESS>

Recording Of The Bug:
https://youtu.be/1T3yl4Qi6uk

This is an IPC issue that will be fixed in the near future, thank you for your report.

I think he has been fixed and will be posted to the release page later.


mutli-select in games list to batch delete/hide

just make selectedGame for whatever into an array and for everything use [0] unless there's mutliple items, then show new screen


Unknown formatter vat

Doesn't find vatId() formatter!
$faker = new \Faker\Generator();
$faker->addProvider(new \Faker\Provider\en_US\Person($faker));
$faker->addProvider(new \Faker\Provider\en_US\Address($faker));
$faker->addProvider(new \Faker\Provider\en_US\PhoneNumber($faker));
$faker->addProvider(new \Faker\Provider\Payment($faker));
$faker->addProvider(new \Faker\Provider\Lorem($faker));
$faker->addProvider(new \Faker\Provider\Internet($faker));
$faker->addProvider(new \Faker\Provider\Company($faker));
$faker->addProvider(new \Faker\Provider\it_IT\Company($faker));
    $type="CARRIER";

    for ($i = 0; $i < 2; $i++) {
        $company = new Company();
        $company->setName($faker->company);
        $company->setEmail($faker->email);
        $company->setWebsite($faker->url);
        $company->setVat($faker->vatId());

        $company->setCreatedAt(new \DateTime());
        $company->setUpdatedAt(new \DateTime());
        $company->setType($type);
        $manager->persist($company);

    }

[InvalidArgumentException]
Unknown formatter "vat"

same issue here... any solution?

I have resolved, because there was an older version of faker in a dependency package!

Hello Thank you christian.
I updated package via composer but the problem persist.
Any idea?

You could see if you have a package in composer.json that has fzaninotto/Faker as dependency, if composer download the package with faker dependency, It will ignore your fzaninotto/Faker in composer.json
I have "fzaninotto/faker":"dev-master" in composer json and I have found the package that has an older version of fzaninotto/faker

it's fixed, thank you for your reactivity;

Hmm I just hit this problem:
Psy Shell v0.9.4 (PHP 7.1.14 — cli) by Justin Hileman
>>> (Faker\Factory::create())->vat
InvalidArgumentException with message 'Unknown formatter "vat"'
>>>

Any hints?

@Kyslik It could be because of locale that was initially. For example, for pt_PT vat function is $faker->taxpayerIdentificationNumber;

Hi there,
I took the functions that I need from Faker\Provider\ar_SA\Person and put in the Faker\Provider\en_US\Person, fixed!


feature request: Comments as list

Similar to clicking on the notification button to get a list of comments, it would be nice to be able to navigate comments in a game.
Comment navigation in game is very restricted at the moment: clicking arrow buttons multiple times

GoKibitz isn't really possible to develop right now (just getting the dependencies up-to-date enough to release new versions would be a considerable task), but there actually is a kinda secret way to navigate between comments. At any place on GoKibitz, you can hold down ? to see a help menu. And that help menu shows that when viewing a game record, you can use alt + left or right arrow to navigate between moves that have comments on them.
Maybe that'll help a little bit!


Add ETSI Website

I am not sure this is relevant enough for this repo. I checked out the transportation standards and there are non for vehicles (on the road). Thank you for the suggestion, though, and sorry for the delay.


MIPS: Don't apply default base address for ELF

Binary is: crack.zip
Ghidra elf loader uses default image base 0x100000, and when I'm loading this binary and decompiling function checkPass() it cannot find strings in .rodata section, because their references are not relative to ImageBase and hardcoded.

You can change the image base on import to a correct image base for your binary, unless the image base is specified in the binary.  A default image base for binaries that do not specify an image base is set in order to keep the image away from 0x0, which can cause erroneous references to be created in certain cases.
If you change the image base on the import of the crack to line up with .rodata, does that help?
Is the image base somehow specified in the binary and it is being incorrectly interpreted?

ELF binary contains virtual addresses in headers, so base addresses for every section are already specified, and it's don't require to apply other image base.
And, yes, changing imagebase at importing binary stage helps.

Another crackle that is cracked up... try to rebuild the ELF file first before filing any bug. How to do this and why you get issues is not in the scope of ghidra. Personally I'd suggest elfsh/libbfd.

Actually as I've stated before I am so perverse I am considering adding Java-bindings for libbfd to be able to import all kinds of files, that's why I added libbfd-support to 'snowman'-decompiler (and yes I did add MIPS-support to snowman as well).


Peer dependency conflic

I'm using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>packages and when I try to install their i catch npm error about conflict peer dependencies. Its cause is strong version printed in react' package.json:
  "peerDependencies": {     "effector-react": "^22.3.0",     "@farfetched/core": "0.4.0"   },
Could you do sth with this error?

Publishing of react binding 0.4.1 was failed, I'll fix it in a moment 🤗

Thank u


Retina / high resolution display support

Has anyone looked into adding support to render at native resolutions on retina / hi res displays?
Usually it amounts to multiplying the canvas.width and canvas.height by window.devicePixelRatio and scaling any rendered shape dimensions bywindow.devicePixelRatio.

Hello, @kbirk 👋 .
There is some initial work done in #260. The initial changes are far from perfect as they only cover the tile size problem.


Won't build under TKL Core

I installed Turnkey Linux Core 13.0 / Debian 7.2 Wheezy on VirtualBox under Windows 7, 64-bit.
When I issued the command go build main.go, these are the errors:
./main.go:60: undefined: newCommand
./main.go:85: undefined: command
./main.go:105: undefined: command
./main.go:106: undefined: newSshClientConfig
./main.go:118: undefined: newNameWriter
./main.go:119: undefined: newNameWriter
However, it is building fine under Windows 7 using go1.3.3 windows/386
I am using go1.3.3 linux/amd64 on my VirtualBox VM.

@ibm2100 you should use go build instead fo go build main.go the code is divided in 3 files.
Most of you undefined stuff are defined in ssh.go

Thanks for looking into this @vieux


#2545 migrated TestBooleanCounterProcessor parameterized test to junit5

Purpose
Closes #2545
PR introduces (a) breaking change(s): no
PR introduces (a) deprecation(s): no

Hi @tenthe, has there been any progress with #2374? I can try working on it and then come back to this. What's the reason your PR wasn't merged?

Hi @IsaakKrut,
that would be awsome. I reopened the PR.
The PR was initially only intended as a basis for discussion and has not yet been finalized.
Perhaps we can discuss in #2375 how to proceed. I'll merge this PR.


Upgrade from 3.4 to 3.5 causes

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
Currently I'm seeing the following warnings in the terminal when running yarn start:
[WARNING] {"moduleIdentifier":"/Users/senders/Projects/top-docs/node_modules/@docusaurus/core/node_modules/babel-loader/lib/index.js??ruleSet[1].rules[5].use[0]!/Users/senders/Projects/top-docs/src/theme/DocVersionBanner/index.js","moduleName":"./src/theme/DocVersionBanner/index.js","loc":"8:765-779","message":"export 'useDocsVersion' (imported as 'useDocsVersion') was not found in '@docusaurus/theme-common/internal' (possible exports: AnnouncementBarProvider, BlogAuthorsListViewAllLabel, Collapsible, ColorModeProvider, DEFAULT_SEARCH_TAG, DraftBannerMessage, DraftBannerTitle, ErrorBoundaryError, ErrorBoundaryErrorMessageFallback, ErrorBoundaryTryAgainButton, ErrorCauseBoundary, HtmlClassNameProvider, NavbarProvider, NavbarSecondaryMenuFiller, PageMetadata, PluginHtmlClassNameProvider, ReactContextError, ScrollControllerProvider, SkipToContentFallbackId, SkipToContentLink, ThemeClassNames, ThemedComponent, UnlistedBannerMessage, UnlistedBannerTitle, UnlistedMetadata, composeProviders, containsLineNumbers, createStorageSlot, duplicates, filterDocCardListItems, getPrismCssVariables, groupBy, isMultiColumnFooterLinks, isRegexpStringMatch, isSamePath, keyboardFocusedClassName, listStorageKeys, listTagsByLetters, parseCodeBlockTitle, parseLanguage, parseLines, prefersReducedMotion, processAdmonitionProps, sanitizeTabsChildren, splitNavbarItems, translateBlogAuthorsListPageTitle, translateTagsPageTitle, uniq, useAlternatePageUtils, useAnnouncementBar, useBackToTopButton, useBlogAuthorPageTitle, useBlogTagsPostsPageTitle, useClearQueryString, useCodeWordWrap, useCollapsible, useColorMode, useContextualSearchFilters, useCurrentSidebarCategory, useDateTimeFormat, useDocsPreferredVersion, useEvent, useFilteredAndTreeifiedTOC, useHideableNavbar, useHistoryPopHandler, useHistorySelector, useHomePageRoute, useKeyboardNavigation, useLocalPathname, useLocationChange, useLockBodyScroll, useNavbarMobileSidebar, useNavbarSecondaryMenu, usePluralForm, usePrevious, usePrismTheme, useQueryString, useQueryStringList, useQueryStringValue, useScrollController, useScrollPosition, useScrollPositionBlocker, useSearchLinkCreator, useSearchQueryString, useSmoothScrollTo, useStorageSlot, useTOCHighlight, useTabs, useThemeConfig, useTitleFormatter, useTreeifiedTOC, useWindowSize)"}
[WARNING] {"moduleIdentifier":"/Users/senders/Projects/top-docs/node_modules/@docusaurus/core/node_modules/babel-loader/lib/index.js??ruleSet[1].rules[5].use[0]!/Users/senders/Projects/top-docs/src/theme/helper/JFrog.js","moduleName":"./src/theme/helper/JFrog.js","loc":"11:128-142","message":"export 'useDocsVersion' (imported as 'useDocsVersion') was not found in '@docusaurus/theme-common/internal' (possible exports: AnnouncementBarProvider, BlogAuthorsListViewAllLabel, Collapsible, ColorModeProvider, DEFAULT_SEARCH_TAG, DraftBannerMessage, DraftBannerTitle, ErrorBoundaryError, ErrorBoundaryErrorMessageFallback, ErrorBoundaryTryAgainButton, ErrorCauseBoundary, HtmlClassNameProvider, NavbarProvider, NavbarSecondaryMenuFiller, PageMetadata, PluginHtmlClassNameProvider, ReactContextError, ScrollControllerProvider, SkipToContentFallbackId, SkipToContentLink, ThemeClassNames, ThemedComponent, UnlistedBannerMessage, UnlistedBannerTitle, UnlistedMetadata, composeProviders, containsLineNumbers, createStorageSlot, duplicates, filterDocCardListItems, getPrismCssVariables, groupBy, isMultiColumnFooterLinks, isRegexpStringMatch, isSamePath, keyboardFocusedClassName, listStorageKeys, listTagsByLetters, parseCodeBlockTitle, parseLanguage, parseLines, prefersReducedMotion, processAdmonitionProps, sanitizeTabsChildren, splitNavbarItems, translateBlogAuthorsListPageTitle, translateTagsPageTitle, uniq, useAlternatePageUtils, useAnnouncementBar, useBackToTopButton, useBlogAuthorPageTitle, useBlogTagsPostsPageTitle, useClearQueryString, useCodeWordWrap, useCollapsible, useColorMode, useContextualSearchFilters, useCurrentSidebarCategory, useDateTimeFormat, useDocsPreferredVersion, useEvent, useFilteredAndTreeifiedTOC, useHideableNavbar, useHistoryPopHandler, useHistorySelector, useHomePageRoute, useKeyboardNavigation, useLocalPathname, useLocationChange, useLockBodyScroll, useNavbarMobileSidebar, useNavbarSecondaryMenu, usePluralForm, usePrevious, usePrismTheme, useQueryString, useQueryStringList, useQueryStringValue, useScrollController, useScrollPosition, useScrollPositionBlocker, useSearchLinkCreator, useSearchQueryString, useSmoothScrollTo, useStorageSlot, useTOCHighlight, useTabs, useThemeConfig, useTitleFormatter, useTreeifiedTOC, useWindowSize)"}
client (webpack 5.89.0) compiled with 2 warnings
When running
Uncaught runtime errors:
ERROR
(0 , docusaurus_theme_common_internal__WEBPACK_IMPORTED_MODULE_9_.useDocsVersion) is not a function
TypeError: (0 , docusaurus_theme_common_internal__WEBPACK_IMPORTED_MODULE_9_.useDocsVersion) is not a function
at DocVersionBanner (webpack-internal:///./src/theme/DocVersionBanner/index.js:23:1180)
at renderWithHooks (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:16305:18)
at mountIndeterminateComponent (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:20069:13)
at beginWork (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:21582:16)
at HTMLUnknownElement.callCallback (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:4164:14)
at Object.invokeGuardedCallbackDev (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:4213:16)
at invokeGuardedCallback (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:4277:31)
at beginWork$1 (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:27446:7)
at performUnitOfWork (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:26555:12)
at workLoopSync (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:26461:5)
It looks like it's complaining about the use of useDocsVersion:
 import {
   useDocsPreferredVersion,
   useDocsVersion,
 } from '@docusaurus/theme-common/internal';

The useDocsVersion method is no longer in the node_modules dependency for @docusaurus/theme-common
Reproducible demo
No response
Steps to reproduce
When upgrading to Docusaurus to 3.5, (0 , docusaurus_theme_common_internal__WEBPACK_IMPORTED_MODULE_9_.useDocsVersion) is not a function is shown on underlying pages.
Expected behavior
Upgrade would occur to 3.5
Actual behavior
localhost-1730318175773.log
Your environment
No response
Self-service

[ ] I'd be willing to fix this bug myself.

No repro provided so we can't troubleshoot this.
Most likely it's not a bug. You are using internal Apis that we moved and even mentioned in v3.5 release notes. If you are using internals, you have to understand the tradeoff you make.

Thank you @slorber , where is it mentioned the internal APIs were moved? I do not see it in the change log. Thanks for your help.

https://docusaurus.io/blog/releases/3.5#other-changes

We don't document this as a breaking change because it's considered an internal change. If you swizzle internal theme components, you need to understand what you opt into: it's now your responsibility to sync with our internal changes on upgrades (even minor).

Thank you! We will investigate.


[mgmt] cosmosdb preview release

https://github.com/Azure/sdk-release-request/issues/5734

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/arm-cosmosdb


Manual order fails in Kraken

System information

Ubuntu 18.04:

Describe the problem
Manual order always fails in Kraken, with a timeout
Source code / Error logs
Trades error:
{ RequestError: Request timed out
at timeoutFn (/home/trader/zenbot-kraken/node_modules/kraken-api/node_modules/got/index.js:131:53)
at asPromise (/home/trader/zenbot-kraken/node_modules/kraken-api/node_modules/got/index.js:134:9)
at got (/home/trader/zenbot-kraken/node_modules/kraken-api/node_modules/got/index.js:386:10)
at rawRequest (/home/trader/zenbot-kraken/node_modules/kraken-api/kraken.js:42:25)
at KrakenClient.publicMethod (/home/trader/zenbot-kraken/node_modules/kraken-api/kraken.js:121:20)
at KrakenClient.api (/home/trader/zenbot-kraken/node_modules/kraken-api/kraken.js:93:16)
at Object.getTrades (/home/trader/zenbot-kraken/extensions/exchanges/kraken/exchange.js:112:14)
at Timeout.forwardScan [as _onTimeout] (/home/trader/zenbot-kraken/commands/trade.js:628:20)
at ontimeout (timers.js:425:11)
at tryOnTimeout (timers.js:289:5)
name: 'RequestError',
code: 'ETIMEDOUT',
host: 'api.kraken.com',
hostname: 'api.kraken.com',
method: 'POST',
path: '/0/public/Trades',
protocol: 'https:',
url: 'https://api.kraken.com/0/public/Trades' }

Your error was to get the info of charts, this occurred when kraken servers
are busy, obviously, manual and auto trades fails because the servers
cannot get it.
 I suggest to you that change to a more serious and professional
exchange... Like GDAX.
El ds., 22 set. 2018, 23:22, Mario Dantas<EMAIL_ADDRESS>va
escriure:

System information

Ubuntu 18.04:

Describe the problem
Manual order always fails in Kraken, with a timeout
Source code / Error logs
Trades error:
{ RequestError: Request timed out
at timeoutFn
(/home/trader/zenbot-kraken/node_modules/kraken-api/node_modules/got/index.js:131:53)
at asPromise
(/home/trader/zenbot-kraken/node_modules/kraken-api/node_modules/got/index.js:134:9)
at got
(/home/trader/zenbot-kraken/node_modules/kraken-api/node_modules/got/index.js:386:10)
at rawRequest
(/home/trader/zenbot-kraken/node_modules/kraken-api/kraken.js:42:25)
at KrakenClient.publicMethod
(/home/trader/zenbot-kraken/node_modules/kraken-api/kraken.js:121:20)
at KrakenClient.api
(/home/trader/zenbot-kraken/node_modules/kraken-api/kraken.js:93:16)
at Object.getTrades
(/home/trader/zenbot-kraken/extensions/exchanges/kraken/exchange.js:112:14)
at Timeout.forwardScan [as _onTimeout]
(/home/trader/zenbot-kraken/commands/trade.js:628:20)
at ontimeout (timers.js:425:11)
at tryOnTimeout (timers.js:289:5)
name: 'RequestError',
code: 'ETIMEDOUT',
host: 'api.kraken.com',
hostname: 'api.kraken.com',
method: 'POST',
path: '/0/public/Trades',
protocol: 'https:',
url: 'https://api.kraken.com/0/public/Trades' }
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DeviaVir/zenbot/issues/1716, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGNJLFZp0iK3RsF0RGDxe3XNinwv5jjzks5udqocgaJpZM4W1ctE
.

--
Diego Mérida
<EMAIL_ADDRESS>637236008

You're right @diegstroyer I'll change to GDAX, even the API is faster on GDAX than KRAKEN


Customize arrow keys in PowerToys Run mode

I hope you can customize the next arrow key in PowerToys Run. When there are multiple search results, if you want to find the result you want, you need to move your hand to the arrow key area. Can you add custom arrow keys in this mode? When there are multiple results, please press Ctrl + j, k to select up and down, Reduce the range of hand movement.

tracked in #1828

tracked in #1828


Fix opportunity import to set partner_string for search index

Close #108

I don't have the VOA data on my machine to test this, but I'm sure everything works fine. If not, we'll create another issue! I'll merge this.


Issue using the zoau e2a and a2e commands with libiconv (using altbin)

Running e2a  and a2e  using the default from and to parameters...
looks like files are being tagged without conversion.
[SYSA] bash-5.2$ which iconv
/hewitt/zopentools/zopen_repo/usr/local/altbin/iconv
[SYSA] bash-5.2$ iconv --version
iconv (GNU libiconv 1.17)
[SYSA] bash-5.2$ which e2a
/usr/lpp/IBM/zoau/v1r3/bin/e2a
[SYSA] bash-5.2$ which a2e
/usr/lpp/IBM/zoau/v1r3/bin/a2e

I tried this one:
$ ls -lT bla
t ISO8859-1   T=on  -rw-rw-r--   1 ITODORO  CDEV           5 Jul 17 21:53 bla
$ cat bla
IGOR
$ a2e -f  ISO8859-1 -t IBM-1047 bla
 $ ls -lT bla
t IBM-1047    T=on  -rw-rw-r--   1 ITODORO  CDEV           5 Jul 17 21:53 bla
$ cat bla # seems to be readable
IGOR
$ type iconv
iconv is hashed (/home/itodoro/zopen/usr/local/altbin/iconv)

@IgorTodorovskiIBM
Case number TS016765505 has been opened.

Additional doc on the issue. It would be nice if e2a would maintain the file perms.

Case closed - TS016765505 (/bin/iconv will be used along with maintaining file perms for the converted file)
Will be fixed with zoau service 1.3.3 in 3rd quarter.
Anthony,
Thanks for making it happen.

@IgorTodorovskiIBM  Is libiconv required by any zopen tools or can /bin/iconv be used by them?
thanks

@IgorTodorovskiIBM Is libiconv required by any zopen tools or can /bin/iconv be used by them? thanks

No dependencies on the zot iconv currently

Does this fail for you?
LIBPATH=/dsfs sftp zoscan2b

Yes
[SYSA] bash-5.2$ LIBPATH=/dsfs sftp mvs-ste3
CEE3512S An HFS load of module EUVFKDLL failed. The system return code was<PHONE_NUMBER>; the reason code was ED076232.
The traceback information could not be determined.
<> LEAID ENTERED (LEVEL 02/26/2016 AT 17.55)
<> LEAID LEAID112 LEAID ACKNOWLEDGES UNSUPPORTED USS ENVIRONMENT
<> LEAID LEAID112 ABEND-AID PROCESSING SUPPRESSED
<> LEAID PROCESSING COMPLETE. RC=4
FOTS0841 Connection closed

I realized I posted this on the wrong issue (meant to post it in https://github.com/ZOSOpenTools/coreutilsport/issues/77#issuecomment-2259123887) :)
The problem appears to be on the dsfs end - the z/OS team will look into it.


chore(remove jquery)

We've already been good about avoiding jQuery in the addon, this just explicitly removes it from tests/dev builds to avoid regressions.

👏


Tween Position When New Follow Target Is Set

Project Type
2D (Might apply to 3D too)
Feature Description
When set_follow_target(value: Node2D) is called, I want the phantom camera to move to the new target, rather than instantly set its position to it.
Use Cases
In my game I have four characters. Only one of them is controllable, so the character follows him. However, I can change who to control at any time. Upon doing so, the phantom camera would pan towards the next player, rather than immediately jumping towards it.
By right, the work-around for this issue is to set up a phantom camera for each character. However, since the cameras would have the same custom properties (such as zoom) plus their roles are similar, I thought I could reduce node count.
(Optional) Proposed Solution
On set_follow_target(value: Node2D), before setting "follow_target = value", set up a tween to transition the position from the old follow target to the new.
If there's no objections, I'll make a new PR for this.

I would prefer not to change the follow target behaviour to trigger a tween when setting a follow_target, as it will start getting a bit murky with how a tween works behind the scenes.
In your case, if you want to reuse the properties of a PCam, then you can save the PCam in question as its own scene and add instances of that to the main scene. Where you then apply the general properties, like zoom, in that separate PCam scene, which will update each instance in the main scene — kinda like a prefab or component.
To get a tweening behaviour working like you described, and to save nodes in the scene, I would use two PCams rather than one for each character. The first being what you would use by default that follows the player. The second one effectively being on stand-by and only gets attached to a given target once needed and then given the highest priority. The initial PCam then becomes the one on stand-by, and you can then just repeat that process and juggle between the two like that. Which should give you the desired tween effect.
Your proposal could be an additional target setter function if anything, but personally, I want to try and only add features that add new functionality to keep the codebase as light as possible — the exception is when it greatly simplified an otherwise very complicated workaround such as set_cull_mask_value(). Think the added codebase complexity cost for this is higher than the value of the problem it would solve.

I guessed as much that there's supposed to be one phantom camera per character. I've gone with your counter-suggestion instead.


Plugin Idea: Word Censoring

A plugin that censors words with *'s when players say specified words (eg. racist words).

Pull requests welcome for functionality, although I am not a proponent of censorship; so you'll have to get one of the other maintainers to apply a patch for such a thing. Closing #68.


Standardize license headers

This PR contains 3 commits:

Add build tooling to automatically generate missing license headers
Regenerate all the license headers for the project
Manual one-time cleanup that wasn't able to be automated

I'm not sure I want to redo this for 1.9, since it's a bit tedious, but it may help with getting things consistent. I'm open to feedback on that.
The main goal of this change is to standardize our style for license headers, and to automate adding missing ones during development.

If this works well its going to be awesome.  Does this new plugin run before the rat plugin?  Is the rat plugin still needed?

Can you make the license template match our current license headers (at least the majority)?  Then you could avoid re-generating one for every file.  I have never not copied and pasted one so I imagine they can't be all that different.

If this works well its going to be awesome. Does this new plugin run before the rat plugin? Is the rat plugin still needed?

Currently, it runs after the rat plugin. This plugin runs at process-resources, while rat runs at validate (see https://maven.apache.org/guides/introduction/introduction-to-the-lifecycle.html for reference on lifecycle phases).
We can re-order if necessary, but I'm not sure it matters much.

Can you make the license template match our current license headers (at least the majority)? Then you could avoid re-generating one for every file. I have never not copied and pasted one so I imagine they can't be all that different.

I thought about that, but we had them too inconsistent, so the change was going to be large either way. I decided to converge on the format that is presented on ASF's official documentation instead.

Currently, it runs after the rat plugin.

It seems like it would be nice if it ran before the rat plugin, that way it could fix up files before rat fails the build.

Currently, it runs after the rat plugin.

It seems like it would be nice if it ran before the rat plugin, that way it could fix up files before rat fails the build.

That makes sense. I'll do that.

Would it make sense to disable this new plugin for travis builds?  That way rat can fail the travis build if files need to be fixed.

I'm not sure I want to redo this for 1.9, since it's a bit tedious, but it may help with getting things consistent. I'm open to feedback on that.

Is there any benefit to making the same changes in 1.9. I am assuming that we will only have merge conflicts when someone modifies the license portion in 1.9.  I think that type of modification in 1.9 would be very rare.  Also if there is conflict, could you just delete the license portion and just let the plugin put it back?

I'm not sure I want to redo this for 1.9, since it's a bit tedious, but it may help with getting things consistent. I'm open to feedback on that.

Is there any benefit to making the same changes in 1.9.

The main benefit is just having the automation to add the new licenses. I'm not worried about conflicts, because it'd probably only happen, as you say, when the header itself was changed, which shouldn't really happen. For that alone, I'm disinclined to pull this back into 1.9. However, the secondary benefit is that it helps keep our build/release tooling sync'd between branches, which makes it easier to ensure consistent release quality across our actively maintained versions.

which makes it easier to ensure consistent release quality across our actively maintained versions.

For this particular case having the rat plugin in 1.9 achieves the quality goal.

Would it make sense to disable this new plugin for travis builds? That way rat can fail the travis build if files need to be fixed.

Since we currently don't have any other formatting validation checks during CI, I wouldn't want to add it in this PR. However, that could be done as a follow-on to create a CI profile that we use for CI that skips all formatting, and does only formatting validation for all our automated formatting.

which makes it easier to ensure consistent release quality across our actively maintained versions.

For this particular case having the rat plugin in 1.9 achieves the quality goal.

Aside from the manual fixes that I did to deal with double-license header issues in the bloomfilter code, I agree. I can backport those fixes manually on their own, though, without all the other changes.


Tabs layout is expensive

Resizing the explorer makes the UI laggy, it looks like the main culprit is tabs:

/duplicate https://github.com/microsoft/vscode/issues/34745

Upon second look, _rerenderTabs is coming from terminal:
https://github.com/microsoft/vscode/blob/4e12faaecebb7984bef524076f5fa2a3850823fb/src/vs/workbench/contrib/terminal/browser/terminalTabbedView.ts#L285

🤯 sorry about the hassle.

@meganrogge FYI I pushed a change to no longer trigger _rerenderTabs on layout as this is an expensive action. I don't think we should need to do this on layout but keep an eye out for regressions.


Improved UI of the website

Description
Improved UI of the website
Issue Resolved
Fixes #3
Screenshots

Checklist
Please make sure to review the following before submitting your PR:

[x] I have read the contribution guidelines.
[x] I have read the code of conduct.
[x] I have reviewed my submission in detail.

@Manobal-Singh-Bagady The footer is ready can you please merge this pr so that I can create a new one for the footer issue

@Manobal-Singh-Bagady The footer is ready can you please merge this pr so that I can create a new one for the footer issue

Alright


Update storybook links in Figma components

Description
We have recently updated our storybook links https://github.com/MetaMask/metamask-extension/pull/17092 they need to be updated in our external sources
Technical Details

Update all links in the Components table in Storybook

Acceptance Criteria

All storybook links in the components documentation links

References

DS Components in Figma https://www.figma.com/file/HKpPKij9V3TpsyMV1TpV7C/DS-Components?node-id=16%3A6&t=cUkhqjDkJwI0lbPy-1

@Akatori-Design this is ready for review https://www.figma.com/file/HKpPKij9V3TpsyMV1TpV7C/branch/ZsrfBd6AUkC8I9KyHUSdod/DS-Components?node-id=11%3A15&t=23Htx4RnkVQHAjU0-1


Tcp connection not connecting

Hi,
I've been developing Ethernet drivers for qemu_cortex_m3 (stellaris lm3s6965).
For testing the driver with qemu, a virtual network was made with vde_switch. One qemu-arm runs zephyr with Ethernet driver and another qemu-x86_64 with linux.
We could successfully test using ping commands and UDP. In UDP zephyr acts as the client and linux machine as server.
For a TCP test, where zephyr acts as the client and linux machine as server, there is no response coming from the server.
tcpdump showed as only SYN packets. No other response was coming from the server.
When tested two different qemu-linux machines for TCP, the system is responding.
Would like to know anything on above, or previous threads on it.
regards

Using the latest zephyr v1.12

Can you make sure that your linux host does not block the packets? In my setup, I had similar issues as my linux firewall was blocking incoming connections from zephyr.

I shall, check on that!

Hi., Got the issue solved.
Had a bug in driver and few config issues
Thanx


Create a doc describing how to find current issues/fixes

With moving to GitHub, developers have access to the work items for .NET Core, CLR, and many others. However, the issues are spread across many repositories, which makes it difficult for users to easily search for an issue. Often an answer can be found fairly quickly if you're in the right repo, but finding that repo can be a challenge.
I think a document that lists the main technologies (including descriptions of what they include) and a link to the repo would be a great starting point for people who are trying to determine if an issue is known and a possible work around, or if it should be filed.

Are you talking about something like the list in this note?

Yes - something like that, but visible from the docs.microsoft.com/dotnet

Maybe we can tweak this .NET Core introduction topic?
https://docs.microsoft.com/en-us/dotnet/articles/core/index#composition
It already talks about the parts it just not clearly lists them as repos with their names. It's a bit subtle.

Does this belong on docs.microsoft.com, or on the .NET Foundation home page?
/cc @martinwoodward

I think a page on docs.microsoft.com encouraging people to go to GitHub to file issues is a very good thing. That shouldn't be the only way Microsoft's customers can file bugs or log issues though - a Microsoft customer shouldn't have to care what project an issue is in, they should be able to get support for the product they got from Microsoft (i.e. a .NET Core binary in a VS "15")
The full detail of the content to help find the project  probably belongs to the core project with a link to that page from docs.microsoft.com (that is if all the projects needed - reference are indeed open source and are in the .NET Foundation).

So the problem today is that the only feedback mechanism for docs.microsoft.com is comments, and so far the response rate on comments has been underwhelming. As a hypothetical casual consumer of our docs, asking me to learn the intricacies of GitHub just to file a bug or complaint is a non-starter.

So think I'm kinda agreeing with @stevehoag - there should be mechanism for end users of Microsoft products to easily file bugs / issues with products from Microsoft. (And the same is true for other vendors shipping .NET Core). What I don't want to see is "if you find an issue in VS, figure out what code is causing it and go track down the project that it came from and send in a PR to fix it and Microsoft will pull that change from upstream eventually". Microsoft still have to support their own product.
But equally, I would strongly encourage all vendors, especially Microsoft, to also encourage developers to investigate the code behind as I think @BillWagner is suggesting, file issues on GitHub when possible and generally get more involved in the open source community.

Reading all the comments, how about this for a proposal:
There is a single location on docs.microsoft.com that solicits bugs / issues with .NET products and docs. From there, readers can either:

File an issue, and MS folk will triage it and make an issue for the correct team.
Explore the list of Open Source repos that are part of the .NET ecosystem, and file an issue at the most likely project.

/cc @richlander
Can folks respond with a +1, or an alternative?

+1 with the caveat of:
1.5 Contact Support if this is a blocking issue or if you are having trouble figuring out what the problem is
1.6 For security bugs, don't forget the BugBounty program
Not sure if this is relevant or not at this level of docs but mentioning it now just in case.

+1, with @martinwoodward's additions

+1 to what Ron said...

ping @DuncanmaMSFT as this old thread has some interesting thoughts related to GitHub issues on docs.
Removing various docs labels & adding the external label.

This issue has been closed as part of the issue backlog grooming process outlined in #22351.
That automated process may have closed some issues that should be addressed. If you think this is one of them, reopen it with a comment explaining why. Tag the @dotnet/docs team for visibility.

This issue has been closed as part of the issue backlog grooming process outlined in #22351.
That automated process may have closed some issues that should be addressed. If you think this is one of them, reopen it with a comment explaining why. Tag the @dotnet/docs team for visibility.


Deprecate low level Diffie Hellman functions.

[x] documentation is added or updated
[x] tests are added or updated

This is extremely early.  Accepting the suite of deprecated functions is the first step.

The command line commit is a clone of the one in #10977 and will be removed once that is merged.
Documentation isn't updated.
The conversions in ssl3_lib.c have problems.
Tests are failing.

Bringing this out of WIP.  There are two outstanding functions: DH_new_by_nid and DH_check_params which should be deprecated but are used by TLS with no immediately obvious replacement.
Travis failures do not appear to be relevant.

Travis failures not relevant here

The corresponding thing for DH_check_params would be the key validation stuff, right?

DH_new_by_nid() is a constructor, just like DH_new(), even though very special case.  I suggest not to deprecate it now.

Yes, DH_check_params would be some kind of key validation.  I tried creating an EVP_PKEY_CTX and calling some of the check functions but none worked.
Happy to leave DH_new_by_nid non-deprecated.  I guess NIDs will work for a while yet.  There will need to be some refactoring to avoid NIDs internally.

There will need to be some refactoring to avoid NIDs internally.

Yes, agreed.  I guess that we can work on that when EC stuff has transferred over to providers

Yes, DH_check_params would be some kind of key validation.  I tried creating an EVP_PKEY_CTX and calling some of the check functions but none worked.

That's because there currently is no validation code added in the KEYMGMT implementations for DSA and DH.

Travis and Appveyor are a bit unhappy...

That's because there currently is no validation code added in the KEYMGMT implementations for DSA and DH.

As I remembered after getting stuck into it.

Travis isn't relevant, ping for review.

Rebased and conflicts merged -- no code changes just infrastructure.
I'll wait for the CIs before merging.

Merged to master.  Thanks for the feedback and assistance.


Liberator team should have access to some concourse pipelines but not all

The liberator team should have access to the concourse pipelines that deploy console.cloud.gov (once it exists) and other liberator services, but not to all cloud.gov pipelines. One proposed solution is to spin up a separate concourse instance that deploys liberator services.
@dlapiduz @jcscottiii @msecret

Bummer that Concourse doesn't offer finer-grained access control.

Are there any downsides to having our own concourse? Or is the main downside the time it would take to set that up?

@msecret aside from spinning up a new Concourse, there is the additional overhead of learning how to use Concourse. http://concourse.ci/flight-school.html

Just having the overhead of maintaining multiple, yeah. @s-matyukevich is this something the BOSH broker could be useful for?

We decided to punt on this as part of https://github.com/18F/cg-atlas/issues/41. Closing for now.


Jest fails to run tests on React Native 0.59.5

🐛 Bug Report
Jest fails to run on React Native 0.59.5
To Reproduce

update react native 0.59.3 -> 0.59.5
have jest config:

  "jest": {
    "preset": "react-native",
    "transform": {
      "^.+\\.(jsx?)$": "<rootDir>/node_modules/react-native/jest/preprocessor.js",
      "^.+\\.tsx?$": "ts-jest"
    },
    "transformIgnorePatterns": [
      "node_modules/(?!react-native)/"
    ],
    "globals": {
      "window": {},
      "ts-jest": {
        "tsConfig": "./tsconfig.jest.json"
      }
    }
  },

run jest ./node_modules/.bin/jest
get the following error

$ ./node_modules/.bin/jest
No tests found
In /Users/ethanneff/Dropbox/active/dev/react-native/example
  712 files checked.
  testMatch:  - 712 matches
  testPathIgnorePatterns: /node_modules/ - 190 matches
  testRegex: (/__tests__/.*|(\.|/)(test|spec))\.(jsx?|tsx?)$ - 2 matches
Pattern:  - 0 matches

Expected Behavior
Jest was working fine and running all tests in 0.59.3
Code Example
See steps to reproduce
Environment
$ react-native info
info
  React Native Environment Info:
    System:
      OS: macOS 10.14.4
      CPU: (8) x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
      Memory: 534.38 MB / 16.00 GB
      Shell: 3.2.57 - /bin/bash
    Binaries:
      Node: 10.15.3 - /usr/local/bin/node
      Yarn: 1.15.2 - /usr/local/bin/yarn
      npm: 6.7.0 - /usr/local/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.2, macOS 10.14, tvOS 12.2, watchOS 5.2
      Android SDK:
        API Levels: 22, 23, 24, 25, 26, 27, 28
        Build Tools: 23.0.1, 23.0.3, 25.0.0, 25.0.1, 25.0.2, 26.0.1, 26.0.2, 26.0.3, 27.0.3, 28.0.0, 28.0.1, 28.0.2, 28.0.3
        System Images: android-19 | Intel x86 Atom, android-22 | Google APIs Intel x86 Atom, android-26 | Google APIs Intel x86 Atom, android-27 | Google APIs Intel x86 Atom, android-27 | Google Play Intel x86 Atom, android-28 | Google APIs Intel x86 Atom
    IDEs:
      Android Studio: 3.3 AI-182.5<IP_ADDRESS>14842
      Xcode: 10.2/10E125 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.8.6 => 16.8.6
      react-native: 0.59.4 => 0.59.4
    npmGlobalPackages:
      react-native-cli: 2.0.1

solved with the following:
  "jest": {
    "preset": "react-native",
    "transform": {
      "^.+\\.(jsx?)$": "<rootDir>/node_modules/react-native/jest/preprocessor.js",
      "^.+\\.tsx?$": "ts-jest"
    },
    "transformIgnorePatterns": [
      "node_modules/(?!react-native)/"
    ],
    "testMatch": [
      "**/__tests__/**/*.[jt]s?(x)"
    ],
+   "moduleFileExtensions": [
+     "ts",
+     "tsx",
+     "js",
+     "jsx",
+     "json",
+     "node"
+   ],
    "globals": {
      "window": {},
      "ts-jest": {
        "tsConfig": "./tsconfig.jest.json"
      }
    }
  },


[GOBBLIN-1392] Replacing unsafe File.createTempfile to java nio's Files.createTempFile

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-1392] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1392

Description

[ ] Here are some details about my PR, including screenshots (if applicable):
Replacing unsafe File.createTempfile to java nio's Files.createTempFile

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

+1 It looks good to me. Thanks for making the change.


Some thoughts about the button component

A few thoughts I had while reviewing the <Button /> component:
1. Why is my text inside an  tag?
This tag is for abbreviations, it is totally out of place here. The  tag is for this:
HTML

2. I'd like to be able to use children for my button content
This feels much more natural to me... hopefully I'm not crazy ;) :
<Button>Hello World</Button>

// or perhaps with an icon

<Button>
  <FontIcon ... />
  Hello World
</Button>

// and a ripple!

<Button ripple>
  <FontIcon ... />
  Hello World
</Button>

3. Where did you find the loading animation?
Not sure the ripple is a good loading animation for UX, where have you seen it in google's apps or spec? From my understanding, the ripples are supposed to be an event that is ink spreading outwards from the point of input from your mouse/finger -- not loading animations.
4. Would be useful to be able to use as an anchor link too
This can be done by checking for a href=..... prop and using React.createElement() to conditionally create an  tag instead of a <button>.

Hi @nathanmarks
I will try to answer all your questions ;)
1. Why is my text inside an  tag?
Our wrong is better use  element
2. I'd like to be able to use children for my button content
My question for you is... why? now you can use the attribute icon for that purpouse. <button> its a simple component (not a composition one) we think is better keep simple.
3. Where did you find the loading animation?
Totally agree, any porpoise?
4. Would be useful to be able to use as an anchor link too
Like <button> we have <link> (now I think anchor is better name). If we respect the HTML Spec is better separate both, and keep them in separate files.
best,
javi

@soyjavi Hey Soyjavi
Thanks for the response!
It is very easy to support label= AND <Button>Label</Button>. This is still a simple component except it is much closer to how you use the <button> HTML element which feels more natural to me.
const { label, children } = this.props

render (
  <button>{label ? label : children}</button>
);

I will take a look through some google apps and see what they tend to do for loading on buttons.

@soyjavi Also, even though the  tag is called anchor, I think most people just use the term "anchor link" to refer to in-page links -- <Link> is just fine :)
However, the problem with the link right now is that it does not let you style <a href...... as buttons. This is what I was referring to when I mentioned making the button smart enough to allow it to be implemented as either a <button> or  tag.

I think to resolve the anchor problem, we can provide a button boolean property to the Link so in case it's true we can apply button styles imported from the Button component. That would keep semantics.  About the children I think it's worthy to support both label and children. Makes sense in case you want to provide a custom icon, I mean an icon not provided in Material Design icon font. Both are very easy changes that can be done and included in a few minutes!!

@javivelasco Great! I think that's an awesome idea.
It would be nice not to have to duplicate any code to use all the same features (accent, ripple, etc etc) that buttons have. How will you achieve that with the link?

Yes, I've just realized as I was reading :sweat_smile:
To keep it separated I think we should add those properties to links as well but that's actually meaning a look of duplication because we are not only importing the button style but also the modifiers. Maybe instead of importing just the style, the Link can hold a dependency with the Button component.
Then, in the button component we can have a static method that gets the properties and returns className string which are actually from buttons but are dumped into the anchor. The ripple can work independently I guess. BUT... it would be a lot easier to accept an href in the button and make the Link just a wrapper.

Accepting href in the button and conditionally making it a  element is much easier.
Also, technically, visually it is still a "button", which is why we're using the <Button> components.
Here's a contrived example of how it would work:
const { children, className, href, ...otherProps } = this.props;
const element = href ? 'button' : 'a';

return React.createElement(element, {
  className,
  href,
  ...otherProps
}, children);

Yes, I agree with you. It's way easier at implementation level and it makes sense. Let's see @soyjavi opinion!

@nathanmarks I'm thinking in the perfomance if we create a composite component I think the render gets more time (because each component has a render function). In my opinion I prefer keep simple, I understand perfectly that card component needs subcomponents... but button why? We need respect the spec of Material Design and button spec is quite simple.
what do you think about perfomance?
best,

@soyjavi It doesn't even need to accept subcomponents other than a string if you'd prefer to keep it tightly controlled:
static propTypes = {
  children: PropTypes.string
}

This is purely so we can use the <Button> component like a regular HTML element:
<Button>Click Here!</Button>

In terms of the  vs <button> issue, the reasoning behind that is because at the moment it is impossible with this library to create a link that is a button visually.
You cannot do Action with a button as href is not a valid attribute for <button>.

@soyjavi just to add RE the performance, I will do some tests later

Regarding the button icon property, right now you can't use other Icon Fonts like Font-Awesome, which supports stacked icons, and thus requires additional markup than a single  tag

@soyjavi @javivelasco
One more thing, about the props... kind= seems like a poor english translation.
Even though you hear people say, "Oh, I like this kind of X", in technical documentation type= would be a much more appropriate prop name.
To take it another step further -- since things like accent and primary are props instead of color=xx, maybe raised should just be a prop for consistency? Also means less writing :)

Yeah I know. Actually it was the best I've found to replace type since it should be used to be transferred to the button tag. Do you think we can use a better one? We can change the props definition for buttons if we can find something that makes more sense of course, but always respecting transferring properties to the button tag :)

@javivelasco I would just do a raised boolean prop, and have flat as default. This makes it consistent with the other button options such as color (accent/primary)

Ok I think it's fine. Tomorrow I'm adding it here. Gotta go now!


Inconsistent spring.cloud.util.enabled property

Bug report
Hi, I noticed some inconsistency regarding properties.
Setting spring.cloud.util.enabled: false prevents the application from starting because of SimpleDiscoveryClientAutoConfiguration and later on because of SimpleReactiveDiscoveryClientAutoConfiguration because they require a InetUtils bean provided by UtilAutoConfiguration.
This can be fixed by also disabling those 2 configuration classes by property but it feels a bit strange because they look unrelated to InetUtils.
I wanted to open a PR for this but I was mixed between suppressing the spring.cloud.util.enabled or turning the other 2 ClientDiscovery configuration conditional upon this property.
Thanks in advance for suggestions 😄

spring-cloud-commons 2.2.0.RC1

I think the autowiring should not be required and checking for null in the right places. PRs welcome.


Ensure "travis" user and db are present for all PostgreSQL versions

The good news is that the existing addon Just Works™ even when installing PostgreSQL from the pgdg repo.  Cycling through both ports 5432 and 5433 is a bit of a hack, as installing previously absent versions of PostgreSQL while there's a server still running will cause the installer to select the next unused port, which is 5433.
Working build in staging: https://staging.travis-ci.org/meatballhat/yolo-octo-adventure/builds/433553
Associated config: https://github.com/meatballhat/yolo-octo-adventure/blob/5b2db2f53e2a8dbd599f067b8dc85addaa26aae0/.travis.yml

Hmm. I am not quite sure this is the right thing to do, though. The problem is running the installer when an instance is already running. Isn't it more appropriate to figure out why we have PostgreSQL listening on a non-default port and eradicate that possibility?
Also, what stops a third PostgreSQL instance from being configured?
We can brute force createuser and createdb for these ports, but isn't it wiser to scan the configuration files and find the defined ports instead?

@BanzaiMan The problem is doing this on container-based where we don't (and probably shouldn't) change the running state of services based on what packages are being installed.  The installation of PostgreSQL while an existing server is running causes the new installation to be configured with the next available port after the default of 5432.  This is expected behavior.  Nothing is stopping more PostgreSQL instances from being configured.  The intent of this PR is to cover a known use case of installing PostgreSQL 9.5.  Scanning config files and changing the running state of services is certainly possible, but this solution is what I know to work given the current PostgreSQL and APT addons on container-based.

OK.

This PR might be the cause of travis-ci/travis-ci#4570.


Next() for Localhost

Whenever I use this package, it doesn't work in my local environment because https://localhost:### throws errors. Can we add another check like this?
if(req.headers && req.headers.host && req.headers.host === 'localhost:####') {
next();
}

Maybe just do something like this?
if (app.get("env") === "production") {
app.use(enforce.HTTPS({ trustProtoHeader: true }));
}


Request: Remove dependence on external resources

The file layouts/partials/meta.html contains links to external fonts (from googleapis.com) and css/js (from Cloudflare). It would be nicer to have relevant features and fonts locally instead of relying on external services. I believe this is also more in line with Hugo sites and philosophy in general.

But I need the font and css/js? Or you think I should download the font, css, and js and include it with the theme?

Absolutely, that would be a very good move. This is also more in line with current development within EU, if that matters. Linking to Google fonts is no longer recommended.

Well I'm happy if you can provide the PR, I am not EU citizen, and this theme can be use and modify by anyone if they need to comply with local regulation.


feat: adding minimal tailwind setup

https://stackoverflow.com/questions/71783177/remove-specific-style-from-tailwind-base
The idea would be to start removing some of the global styling in layout.css and creating components using tailwind rather than CSS modules https://ui.shadcn.com/docs/components/
This can be done when when moving to app directory or as a slow migration.
Note: not everyone likes tailwind, so leaving this as optional but it would help to rewrite some of the components using radix + tailwind for accessibility reasons.

First step toward https://github.com/react-hook-form/documentation/issues/915#issuecomment-1597798175


Tflite convert

Hi, can you provide the tflite checkpoint for this project. Thank you

You can convert Pytorch model to TFlite model.


Add Simple Icons

Hi, maintainer of Simple Icons here 👏🏼
Have you considered to add simple-icons as part of the package? You can see all icons that the library provides at https://simpleicons.org
All icons are in a 24x24 px format using viewBox="0 0 24 24" (see example here), including the brand color as part of the npm package. You may find useful NodeJS usage section of the docs and metadata explanation.
If you need help with the implementation I would be happy to help. Cheers!

Thank you for the suggestion; I've added them via version 0.7.1 :tada: and included a small (b/c they are very detailed, so they have a large impact on loading time and binary size) sample in the gallery:
.


QC pass command is not working

“QC   Pass” was not happening from PES device number 703, “Invalid Punch” was   showing even      though everything was fine

Solved by software team


”Golăiești”, ”Bran”, ”Cilibiu”, ”Cotu lui Ivan” are listed in the drop-down menu

Description:
When user writes ”Golăiești”, ”Bran”, ”Cilibiu”, ”Cotu lui Ivan” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Golăiești"

Expected results
The location appears in the drop-down menu as "Sat Golăiești, IAȘI (Golăiești)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Bran"

Expected results
The location appears in the drop-down menu as "Sat Bran, IAȘI (Golăiești)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Cilibiu"

Expected results
The location appears in the drop-down menu as "Sat Cilibiu, IAȘI (Golăiești)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Cotu lui Ivan"

Expected results
The location appears in the drop-down menu as "Sat Cotu lui Ivan, IAȘI (Golăiești)".


Missing listener after copying pool

Describe the Bug
Listener is missing after copying a pool
Steps to Reproduce
Steps to reproduce the behavior:

create a pool and add a listener
copy the pool

Expected Behavior
Listener will be copied too
Environment
Please complete the following information:

OS: win10
Camunda Modeler Version: 3.0.0-beta0

Depends on https://github.com/camunda/camunda-modeler/issues/1223

related to #681 and #540

Fixed in Camunda Modeler 3.3+.


How to get tomorrow's trading order

❓ Questions and Help
We sincerely suggest you to carefully read the documentation of our library as well as the official paper. After that, if you still feel puzzled, please describe the question clearly under this issue.
I read the Online Serving in https://qlib.readthedocs.io/en/latest/component/online.html ，but I can‘t understand how to get tomorrow's trading order，just like the backtest.

@zzz6060092
Ask if you want sell_ORDER_list & buy_order_list
https://github.com/microsoft/qlib/issues/215


WDAC Integration - Tenant Restrictions V2

Hi
Is the further documentation around using WDAC with tenant restrictions? For example there is no documentation to state what the app tag should be.. Just "Before enabling firewall protection, ensure that a Windows Defender Application Control (WDAC) policy that correctly tags applications has been applied to the target devices. Enabling firewall protection without a corresponding WDAC policy will prevent all applications from reaching Microsoft endpoints. This firewall setting is not supported on all versions of Windows - see the following link for more information.
For details about setting up WDAC with tenant restrictions, see https://go.microsoft.com/fwlink/?linkid=2155230"
That URL goes to a generic page about WDAC and not within context to using it with Tenant Restrictions.
Thank you

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

ID: 18542d7f-4faa-5a87-aa08-45f1b4117f51
Version Independent ID: ef879213-0fe2-8cd8-1211-6e2fea8273f9
Content: Configure tenant restrictions - Microsoft Entra ID - Microsoft Entra External ID
Content Source: docs/external-id/tenant-restrictions-v2.md
Service: active-directory
Sub-service: b2b
GitHub Login: @msmimart
Microsoft Alias: mimart

@SirdiJ
Thanks for your feedback! We will investigate and update as appropriate.

@msmimart
Can you please check and add your comments on this doc update request as applicable.

@SirdiJ
I'm going to assign this to the document author so they can take a look at it accordingly

Hiya, is there a time frame I can look out for, for this? Looking forward to testing it, thanks

Hi @SirdiJ, I'm checking with the product team for clarification and will post here when I have more information. Thanks again for bringing this to our attention.

@SirdiJ, thanks for bringing this one up - We have exactly the same query, there isn't information/links that seem appropriate to implement a working solution with the flag set.

Hi @SirdiJ, I'm checking with the product team for clarification and will post here when I have more information. Thanks again for bringing this to our attention.
@vimrang fyi

Hi please may we have an update on this ? Thank you

Same boat @SirdiJ, haven't managed a working solution based on what's provided yet when trying to wrangle chrome in. Hoping an update is posted soon.
I've added

@SirdiJ, i've made a little progress testing this out:

Chrome blocked when logging into 365
Edge not blocked when
Native applications can't login.

I've ended up creating the file as:
$rule = New-CiPolicyRule -FilePathRule "C:\Program Files\Google\Chrome\Application\Chrome.exe" -Block
New-CIPolicy -rules $rule -FilePath "C:\Temp\WDAC Testing\AppTagging-Chrome.xml" -AppIdTaggingPolicy -AppIdTaggingKey "M365ResourceAccessEnforcement" -AppIdTaggingValue "Allowed"
Once this is done i've created another policy referencing $rule = New-CiPolicyRule -FilePathRule "C:\*" and merged the two policies together before finishing things off:
Set-CIPolicyIdInfo -ResetPolicyId -PolicyName "Tenant Restrictions" -PolicyId "a530a518-f160-48cb-8595-ce1ebcfe6b2c" -AppIdTaggingPolicy -FilePath "C:\Temp\WDAC Testing\AppTagging-Chrome.xml"
Set-RuleOption -Option 0 .\AppIdPolicy.xml  # Usermode Code Integrity (UMCI)
Set-RuleOption -Option 16 .\AppIdPolicy.xml # Refresh Policy no Reboot
Set-RuleOption -Option 18 .\AppIdPolicy.xml # (Optional) Disable FilePath Rule Protection
Convertfrom-CIPolicy "C:\Temp\WDAC Testing\AppTagging-Chrome.xml" "C:\Temp\WDAC Testing<GUID>.cip"
copy "C:\Temp\WDAC Testing<GUID>.cip" c:\windows\system32\codeintegrity\CiPolicies\Active\
Start-Process "C:\Temp\RefreshPolicy(X86).exe"

The debugger shows the tag on processes other than Chrome. Unfortunately, native applications (Office, Teams, Intune) still can't connect once applied (I was hoping the C:\* whitelist would address this).
Hopefully updated documentation is available soon.


Connecting to bot opens Azure Gov webpage

Screenshots

Version
4.7.0
Describe the bug
Connecting to bot opens Azure Gov webpage if both: using appId/pass AND pressing enter/return instead of clicking Connect
Steps to reproduce

Click Open Bot
Enter endpoint, appid, and pass
Press enter or return instead of clicking connect
Watch the azure gov page open

This could be repro'd by a few others on the support team, as well.
[Bug]

Sorry, it seems I don't have permissions to remove the labels.

Shouldn't this be an Emulator issue?