Add /import/portfolio endpoint to enable integration with third-party tools

Pull request details

Summary

New POST /import/portfolio endpoint to temporarily upload portfolios from third-party apps (60 seconds time-to-live)
New endpoints API docs: rendered
Portfolio JSON Schema: schema.json
New client /import?p=[id] route to trigger FE import of uploaded portfolio

Demo
https://github.com/leonardoarcari/dcapal/assets/811969/943b4543-c0cc-4943-8fb8-119664819871
Breaking changes

None
Issues fixed

#34

Cancel should redirect to homepage from import functionality
If I click the import portfolio, select the json file I'm redirected correctly to the allocate step. If I click the "cancel" button here I'm getting redirected to the currency step at the moment. I understand that it's the same step of the manual flow, but I would consider to differentiate the behaviour depending where the user came from. If the user came from the import portfolio functionality I would expect to be redirected to the homepage (maybe I've choosen the wrong JSON and I want to re-import the correct one).
What do you think?

Cancel should redirect to homepage from import functionality
If I click the import portfolio, select the json file I'm redirected correctly to the allocate step. If I click the "cancel" button here I'm getting redirected to the currency step at the moment. I understand that it's the same step of the manual flow, but I would consider to differentiate the behaviour depending where the user came from. If the user came from the import portfolio functionality I would expect to be redirected to the homepage (maybe I've choosen the wrong JSON and I want to re-import the correct one).
What do you think?

@KristianLentino99 Where are you experiencing that behavior? Current stable? I'm asking because after #177 got merged, allocate step brings you back to the My Portfolios page, which superseded the currency step. Maybe you can attach a quick video? Thanks!

Cancel should redirect to homepage from import functionality
If I click the import portfolio, select the json file I'm redirected correctly to the allocate step. If I click the "cancel" button here I'm getting redirected to the currency step at the moment. I understand that it's the same step of the manual flow, but I would consider to differentiate the behaviour depending where the user came from. If the user came from the import portfolio functionality I would expect to be redirected to the homepage (maybe I've choosen the wrong JSON and I want to re-import the correct one).
What do you think?

@KristianLentino99 Where are you experiencing that behavior? Current stable? I'm asking because after #177 got merged, allocate step brings you back to the My Portfolios page, which superseded the currency step. Maybe you can attach a quick video? Thanks!

Oh yeah sorry, but still in my opinion if as a user I click "back" button coming from the new import functionality I would expected to come back to the homapeg to retry to import a new file. Maybe I'm just biased.
https://github.com/leonardoarcari/dcapal/assets/60007558/91af46c8-d170-4048-a382-690b5ed5d085

If I import two times a file without a name I get two "Portafoglio principale", in my opinion could be a little misleading and we can add a number maybe to make the user understand which is the older (and so he can delete it).

If I import two times a file without a name I get two "Portafoglio principale", in my opinion could be a little misleading and we can add a number maybe to make the user understand which is the older (and so he can delete it).

@KristianLentino99 Very good point, I agree. Took me a while, but made it work 😄 Can you give it a try? Thanks!

@leonardoarcari the increment of the portfolio count looks great!


withAuthenticator throwing randomBytes is not a function

After updating amplify to "aws-amplify-react-native": "^0.2.0" I'm getting this error in my login screen undefined is not a function (Evaluating '_global.util.crypto.lib.randomBytes(128)')

I've searched and found that this might be a amazon-cognito-identity-js version issue https://github.com/aws/amazon-cognito-identity-js/issues/646 but adding the line package.json doesn't solve the issue "aws-sdk": "2.177.0"
I'm trying with a this bare-bones app, but I'm getting the same Issue
// AWS Imports
import aws_exports from './aws-exports';

// React Imports
import React, { Component } from 'react';
import { View, Text } from 'react-native';
import Amplify from 'aws-amplify';
import { withAuthenticator } from 'aws-amplify-react-native';
Amplify.configure(aws_exports);

class App extends Component<{}> {
    render() {
        return (
            <View>
            <Text>Test</Text>
            </View>
        );
    }
}

export default withAuthenticator(App);

I'm using aws-amplify to configure as suggested to me here https://github.com/aws/aws-amplify/pull/109#issuecomment-362676278
Any ideas what might be going on? Thanks!

Hi @Darkade aws-amplify-react-native 0.2.0 still referencing aws-sdk higher than 2.177.0. Could you clear node_modules then reinstall aws-amplify-react-native to make sure it is on 0.2.1 and see if problem still exists.

Hey @richardzcode, thanks for your answer. The issue persists.
I deleted node_modules and removed the aws-amplify-react-native from package.json
Then I npm install'd everything. Finally i ran npm install aws-amplify-react-native --save
This is my package.json in case it clarifies anything.
{
  "name": "portatic",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "start": "node node_modules/react-native/local-cli/cli.js start",
    "test": "jest"
  },
  "dependencies": {
    "aws-amplify-react-native": "^0.2.1",
    "react": "16.0.0",
    "react-native": "0.51.0",
    "react-native-elements": "^0.19.0",
    "react-native-gifted-chat": "^0.3.0",
    "react-native-vector-icons": "^4.5.0",
    "react-navigation": "^1.0.0-beta.27"
  },
  "devDependencies": {
    "babel-jest": "22.0.4",
    "babel-preset-react-native": "4.0.0",
    "jest": "22.0.4",
    "react-test-renderer": "16.0.0"
  },
  "jest": {
    "preset": "react-native"
  }
}

@Darkade this occurs when you don't run
react-native link amazon-cognito-identity-js

did you run this command? This is needed for the randombytes function if you are not using Expo 25+

Thanks. I did ran react-native link with 0.2.0 with 0.2.1 I just ran it, but the issue persists.
$ react-native link amazon-cognito-identity-js
Scanning folders for symlinks in /Users/ivan/git/portatic/client/node_modules (18ms)
rnpm-install info Android module amazon-cognito-identity-js is already linked
rnpm-install info iOS module amazon-cognito-identity-js is already linked

We do have an Expo branch, I'm going to try it there
Thanks guys

@Darkade The issue fixed. Could you get the latest aws-amplify-react-native (0.2.2) and have a try?
Thanks!
Richard

Hey guys, I tried updating to<EMAIL_ADDRESS><EMAIL_ADDRESS>but I'm getting the same error.
Here's what I did. Am I missing something?
$ rm -rf node_modules android ios
$ yarn install
$ react-native eject
$ react-native link amazon-cognito-identity-js
$ react-native run-android

Also, here's my package.json
{
  "name": "portatic",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "start": "node node_modules/react-native/local-cli/cli.js start",
    "test": "jest"
  },
  "dependencies": {
    "aws-amplify-react-native": "^0.2.2",
    "react": "16.0.0",
    "react-native": "0.52.2",
    "react-native-elements": "^0.19.0",
    "react-native-gifted-chat": "^0.3.0",
    "react-native-vector-icons": "^4.5.0",
    "react-navigation": "^1.0.0-beta.27"
  },
  "devDependencies": {
    "babel-jest": "22.0.4",
    "babel-preset-react-native": "4.0.0",
    "jest": "22.0.4",
    "react-test-renderer": "16.0.0"
  },
  "jest": {
    "preset": "react-native"
  }
}

What's interesting to me, but maybe is not important. Is that neither the metro window, or the log-android show me the error. Only the little message in the withAuthenticator sign in screen shows it.
I really appreciate your support, guys

@Darkade can you use npm instead of yarn? We did find that there was an issue that causing the error when using yarn. But npm had no problem.
We fixed that issue and yarn works for us now. We'll look into more, but in the mean time maybe you could try npm?

@richardzcode it worked! Thanks so much, Richard

I'm getting this same error when logging in a user
"aws-amplify": "^0.2.4",
"aws-sdk": "^2.192.0"

@rayhaanq in react native you have to run
react-native link amazon-cognito-identity-js

for that function.

oh sorry, not using react native. Just the normal js library

@rayhaanq Ah ok thanks. It is the same requirement for both (currently). So if you are using js/rn right now, our libraries require a pinned version of the aws-sdk to 2.177.0 since later versions dropped the crypto browserify library. We are releasing an updated cognito sdk this week that will remove this requirement, but for now if you can pin that version for aws-sdk it should solve this for you.


[stable/jenkins] Issue 22633, remove the wrong value 'master.overwritePluginsFromImage'

What this PR does / why we need it:
Fix the issue #22633, change the wrong value master.overwritePluginsFromImage
Which issue this PR fixes
Fix the issue #22633
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

In fact, this is a very simple but important bug, just have no idea why you guys have not found and fixed this

@UkonnRa Thanks for the PR. Can you fix the DCO check? CI system also requires that you bump the version of the chart. Also an entry in CHANGELOG.md would be great.
Thanks you.

Hi. Please merge.

@UkonnRa I included you fix in #23175 so that we can merge it soon.


Data not dispay in autocomplete view

When i enter key data gets from api and also print in console.log but that data not showing in autocomplete view

I passing wrong data


EID-1586: Removed Connector Node Metadata generation

As this is now generated my the VMC and MDGen, remove it from this repo as its no longer needed.

Do we also need to update the verify-metadata repo before merging this as it uses this script?


Remove recentering for HashLifeAlgorithm

Codecov Report

Merging #442 (3a0b6aa) into main (96b80e2) will increase coverage by 1.46%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #442      +/-   ##
==========================================
+ Coverage   75.36%   76.82%   +1.46%
==========================================
  Files          78       79       +1
  Lines        4083     4022      -61
  Branches      697      686      -11
==========================================
+ Hits         3077     3090      +13
+ Misses        596      531      -65
+ Partials      410      401       -9

Impacted Files
Coverage Δ

...exvanyo/composelife/algorithm/HashLifeAlgorithm.kt
93.85% <100.00%> (+21.27%)
:arrow_up:

...lexvanyo/composelife/patterns/SpaceshipPatterns.kt
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Master

Make TdtIO more robust:
Add a function “is_block” to class TdtIO. This function checks the
directory to read is indeed a tdt block, to make this IO more robust to
unexpected files in the tank folder, like temporary files such as
“.DS_Store” on the Mac Platform

Thank you for this.
OK for me.
@apdavison : ok for you ?


Cannot run ./install.sh

I sucessfully installed commanderPi the first time but I was unable to run so I had to reinstall, I accidently deleted commanderpi.desktop from /usr/share/applications. After that I moved CommanderPi directory from /Downloads to /home/pi, now running sudo ./install.sh returns this error message:
Save desktop shortcut to /root/Desktop/commanderpi.desktop
Traceback (most recent call last):
  File "c_desktop.py", line 20, in <module>
    f = open(d_dir, "w+")
FileNotFoundError: [Errno 2] No such file or directory: '/root/Desktop/commanderpi.desktop'
chmod: cannot access '/root/Desktop/commanderpi.desktop': No such file or directory

How do I fix it?

Rather than raise basically the same issue, i thought i would add. I hope that is ok.
First great product!
Have installed this fine on raspbian and then rpi os 32 bit.
on the beta version of rpi os 64 bit, when i go to install, then i got multiple errors.
Your current master fixed a lot of the issues
hence why i have one last issue which is as below and the same as ashutoshkvdlw123
Thank you
Save desktop shortcut to /root/Desktop/commanderpi.desktop
Traceback (most recent call last):
File "c_desktop.py", line 20, in
f = open(d_dir, "w+")
FileNotFoundError: [Errno 2] No such file or directory: '/root/Desktop/commanderpi.desktop'
chmod: cannot access '/root/Desktop/commanderpi.desktop': No such file or directory

Save desktop shortcut to /root/Desktop/commanderpi.desktop

I don't know how to do that. That's the problem.

Hi my comments were geared so Jack477 would see there is an issue with more than one person and would hopefully guide us in the right direction

Hi, you should install it without sudo beacuse then it create shortcuts for root user. Just use ./install.sh . Tell me if it works.  There is my fault beacuse in readme.txt I tell to use 'sudo ./install.sh' but I removed it yesterday.

No worries, will try that. Thank you for coming back so quickly

Hi Jack477, did as stated. script pushes through on terminal and a desktop image appears now. However when i double click on the icon and choose either execute or execute from terminal i cant get the GUI to pop up

Maybe it's old desktop shortcut created when you use sudo to install, remove it and use ./install.sh it should create new one.
What OS do you have?

i am using rpi os 64 bit
sorry how do i remove it - just delete

Yes, just delete

so i deleted everything, i re-downloaded your master followed the script including revision and still no GUI?

Try to run by terminal sudo python3 /home/pi/CommanderPi/src/main.py /home/pi is it works?

pi@raspberrypi:~ $ sudo python3 /home/pi/CommanderPi/src/main.py /home/pi
python3: can't open file '/home/pi/CommanderPi/src/main.py': [Errno 2] No such file or directory
pi@raspberrypi:~ $

I know, rename CommanderPi-master to CommanderPi

just bear with me, i was trying the same thing but the other way

so after renaming i get
Invalid desktop entry file: '/home/pi/Desktop/commanderpi.desktop'

ok, big sigh of relief. i started from scratch. i extracted the master. renamed to CommanderPi then did the install script and hey presto

and your script from the terminal also brought up the GUI

I'm glad to hear it, on github I have section 'Download' where is my own link, I don't prefer to download master, but I do some change for github-master download to rename this folder automaticly.


Don't understand how to use the pulse option on a switch

I'm trying to control a relay thanks to your plugin but for some reason I can't use the pulse option. So I think i'm not using it properly. What I would like to do is to switch my relay ON for a short period of time and switch back OFF after that. Is this how the pulse is supposed to work ?
In my case the switch is turned OFF after around 20-30 seconds even if I specify a few ms.
My configuration :
{
"device": "switch",
"name": "test",
"gpio": 5,
"reversed": true,
"pulse": 50
},
Can you help me with this issue ?

[5/30/2020, 23:19:29] [Raspberry GPIO] localhost test: On changed from false to true
[5/30/2020, 23:19:29] [Raspberry GPIO] localhost: command WRITE (4) 5 0
[5/30/2020, 23:19:29] [Raspberry GPIO] localhost: command WRITE (4) => 0
[5/30/2020, 23:20:10] [Raspberry GPIO] localhost test: gpio 5: low
[5/30/2020, 23:20:10] [Raspberry GPIO] localhost: command WRITE (4) 5 1
[5/30/2020, 23:20:10] [Raspberry GPIO] localhost: command WRITE (4) => 0

What's you full config?  Did you see any errors in the log during Homebridge startup?

Well I tried again with the new version 1.1.7 and a new version of node 12.17.0 and now it works. I don't what was wrong in the first place. Thanks anyway.

v1.1.7 fixed a bug where changed GPIO values wouldn't be pushed to Homebridge RPi when localhost was used in config.json.  The timer for turning the switch of is started on receipt of the notification.


Executing scripts by relative path without a leading ./ results in a limited command search.

Steps to reproduce using wildcards (verified on macOS and Windows)
new-item -ItemType Directory -Path repro2
'repro' | Out-File -Path repro/1.ps1
& 'repro/[1].ps1'

Expected behavior
repro

Actual behavior
repro\1.ps1 : The module 'repro' could not be loaded. For more information, run 'Import-Module repro'.
At line:1 char:1
+ repro\1.ps1
+ ~~~~~~~~~~~
+ CategoryInfo          : ObjectNotFound: (repro\1.ps1:String) [], CommandNotFoundException
+ FullyQualifiedErrorId : CouldNotAutoLoadModule

Actual behavior for wildcard case
repro/[1].ps1 : The term 'repro/[1].ps1' is not recognized as the name of a cmdlet, function, script file, or operable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1
+ repro/[1].ps1
+ ~~~~~~~~~~~~~
+ CategoryInfo          : ObjectNotFound: (repro/[1].ps1:String) [], CommandNotFoundException
+ FullyQualifiedErrorId : CommandNotFoundException

Environment data

> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.1.2
PSEdition                      Core
GitCommitId                    6.1.2
OS                             Darwin 18.5.0 Darwin Kernel Version 18.5.0: Mon Mar 11 20:40:32 PDT 2019; root:xnu-4903.251.3~3/RELEASE_X86_64
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

By design failure, as it overlaps with Module Qualified syntax, which should take precedence
new-item -ItemType Directory -Path repro2
'repro' | Out-File -Path repro/1.ps1
repro\1.ps1

See #10997 for a related issue with commands in PATH failing to be found if they contain wildcard's.  Both issues have the same root cause, fixed in #11008.


Reason for lora_radio_helper.h includes LoRaRadio.h outside the include guard?

Is the inclusion of LoRaRadio.h supposed to be outside the include guard?

https://github.com/ARMmbed/mbed-os-example-lorawan/blob/master/lora_radio_helper.h#L18

ARM Internal Ref: IOTCELL-1996

Moved inside guard


Port Animatron to Godot 4

I started to look at this, but it is currently stalled until we find an OSC solution, using GDExtension. Besides missing the OSC library, there are lots of small changes to all the scripting code (even if the Godot 4 project porting tool helps with many things, there are plenty more things it misses).
See  my query to the creator of the GDNative add-on we're currently using with Godot 3.

Too bad. Thanks for looking it up. Not sure if it makes a big difference for Animatron, but being up to date is always better

Checking with Process Monitor I haven't checked them all, but the first 2 I found were:

mswsock
vcruntime

...and more that contain *140 in their names.

The solution for the OSC problem might be using PacketPeerUDP. Joan from Toplap pointed us to this post which shows one way of doing it.


Reload authentication usersFile on changes

Do you want to request a feature or report a bug?
feature

Did you try using a 1.7.x configuration for the version 2.0?

[ ] Yes
[x] No

What did you do?
I added basic/digest authentication to a frontend via usersFile = "/path/to/.htdigest".
Then I started traefik. Then I added another user to the .htdigest.
What did you expect to see?
That I can login with the new user.
What did you see instead?
That I can not login with the new user. After I restarted Traefik I could login.
Output of traefik version: (What version of Traefik are you using?)

containous/traefik:experimental-v1.7

What is your environment & configuration (arguments, toml, provider, platform, ...)?

traefik.toml
logLevel = "INFO" #DEBUG, INFO, WARN, ERROR, FATAL, PANIC

sendAnonymousUsage = false

defaultEntryPoints = ["https", "http"]

[entryPoints]
  [entryPoints.http]
  address = ":80"
    [entryPoints.http.redirect]
    entryPoint = "https"
  [entryPoints.https]
  address = ":443"
    [entryPoints.https.tls]

[file]
  directory = "/etc/traefik/rules/"
  watch = true

[docker]
  endpoint = "unix:///var/run/docker.sock"
  domain = "mydomain.tld"
  watch = true
  network = "traefik"
  exposedbydefault = false

[acme]
  email =<EMAIL_ADDRESS>  storage="/etc/traefik/acme.json"
  entryPoint = "https"
  acmeLogging=true
  [acme.dnsChallenge]
    provider = "netcup"
  [[acme.domains]]
    main = "*.mydomain.tld"
    sans = ["mydomain.tld"]

[api]
  entryPoint = "traefik"
  dashboard = true
  [api.statistics]

[accessLog]
  filePath = "/etc/traefik/access.log"
    [accessLog.fields.names]
    "StartUTC" = "drop"

docker-compose.yml
version: "3.6"
services:
  traefik:
    hostname: traefik
    image: containous/traefik:experimental-v1.7
    container_name: traefik
    restart: always
    domainname: ${DOMAIN0}
    networks:
      - default
      - traefik_proxy
    ports:
      - "80:80"
      - "443:443"
    env_file:
      - netcup.env
    environment:
      - TZ=${TZ}
    labels:
      - "traefik.enable=true"
      - "traefik.backend=traefik"
      - "traefik.frontend.rule=Host:traefik.${DOMAIN0}, traefik.${DOMAIN1}"
      - "traefik.port=8080"
      - "traefik.docker.network=traefik_proxy"
      - "traefik.frontend.auth.digest.usersFile=/shared/admin.htdigest"
      - "traefik.frontend.headers.SSLRedirect=true"
      - "traefik.frontend.headers.STSSeconds=315360000"
      - "traefik.frontend.headers.browserXSSFilter=true"
      - "traefik.frontend.headers.contentTypeNosniff=true"
      - "traefik.frontend.headers.forceSTSHeader=true"
      - "traefik.frontend.headers.STSIncludeSubdomains=true"
      - "traefik.frontend.headers.STSPreload=true"
      - "traefik.frontend.headers.frameDeny=true"
    volumes:
      - "/var/run/docker.sock:/var/run/docker.sock"
      - "./etc-traefik:/etc/traefik"
      - "./shared:/shared"
      - "/usr/share/zoneinfo:/usr/share/zoneinfo:ro"

networks:
  traefik_proxy:
    external:
      name: traefik_proxy
  default:
    driver: bridge

If applicable, please paste the log output at DEBUG level (--log.level=DEBUG switch)
nothing relevant

FWIW, it's possible to work around this by temporarily changing the usersFile path to a blank file then changing it back. Really hacky and hard to automate, but it works.

On docker, sending the SIGHUP signal to the traefik process worked for me, but you need root access to the host.


Java Selenium + PhantomJS 2.0 can not change UserAgent

Hallo!
I try to start parsing page from Google-Search-results:
DesiredCapabilities cap = DesiredCapabilities.phantomjs();
		cap.setCapability(PhantomJSDriverService.PHANTOMJS_PAGE_SETTINGS_PREFIX + "useragent", USER_AGENT);
		cap.setCapability(PhantomJSDriverService.PHANTOMJS_PAGE_SETTINGS_PREFIX + "loadImages", false);
		cap.setCapability(PhantomJSDriverService.PHANTOMJS_PAGE_SETTINGS_PREFIX + "javascriptEnabled", true);
		cap.setBrowserName("chrome");
		cap.setVersion("42");
		WebDriver page = new PhantomJSDriver(cap);

but can not find button with id "#pnnext", because Google hide this. For this case I can change UserAgent and Google allow me to navigate.
but with java+selenium webdriver I can't. And I think UserAgent with PhantomJS 2.0 was not changed.
Capabilities
[{
    applicationCacheEnabled = false,
        rotatable = false,
        handlesAlerts = false,
        databaseEnabled = false,
        version = 2.0.0,
        platform = XP,
        browserConnectionEnabled = false,
        proxy = {
            proxyType = direct
        },
        nativeEvents = true,
        phantomjs.page.settings.useragent = "Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36(KHTML, like Gecko) Chrome/41.0.2272.101 Safari/537.36",
        phantomjs.page.settings.javascriptEnabled = true,
        phantomjs.page.settings.loadImages = false,
        acceptSslCerts = false,
        driverVersion = 1.2.0,
        locationContextEnabled = false,
        webStorageEnabled = false,
        browserName = phantomjs,
        takesScreenshot = true,
        driverName = ghostdriver,
        javascriptEnabled = true,
        cssSelectorsEnabled = true
}]

Maybe do I something wrong? Can you help me?
PS:
windows - 10
phantom - 2.0
selenium.jar - 2.48.2
PPS: the same UA I use in pure PhantomJS js-script, and all work perfekt.
PPPS:
http://luhonghai.com/blog/2014/07/10/selenium-webdriver-set-browsers-user-agent-and-proxy/
doesn't help me :(

Seems like a typo:
phantomjs.page.settings.userAgent
instead of
phantomjs.page.settings.useragent

@nico101 oh... right... I'm sorry. All work!
Thank you very much!

@cyberkatze No problem!


Find the path

Given the generated maze as input, find the path and mark it with 👣.

you want me to debug your script or play with the game ?

you want me to debug your script or play with the game ?

Oh, no :)
I created the issue just after the completed generations script.
I am trying to figure out how to write the script to find the path.
Fortunately, it was pretty easy. :)

one minor note (just a styling thing not a bug):::

Thank you for your comment!
I don't know about this and implemented it in the script.


Fix Invoke actor for scripts in other banks

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce?
Bug fix for invoke actor, similar to the last invoke actor fix.

What is the current behavior? (You can also link to an open issue here)
#369  Invoke actor will cause a crash on return if the actor scripts are in different script banks.
Rare bug, only dealing with large scripts that happen to be in different banks.

What is the new behavior (if this is a feature change)?
May be slightly slower for invoke actor events, but will fix a game crash that is not easy to work around. the demo in #369 will work now.

Does this PR introduce a breaking change?
Scripts that call actor invoke frequently in a loop may notice more lag, but it's already pretty slow, label define and label go to will likely be faster if you don't run out of space for a single script.

Other information:
https://discordapp.com/channels/554713715442712616/570925559346102273/662789686598565888
https://discordapp.com/channels/554713715442712616/575070105919029295/662798184996667405
A check / warning for scripts that are too large and spill into another bank would help, as it creates a similar issue.

Added a warning for scripts that can't fit in a single 16384 byte bank, so it's not just BANKED_DATA_TOO_LARGE, but also says by how much, what script is too big, and to try splitting with Invoke Actors.
The test project in #369 is a good for this. You can copy all 4 actor scripts into the start script and be just over the limit, disabling some chunks of this script will get it working again.
It seems unlikely for many smaller scripts to be too big for a single bank, but if that is a case, BANKED_DATA_TOO_LARGE  will show without the other warnings.

Works great from my testing and seems to fix the issues seen in the project attached to #369
Nice improvement on the error message too.
I've been trying to think for a while if there's a good way to automatically split large scripts across multiple banks but it gets tricky if the split happened in the middle of a loop or if you use label/gotos anywhere.
You could make the JUMP command include a bank number, and add a JUMP to the start of the next bank at points you need to split but it's awkward because at the time the scripts get compiled you don't know which bank the script will eventually end up in and at which points the script will need splitting. Not sure the performance impact of this either.
Maybe a post script compilation step that before banking the data in compileData.js takes the large script, the current bank and the amount of space left in the current bank and splits it into an array of smaller scripts linked together where the first one fits the remaining space and every additional block fits in an empty bank.
(LARGE SCRIPT, currentBank, spaceRemaining) => [SMALL SCRIPT 1 (smaller than remaining space in current bank), SMALL_SCRIPT 2 (smaller than 16384), SMALL_SCRIPT_N...]
I'll keep thinking about this.


Update general UG formatting errors

#174

Codecov Report
Base: 70.72% // Head: 70.72% // No change to project coverage :thumbsup:

Coverage data is based on head (f0a9132) compared to base (b044770).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #201   +/-   ##
=========================================
  Coverage     70.72%   70.72%
  Complexity      713      713
=========================================
  Files           106      106
  Lines          2398     2398
  Branches        317      317
=========================================
  Hits           1696     1696
  Misses          585      585
  Partials        117      117

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Shim away RTCSessionDescription/RTCIceCandidate.

Description
Shim implicit creation of (now redundant) RTCSessionDescription/RTCIceCandidate interfaces in setLocalDescription/setLocalDescription/addIceCandidate.
Purpose
https://github.com/w3c/webrtc-pc/pull/302. I started fixing this in Firefox, then realized people wouldn't benefit in years without having a polyfill.

LGTM, I agree that Frob is a bit ambiguous. What about methodShim ?

LGTM


Integrate concourse

This is a draft PR to update Voyager with the things we need for Concourse.
All feedback is welcome.
Docs incoming

we still need to change Concourse to use voyager (it doesn't currently use it, the code is embedded in the Concourse codebase)


Version bump for a new release

Renamed the app to its final name. :)
@RocketChat/android

 I like it!


Implement base paths (RFC 3529) 1/n: path dep and patch support

RFC: https://github.com/rust-lang/rfcs/pull/3529
Tracking Issue: https://github.com/rust-lang/cargo/issues/14355
This PR add support in path dependencies and patch table and the workspace built-in path base.

Done

Wanting to double check that you intended to not push the mentioned changes.

Wanting to double check that you intended to not push the mentioned changes.

Yes, sorry, not sure what the correct etiquette is with this. Usually I reply to comments and resolve conversations -> push updated code -> require reviews.

Yes, sorry, not sure what the correct etiquette is with this. Usually I reply to comments and resolve conversations -> push updated code -> require reviews.

Not many people remember to "request review" and you never know whether someone will or not (in general, I just assume people won't do it).
In general, its strange to look at a PR with things marked "done" without code pushes, particularly if I'm coming back to the PR several days after the message.

:pushpin: Commit 502c74e0a5ef7a1816456e3d5fb62af6b542e2d7 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 502c74e0a5ef7a1816456e3d5fb62af6b542e2d7 with merge ff6df29857a5fc5f6bb0b9da8436cb501ee5894c...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing ff6df29857a5fc5f6bb0b9da8436cb501ee5894c to master...


Multi-line input for llm chat

Just found myself wanting to paste in several lines of example code and it started executing after the first one.

Brainstormed a bit with GPT-4: https://chat.openai.com/share/a62e9c18-a713-4381-a8dc-417ddeddc15d
I've decided to go with this:
!multi

Starts a multi-line section, which ends with:
!end

But you can also set a custom delimiter, using this:
!multi abc
...
!end abc

Documented here :https://github.com/simonw/llm/blob/9c33d3084376341ca195286ceb6afa8bacb62ed8/docs/usage.md#starting-an-interactive-chat


Update purefb_fs.py

My ansible playbook fails deleting a filesystem with the following:
TASK [Remove backing filesystem] *********************************************************************************************************************************************************************************
fatal: [sn1-r740-d03-13]: FAILED! => {"changed": false, "msg": "Failed to delete filesystem elasticsearch-pipeline."}
Specify named parameter to delete_file_systems() call.
SUMMARY

ISSUE TYPE

Bugfix Pull Request
Docs Pull Request
Feature Pull Request
New Module Pull Request
New Role Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

You missed another issue - I have dealt with both of these in https://github.com/Pure-Storage-Ansible/FlashBlade-Collection/pull/76


Use PureComponents for react-router-dom

Version
4.3.1
This is a feature request.
The problem
When using tools to monitor & optimize unnecessary rendering of components like why-did-you-update https://github.com/maicki/why-did-you-update
It shows that react-router-dom components like Route , Switch, Link, etc.. are being rendered unnecessarily when the props don't change. (i.e: when typing in a text field of a form calling setState in onChange)

Solution
This can be optimised by making the component a pure component and avoiding arrow functions and such in the render methods.
This not only improves the application's performance by avoiding unnecessary renders high up the component tree, but also helps with the developer experience when you're actually trying to figure out which components can be optimized, but find a lot of third party components in your console that you can't really optimize

Unfortunately, this will block any context updates from legacy context API users. While not a problem when using createContext(), we're not quite there yet as an ecosystem.


HMACWith algorithm in Mule 4.1

SD 12:23 PM
ok, the bug is for Mule 4.1. I'm not sure what algorithm we were using and what was configurable then, before DW 2.2.0.
MDA 12:24 PM
yes 4.1 didn’t have that capability
and it was “HmacSHA1”


JRuby Support

twitter-text 2.x depends on the idn-ruby gem which doesn't currently support JRuby. This means that twitter-text also doesn't work on JRuby.
Was this intentional? Is there any workaround? Can the dependency on idn-ruby be made optional?

I second making idn-ruby optional.  I'd happily take an error message saying that a domain name in kanji can't be supported over having to deal with idn-ruby ( which looks like a dead gem dependent on some ugly hackery to install libidn )

Also, having trouble installing idn-ruby even in a standard Rails-MRI setup. @kaushlakers Any thoughts on what and when solution are you folks considering for this?

brew install libidn

@zegomesjf is there anything like that for Ubuntu users?
I tried sudo apt-get install idn, but the gem can't find it
Installing idn-ruby 0.1.0 with native extensions
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.

    current directory: /home/ubuntu/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/idn-ruby-0.1.0/ext
/home/ubuntu/.rbenv/versions/2.5.0/bin/ruby -r ./siteconf20180301-15079-169teyr.rb extconf.rb
checking for -lidn... no
ERROR: could not find idn library!

  Please install the GNU IDN library or alternatively specify at least one
  of the following options if the library can only be found in a non-standard
  location:
    --with-idn-dir=/path/to/non/standard/location
        or
    --with-idn-lib=/path/to/non/standard/location/lib
    --with-idn-include=/path/to/non/standard/location/include

*** extconf.rb failed ***
Could not create Makefile due to some reason, probably lack of necessary
libraries and/or headers.  Check the mkmf.log file for more details.  You may
need configuration options.

Provided configuration options:
	--with-opt-dir
	--without-opt-dir
	--with-opt-include
	--without-opt-include=${opt-dir}/include
	--with-opt-lib
	--without-opt-lib=${opt-dir}/lib
	--with-make-prog
	--without-make-prog
	--srcdir=.
	--curdir
	--ruby=/home/ubuntu/.rbenv/versions/2.5.0/bin/$(RUBY_BASE_NAME)
	--with-idn-dir
	--without-idn-dir
	--with-idn-include
	--without-idn-include=${idn-dir}/include
	--with-idn-lib
	--without-idn-lib=${idn-dir}/lib
	--with-idnlib
	--without-idnlib

To see why this extension failed to compile, please check the mkmf.log which can be found here:

  /home/ubuntu/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/extensions/x86_64-linux/2.5.0-static/idn-ruby-0.1.0/mkmf.log

extconf failed, exit code 1

Gem files will remain installed in /home/ubuntu/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/idn-ruby-0.1.0 for inspection.
Results logged to /home/ubuntu/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/extensions/x86_64-linux/2.5.0-static/idn-ruby-0.1.0/gem_make.out

An error occurred while installing idn-ruby (0.1.0), and Bundler cannot continue.
Make sure that `gem install idn-ruby -v '0.1.0'` succeeds before bundling.

In Gemfile:
  twitter-text was resolved to 2.1.0, which depends on
    idn-ruby

I have this same error , help us to resolve it.
[
Fetching idn-ruby 0.1.0
Installing idn-ruby 0.1.0 with native extensions
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.
current directory<EMAIL_ADDRESS>
/home/inuscg/.rvm/rubies/ruby-2.3.0/bin/ruby -r ./siteconf20180427-3409-q264pw.rb extconf.rb
checking for main() in -lidn... no
ERROR: could not find idn library!
Please install the GNU IDN library or alternatively specify at least one
of the following options if the library can only be found in a non-standard
location:
--with-idn-dir=/path/to/non/standard/location
or
--with-idn-lib=/path/to/non/standard/location/lib
--with-idn-include=/path/to/non/standard/location/include
*** extconf.rb failed ***
Could not create Makefile due to some reason, probably lack of necessary
libraries and/or headers.  Check the mkmf.log file for more details.  You may
need configuration options.
Provided configuration options:
--with-opt-dir
--without-opt-dir
--with-opt-include
--without-opt-include=${opt-dir}/include
--with-opt-lib
--without-opt-lib=${opt-dir}/lib
--with-make-prog
--without-make-prog
--srcdir=.
--curdir
--ruby=/home/inuscg/.rvm/rubies/ruby-2.3.0/bin/$(RUBY_BASE_NAME)
--with-idn-dir
--without-idn-dir
--with-idn-include
--without-idn-include=${idn-dir}/include
--with-idn-lib
--without-idn-lib=${idn-dir}/lib
--with-idnlib
--without-idnlib
To see why this extension failed to compile, please check the mkmf.log which can be found here:
<EMAIL_ADDRESS>extconf failed, exit code 1
Gem files will remain installed in<EMAIL_ADDRESS>for inspection.
Results logged to<EMAIL_ADDRESS>An error occurred while installing idn-ruby (0.1.0), and Bundler cannot continue.
Make sure that gem install idn-ruby -v '0.1.0' succeeds before bundling.
In Gemfile:
twitter-text was resolved to 2.1.0, which depends on
idn-ruby
](url)

Try with this:
sudo apt-get install libghc-gnuidn-dev
This fix it for me!

FYI, @shibafu528 has patches for twitter-text to use simpleidn https://github.com/mmriis/simpleidn instead of idn-ruby
https://github.com/shibafu528/twitter-text/commit/80cdb0c76e64a71401c1bd4cb2f58327c82d99ff
with some modification
https://github.com/shibafu528/simpleidn/commit/69420ce9777a4b2a338dd981b31a3d4fc39dd72b

For mac, do brew install libidn and then you can install  gem install idn-ruby -v '0.1.0' --source 'https://rubygems.org/'

I'd like to bump this issue. On apple m1 arm64 architecture libidn does no longer compile, so brew install libidn is no longer valid.
Is there any reason why switching to simpleidn is not a good idea?

For those using the apple m1, you can install it by passing the path to the lib as it does not seem to be able to find it otherwise. Something like:
bew install libidn
gem install idn-ruby -v '0.1.0' -- --with-idn-dir=/opt/homebrew/Cellar/libidn/1.36/

For those using the apple m1, you can install it by passing the path to the lib as it does not seem to be able to find it otherwise. Something like:
bew install libidn
gem install idn-ruby -v '0.1.0' -- --with-idn-dir=/opt/homebrew/Cellar/libidn/1.36/

This solution worked for me except libidn is now 1.41. Thank you so much!


Nested views: Using views object problem

View the Wiki examples show how to usingviews object
Backbone.Layout.extend({
  views: {
    "header": new HeaderView(),
    "section": new ContentView(),
    "footer": new FooterView()
  }
});

See the code, views is defined in the prototype, so when reload this view (first step remove this view instance, then add another this view instance), it will be broken, because in the first step view remove will change the prototype views, so in the second step views property was empty object, then no subViews render.
The views property should be in instance not in proptype, every view instances should be have one views not share with each other.
I write a exmaple code, view Plunkr
var GrandChildView = Backbone.Layout.extend({
  tagName: 'button',
  events: {
    'click': 'onClick'
  },

  afterRender: function() {
    this.$el.text('Click me');
  },

  onClick: function() {
    alert('GrantChild click');
  }
});

var ChildView = Backbone.Layout.extend({
  views: {
    '': new GrandChildView()
  }
});

var ParentView = Backbone.Layout.extend({
  el: '#content',
  views: {
    '': new ChildView()
  }
});

var parentView = new ParentView().render();

function reload() {
  parentView.setView(new ChildView());
  // the GrandChildView does render anymore.
}

Right, it's intentionally limited (by how JavaScript prototypes work), if you need more power use setViews instead. I never used that declarative syntax on anything substantial, but it works fine in for one off demos as it's nicer than having to call setViews all the time.


CALDAV support

Hi,
would it be possible to add caldav support to sync todos?
DAVx5 could be helpful

I don't quite understand you. The application is translated automatically, determining the system language of the phone. The translation is located in the file ToDark/lib/l10n/translation.dart.


what is the license of this package?

Hello VATr team,
I was wondering if you could help me find license for this project.

Thanks for your comment vchhasatia! Now you can find the license (it is an MIT License by the way) on the main page.

Thank you for quick response.


Start build/deploy

In order to construct build/deploy pipelines, I would like a downstream build or deploy to start when a current build or deploy is succesfull. The API should allow to start builds or deploys of an application.
Once this is in place in the API, I can implement a Wercker step to start the build or deploy.

Currently we're going to focus on readonly data. Being able to trigger builds, and other action is something which will be worked on after this.
Thanks for the feedback :+1:

Any progress on this one?

We're almost done with triggering a build. Will be released next week.

Double :+1:

@hatchan will Wercker release a step to trigger a build for another project? This results in pipelines over projects...

Forgot to close this item, so hope you already read the announcement: http://blog.wercker.com/2015/06/23/Trigger-builds-with-the-api.html
Currently we do not have a step, but I think I already saw someone else create a trigger build step.


[TextField] TextField not working since the update

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example:
Steps:

Current behavior 😯
All the TextField from MUI are creating this bug since I upgraded from 5.13.5 to 5.13.7

Expected behavior 🤔
/
Context 🔦
/
Your environment 🌎
System:
    OS: macOS 11.2.2
  Binaries:
    Node: 20.2.0 - /opt/homebrew/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 9.6.6 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 114.0.5735.198
    Edge: Not Found
    Safari: 14.0.3
  npmPackages:
    @emotion/react: ^11.10.8 => 11.11.1
    @emotion/styled: ^11.10.8 => 11.11.0
    @mui/base:  5.0.0-beta.6
    @mui/core-downloads-tracker:  5.13.7
    @mui/icons-material: 5.11.16 => 5.11.16
    @mui/lab: ^5.0.0-alpha.134 => 5.0.0-alpha.135
    @mui/material: ^5.13.5 => 5.13.7
    @mui/private-theming:  5.13.7
    @mui/styled-engine:  5.13.2
    @mui/styles: ^5.13.2 => 5.13.7
    @mui/system:  5.13.7
    @mui/types:  7.2.4
    @mui/utils:  5.13.7
    @mui/x-data-grid:  5.17.26
    @mui/x-data-grid-premium: ^5.17.26 => 5.17.26
    @mui/x-data-grid-pro:  5.17.26
    @mui/x-date-pickers:  5.0.20
    @mui/x-date-pickers-pro: ^5.0.20 => 5.0.20
    @mui/x-license-pro:  5.17.12
    @types/react:  18.2.14
    react: ^18.2.0 => 18.2.0
    react-dom: ^18.2.0 => 18.2.0
    styled-components:  5.3.11

@yohavrotsztein I'm afraid there isn't enough information for us to look into this, would you mind providing a repro? You can fork this template: https://mui.com/r/issue-template-latest
PS: here are some tips for providing a minimal example/repro: mcve

I have the same error in RTL, but i can not reproduce it with small example...
I am trying to find the root cause..

@mj12albert @mohsenomidi I found the error, it was with the prefixer plugin from stylis. I updated stylis it by mistake and it started to create problem, I fixed it after downgrading stylis to the previous version I had.


nit: markdown linter image has no arm64 build

Bug Report
When running the make mdlint target from an arm64 machine, I see the following:
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested

I am able to reproduce this by running the cluster-api-provider-vsphere/extra/mdlint image by itself
❯ docker run gcr.io/cluster-api-provider-vsphere/extra/mdlint:0.23.2
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested

  Usage: lint [options] <files|directories|globs>

  MarkdownLint Command Line Interface

  Options:

    -h, --help                                  output usage information
    -V, --version                               output the version number
    -f, --fix                                   fix basic errors (does not work with STDIN)
    -s, --stdin                                 read from STDIN (does not work with files)
    -o, --output [outputFile]                   write issues to file (no console)
    -c, --config [configFile]                   configuration file (JSON, JSONC, JS, or YAML)
    -i, --ignore [file|directory|glob]          file(s) to ignore/exclude
    -p, --ignore-path [file]                    path to file with ignore pattern(s)
    -r, --rules  [file|directory|glob|package]  custom rule files

Although, weirdly enough, as we can see from the output above, the utility still runs and reports markdown lint errors.
This issue is to surface this error/warning message and also maybe touch base and see if we want to use a different markdown linter image? Is the cluster-api-provider-vsphere/extra/mdlint one being maintained? It hasn't had a new tag in quite some time.
Expected Behavior
Able to run make mdlint on arm64 without warning or error
Steps to Reproduce the Bug
Run make mdlint from a arm64 (M1 macbook) machine

The mdlint-ci tool that is run inside the container is Node.js code, so I don't think the tool itself is restricted to a given platform. I think it's more due to the fact that the cluster-api-provider-vsphere project we borrow that container image from is only publishing amd64 images. We may want to check with that team if they would be interested in building multiple platform images.
Another option would be to just create our own container. We basically just need to use a base nodejs image and add in the mdlint source.


Propagate oci-ta Pipelines to e2e-tests

This commit passes new environment variables to the e2e tests containing references to the oci-ta build Pipelines. This allows us to test those Pipelines.

Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

I inspected the PipelineRun and verified the new env vars were getting propagated to the e2e Pod.


Corporate Public Page functionality on the site

With corporate logo upload functionality.

Advantages to corporate users:

Incoming link to their site (if they have a site)
Business Page on Svet Media Group site if they are not presented in web.
Social Hub aggregation
Banner order ad functionality with free try out for people who registered to connection event via site


Proposal: standings

Hi,
I propose standings as a missing feature that would be highly useful. It would provide the relevant standings, split into conferences and divisions.
For example:

nba-go standings shows the top 8 teams in both conferences.
nba-go standings east shows the entire Eastern conference.
nba-go standings west shows the entire Western conference.
nba-go standings -d Atlantic shows the Atlantic division.
...

Sure, this is a good suggestion. I'll provide this feature as soon as possible.
Btw, PR is welcome.

I'm down to try to tackle this feature.

As soon as this gets merged in the underlying API that's used, I'll implement the rest in here. https://github.com/bttmly/nba/pull/47

Whats the word on this, anything I can do to help ?

@xxhomey19 I noticed that the data folder is populated with outdated JSON data and I'm curious if this is because you're never actually storing the data after scraping it and instead just printing it to command line. Any insight would be helpful.


Build pipeline

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

sending the pull request


[Bug]: Windows "Terminate batch job (Y/N)?" - Terminal Hangs

Please confirm that you have:

[X] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
Theme
Expected behavior
When running shopify theme dev in the terminal, when I want to close my session I use the shortcut CTRL + C and the session should close.
Actual behavior
When running shopify theme dev in the terminal, when I want to close my session I use the shortcut CTRL + C and the session closes but I then see this message in the terminal Terminate batch job (Y/N)? and when I reply Y the terminal hangs and I need to close it and open a new terminal.
I should note, that with shopify-cli v3.40+ I sometimes get a bug error in the terminal that says I've been logged out and should use shopify login to login again, which I believe is legacy messaging and functionality from shopify-cli v2+. When I get this error I need to close my session and start a new session, and when you combine that bug with this terminal hanging bug, it's just a massive workflow disruptor.
Stack trace
No response
Reproduction steps

shopify theme dev in the terminal.
CTRL + C keyboard shortcut to close the session.
This message appears Terminate batch job (Y/N)? and when I try to proceed with Y the terminal hangs.

Operating System
Windows 10
Shopify CLI version (check your project's package.json if you're not sure)
v3.42.0
Shell
Git Bash
Node version (run node -v if you're not sure)
v18.14.0
What language and version are you using in your application?
Ruby v3.0.5

This has been happening on all versions of cli 3 since i moved from 2 - Windows

Duplicate of https://github.com/Shopify/cli/issues/1104

Not a duplicate, this is for themes while #1104 is for apps. I'm closing 1104 now, and leaving this one.

how can we figure out if an issue is for Theme or App? I thought #1104 was for Theme so my comment over there is in regards to Theme. I'll post it again here for clarity:
the confirmation prompt no longer appears when pressing ctrl+c to terminate the CLI. I first noticed it when I upgraded to the latest version, 3.44.1. It should be safe to close this issue.
On a side note, maybe issues should be prepended with something like [Theme] or [App] to cut down on confusion?

Commenting to keep open. I think this will be an issue as long as we use the Node CLI calling the Ruby server, and aren't intercepting CTRL+C.


Invalid bulk requests with custom contract resolver

I am not exactly sure if this is an issue or if I'm using the API incorrectly.
We also rely heavily on typeconverters to write our objects correctly.
If the classes in your external library do not quite represent the way they should look in your elasticsearch index I'd recommend create a specific POCO for NEST that does. AutoMapper is a great way to convert instances of your external library to instances of indexable object.


Textarea field formatting issue

make this field dinamic, if you write more more characters than it is possible to show, spread this field
currenlty is will go in one line only and without scroll bar it is not visible at all, but spread it down when there is too much content typed

Closed with Commit c7d73fd


Using the library for iOS development

Hi
Has anyone successfully been able to use this library on an iOS app? Any ideas on how to go about doing this?
Thanks

Hello,
Maybe if you make the app as an html5 app. I don’t see any obvious reasons why would not work. No way to know for sure without trying though :)
--
Knut Sveidqvist
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)
On Sunday 8 March 2015 at 13:15, Ali1 wrote:

Hi
Has anyone successfully been able to use this library on an iOS app? Any ideas on how to go about doing this?
Thanks
—
Reply to this email directly or view it on GitHub (https://github.com/knsv/mermaid/issues/130).

Thanks. No, I'm wanting an actual app. I'm thinking the best way would be to get a server to generate the an svg and send it back. But could I keep this open for a bit longer just to see if anyone else has any ideas :)

@Ali1 did you find a way in the end? I am closing this issue now. Let me know if you want me top re-open it.


Add light Go modules

Fixes https://github.com/open-telemetry/opentelemetry-proto-go/issues/100
Thanks to it consumers that do not use gRPC (but send protobuf over HTTP) can import go.opentelemetry.io/light/proto/otlp that does not depend on google.golang.org/grpc.
Side note: I welcome any comments that would improve Makefile.

Postponing (changing to draft as I imagine it may be helpful).

Would go.opentelemetry.io/proto/slim/otlp better communicate the smaller dependency nature of the package?

I like the name as it shows that it has also "has less abstractions" (offers less types, functionality). I will update the PR. Marking as draft before I rename.

PTAL @open-telemetry/collector-maintainers. Just to double-check. AFAIK this codebase is not used by OTel Collector, right? I see that you are generating the code from proto files on your own in https://github.com/open-telemetry/opentelemetry-collector/blob/main/Makefile.

@open-telemetry/go-approvers Bump. I also double-checked this PR here: Tested on https://github.com/open-telemetry/opentelemetry-go/pull/5031


Support native ES6 module loading

According to v5.0 documentation Bootstrap provides a version of Bootstrap built as ESM (bootstrap.esm.js and bootstrap.esm.min.js) which allows you to use Bootstrap as a module in your browser. It's not true if you want to imort bootstrap.esm.js module in vanilla Javascript (tested in 5.0.0-alpha2). In documentation we have example with such module import:
import { Toast } from 'bootstrap.esm.min.js'

It wouldn't work in pure Javascript because so-called “bare” module specifiers are currently not supported in ECMAScript modules specification. Module specifiers must be full URLs, or relative URLs starting with /, ./, or ../. It's not a problem to set proper URL:
import { Toast } from './js/bootstrap.esm.min.js'

Unfortunately such import raises browser exception Uncaught TypeError: Failed to resolve module specifier "popper.js". Relative references must start with either "/", "./", or "../"..

Changing import Popper from 'popper.js' in to import Popper from './popper.js' doesn't help. Such import raises another browser exception  Uncaught SyntaxError: The requested module './popper.js' does not provide an export named 'default'.

As far I find out Propper 1.16 doesn't provide native ES6 module. So for testing I commented out Propper import from bootstrap.esm.js source code:
// import Popper from 'popper.js';

And now native ES6 import system is working!
    <script type="module">
        import { Toast } from './js/bootstrap.esm.js'

        let toastElement = document.getElementById("toast");
        let toast = new Toast(toastElement);

        let buttonElement = document.getElementById("button");
        buttonElement.addEventListener("click", ()=> {
            toast.show();
        });
    </script>

Demo page: http://localhost:5500/bootstrap-v5/bootstrap-v5.0.0-alpha2-esm-test/
I know removing Popper causes Dropdowns and Tooltips not working but for me it's not a problem. I think it would be great to have each component (Toast, Alert, Button, ...) in separate ESM module. Folks loving vanilla Javascript like me would be able to import classes that don't depend on Propper without modifying Bootstrap's source code.
I also think that writing in documentation that Bootstrap provides a version of Bootstrap built as ESM is misleading. It's rather UMD module.

Operating system and version: Windows 10
Browser and version: Opera 71
Working example using pure ES6 module system: https://qsites.github.io/bootstrap-v5/bootstrap-v5.0.0-alpha2-esm-test/

Se also #28386 and #27119

Popper 1.6 does support ES modules
<EMAIL_ADDRESS>

Any PRs to improve things are welcome. It's been a while since we set up ESM in v5 and things have progressed since then.

I do not see this working with popper in the near future. The best move preventing any confusion would be to adjust the docs to explain that this only works when using a bundler like webpack or similar.
The problem is that popper (even though it supports es modules) is not able to be resolved by the browser. The spec currently only covers relative references like /script.js and ./script.js but not specifiers like @popperjs/core which is used in the code. This leads to the aforementioned errors and the module becomes unusable.
The only way I know to fix this would be to use a module bundler which correctly resolves these modules, generate code which uses a relative reference (and therefore locks the location of the dependency in place) or just creating a module with all components except Popper. The first is what I propose we mention in the docs while the last one might be something we could consider for v5.1 or similar although I do not see a big benefit as bundler should solve this IIRC.

How to use Bootstrap as a module in Angular Project?

I'm confused; isn't this supposed to be the EJS module? Why are there non-EJS things in the EJS-only module? Why has someone/something been compiling non-EJS compliant code into a specifically EJS-compliant CDN link for the past two years?
??? Can someone advise me?

@earcam could you open a PR to add this in docs please?

@XhmikosR, please see #36017 (pipes not yet run)

@GeoSot isn't this fixedx already on main by #36414 and/or 36854?

Wouldn't it be much easier to import popper as a esm module rather than deal with import maps?

i am getting this error
Uncaught TypeError: Failed to resolve module specifier "bootstrap". Relative references must start with either "/", "./", or "../".
this is the error i am getting any one provide the solution how to fix this error?

Here's how I did it:

Download Bootstrap esm: https://cdnjs.cloudflare.com/ajax/libs/bootstrap/5.3.2/js/bootstrap.esm.min.js
npm i esbuild
npm i @popperjs/core
npx esbuild --bundle bootstrap.esm.min.js --format=esm --minify --outfile=bootstrap.min.mjs

You can now import { Tooltip } from './bootstrap.min.mjs'; and it should work as expected in-browser


Extract cross-platform spawn with kill

I'd really love to use this functionality in p-s. Any chance you'd be willing to open source that bit as a separate module (or would you be ok with me doing it)?
Also, I totally love this module. Like a ton. Thank you!

To be clear, these are the files that'd need to be extracted and open sourced:

src/lib/spawn.js
src/lib/spawn-win32.js
src/lib/spawn-posix.js

Thank you for this issue.
Interesting.
I searched cross-platform kill package in npm, kill-tree seems it.
If it add the method to the returned value (ChildProcess instance) of spawn, maybe we can offer it to cross-spawn package.... as killTree() or something like.

I think we should do this. Is that what you mean by:

If the offer is rejected, I'm happy to make the package with you!

? :)

Yes, I agree, https://github.com/IndigoUnited/node-cross-spawn/issues/40 is best.

If the offer is rejected, I'm happy to make the package with you!

I had misunderstood this English sentence ( (or would you be ok with me doing it) ).
Just I thought I might need 2nd plan to make a pakcage like cross-spawn-with-kill.

Gotcha. We'll see what they say!

Looks like the request was denied. cross-spawn-with-kill coming soon :)
I'll let you know when I start working on it. If you beat me to it then that's fine :)

Alrighty, I'm going to try to knock this out!

It's a thing now! Thanks a ton!
https://github.com/kentcdodds/cross-spawn-with-kill

Awesome!


Auto import d.ts files with the same name

For example:
// foo.js
/** @type {import('./foo.d.ts').bar} */
function bar(a,b){
  return a+b
}

// foo.d.ts
export function bar(a:number,b:number):number

It is common practice to obtain type declarations in JavaScript files, but it can be cumbersome.
Expect:
// foo.js

/** @type {bar} */
function bar(a,b){
  return a+b
}

// or

// No need to do anything, just match the same name of variable .
function bar(a,b){
  return a+b
}

/extJS
This should be provided by TS, not by editor itself.
Please check existing TS issue (including closed) as something similar was requested already there.

It looks like this is caused by the TypeScript/JavaScript language service. Please file the issue to the TypeScript repository. Make sure to check their contributing guidelines and provide relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


Ansible 101 Live Stream Episode 10 Agenda - May 27

Previous episode agenda: #269
Video link: https://www.youtube.com/watch?v=iKmY4jEiy_A

[x] SPACE!
[x] Thank you SO MUCH to GitHub Sponsors and Patreon Patrons:

GitHub: @giner, @nord3l, (a private sponsor)

[x]  Questions and notes from last week's episode:

Alexandre Alonso: "which method you use to install kubernetes using ansible?"
Jordan Peterson: "This stream is well beyond the scope of advanced fleet security but I strongly recommend automated key rotation. My company personally uses HashiCorp Vault to rotate keys every 5 minutes."
Captain Ironbutt: "The first step after installing Ansible is to also install Cowsay"
TEJ SINGH RANA: "What is the best way to learn Regexp?"
captainwasabi: "maybe drop the session that is using 22" (PERSISTENT_CONNECT_TIMEOUT)
Aaron Kulbe: "One problem with using /etc/sudoers is that changes may not persist across upgrades. The newer way to do it is /etc/sudoers.d/<username> and that will persist over upgrades."
Aaron Kulbe: "@Jeff Geerling Do you prefer RPM-based distros over DEB-based?"
Dan Quaye: "🍿for these comments. Great conversations happening here"

[x] Ansible Tower and AWX

[x] RHEL and Fedora :: Tower and AWX
[x] How to install (many ways)
[x] Setting up your first Job template

New agenda: #275.


Remove deleted user profile pics from s3 within a month

Prior to this we were only removing the files when a user asked to be forgotten. Patrick, I think this should work, but it's a bit hard to test so I'd appreciate another pair of eyes.

Looks good to me. I tested locally with my own S3 account and icons were indeed deleted


Debugger friendly interfaces to SDK types

@graydon Should I transfer this to the host crate? It would probably be helpful to have Val and other types debugger friendly in the host crate, not just in the SDK, and since the types are defined there it might be best to do this there. Assuming this is even possible at all.


cross tenant migration using the Microsoft Preview failing on test

I am trying to do a cross tenant migration using the Microsoft Preview features and I am testing the setup by running the command Cross Tenant mailbox migrations as follows:
Test-MigrationServerAvailability -Endpoint " ORZCrossTenantMigrationRelationship]”
And I am seeing the error:
The migration endpoint 'ORZCrossTenantMigrationRelationship]' couldn't be found. Reason: The migration endpoint
'ORZCrossTenantMigrationRelationship]' couldn't be found.
+ CategoryInfo          : NotSpecified: (:) [Test-MigrationServerAvailability], ManagementObjectNotFoundException
+ FullyQualifiedErrorId : [Server=SJ0PR10MB5765,RequestId=c3372f77-7d23-4a81-b04e-f83c75379fd8,TimeStamp=6/12/2022
6:13:22 PM] [FailureCategory=Cmdlet-ManagementObjectNotFoundException] A9B37BF0,Microsoft.Exchange.Management.Mig
ration.MigrationService.Endpoint.TestMigrationServerAvailability
+ PSComputerName        : outlook.office365.com
You say in the response to a Reddit query that:
“The documentation is incorrect for Test-MigrationServerAvailability (we are in the process of updating this), and you should be using the target email address, not the source address.”
But I don’t quite know what that means….. as I am meant to be able to simply specify the ENDPOINT in the command and it should find it.
When I set all of this up about a month ago, it worked and responded back with a positive response but now I come to start the migration, it’s not responding and I need to figure out why before continuing.

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

ID: c61c960c-f5c3-76f7-ac18-dfeaf30230e4
Version Independent ID: 3dc607b2-869e-976f-80da-d73cfb1210e5
Content: Cross-tenant mailbox migration - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/cross-tenant-mailbox-migration.md
Product: microsoft-365-enterprise
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@kelleyvice-msft Please help us in resolving this issue. Thanks

@amhiggins1962, thanks for reaching out. This isn't strictly a content issue, and issues with PowerShell modules are outside the scope of content feedback. Having said that, I will update the content to point to the Test-MigrationServerAvailability documentation rather than explicitly suggest a usage that is evidently broken. I will also pass along this issue to the Exchange PowerShell engineering team as additional substantive evidence to support actively resolving the module bug. -Kelley

I also meant to add that you should try one of the other switches in the Test-MigrationServerAvailability documentation as a workaround to your specific problem. -Kelley

Hi Kelly,
Thanks for responding – I ended up getting support directly from someone at Microsoft who explained my error, in this case… the problem was that I was using the name of the migration endpoint rather than the associated appname, which is what should be used……
Thanks for your help and I hope this can be posted to help others who might make the same error as I did…
Regards,
Andy Higgins
[signature_859793916]
++1<PHONE_NUMBER>
www.imcollaboration.comhttps://www.imcollaboration.com/
From: Kelley Vice @.>
Date: Tuesday, June 14, 2022 at 12:59 PM
To: MicrosoftDocs/microsoft-365-docs @.>
Cc: Andy Higgins @.>, Mention @.>
Subject: Re: [MicrosoftDocs/microsoft-365-docs] cross tenant migration using the Microsoft Preview failing on test (Issue #8888)
I also meant to add that you should try one of the other switches in the Test-MigrationServerAvailability documentationhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fpowershell%2Fmodule%2Fexchange%2Ftest-migrationserveravailability%3Fview%3Dexchange-ps&data=05|01|andy%40imcollaboration.com|62a4a47f04ed456044c708da4e2fab31|e975bfedaa7d415aaad92b6898099194|0|0|637908263906894580|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=GALJE65hpM%2BD%2BvvQODRmXdbKVJ%2FCTqATURFDegFHI7M%3D&reserved=0 as a workaround to your specific problem. -Kelley
—
Reply to this email directly, view it on GitHubhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fmicrosoft-365-docs%2Fissues%2F8888%23issuecomment-1155520178&data=05|01|andy%40imcollaboration.com|62a4a47f04ed456044c708da4e2fab31|e975bfedaa7d415aaad92b6898099194|0|0|637908263906894580|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=WiFfb8BCbuMKh9nn2ZtZ9TLa20WURDG7ukkctOhP9JU%3D&reserved=0, or unsubscribehttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKFUHR27EUH3FF2V23KYCOTVPDCANANCNFSM5YSDX2IA&data=05|01|andy%40imcollaboration.com|62a4a47f04ed456044c708da4e2fab31|e975bfedaa7d415aaad92b6898099194|0|0|637908263907050815|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=FyK2YP0tvAZuu%2B%2BFcrg8DGKBaK0TEfuVeUxaKg%2BsNWE%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

Updated in https://github.com/MicrosoftDocs/microsoft-365-docs-pr/pull/14696.

Hi Kelly, Thanks for responding – I ended up getting support directly from someone at Microsoft who explained my error, in this case… the problem was that I was using the name of the migration endpoint rather than the associated appname, which is what should be used…… Thanks for your help and I hope this can be posted to help others who might make the same error as I did… Regards, Andy Higgins

Hello im stuck with same problem when you say "I was using the name of the migration endpoint rather than the associated appname" should the parameter in "Test-MigrationServerAvailability - Endpoint [guid of appid]"  or some other parameter?

Hi @quimguito - apologies for not getting back to you sooner but I got Covid and was ill... better now thanks..
The answer to your question is that you should use the name of the associated appname rather than the migration endpoint name (which is what I assumed they meant by "Endpoint" but that's not what is required)
Let me know if that works for you


[Feature Request]: Persistent recording notification

Feature description
Hello, I was wondering if it would be possible to add a persistent recording notification similar to how NVIDIA shows when you're recording and/or have replay enabled. Possibly a lower profile version as well as well like having a few pixels in the corner showing a small red square.
I also wanted to say thank you very much for this plugin, it's way better than any other OBS notification system that I've seen!
Implementation Ideas
No response

I also tried sending a donation but KoFi and PayPal both says you can't accept payments at the moment.

Good idea!
I was thinking that I needed to somehow display the current state of the recording/replay/stream, but I didn't think about making it another "window" with color indication/icons.
I think that 1-3 squares at any point on any screen will be enough. I can also add icons/styles.

(But there is a problem #9)

I also tried sending a donation but KoFi and PayPal both says you can't accept payments at the moment.

Thank you. I have not received any notifications about the explicit blocking of the PayPal dollar account (and even nothing is written on the site). But this is quite expected...
No more coffee for me 🙃 (I prefer tea)
I'll try to replace it with something. By the way, does DonationAlerts or Boosty work for you?

(But there is a problem https://github.com/DmitriySalnikov/OBSNotifier/issues/9)
I mean windowed fullscreen or having it on a second monitor is better than nothing.
I also was able to send it via DonationAlert I think? Boosty was being garbage as per the 100th time I've used it lol

I also was able to send it via DonationAlert I think?

I think it works! Thank you very much!

Cheers, you deserve it :)


Fixes issue with resetting redis job keys on startup

This addresses https://github.com/flippa/allora/issues/2

Good catch, thanks!

Ah, I can't merge this for you, as I no longer work at Flippa. Will have to wait for somebody else @ags? :)

:+1: Thanks @chenkirk and @d11wtq :)

CHANGELOG updated and 0.1.8 pushed to rubygems.


ERROR: Unable to create powershell session.

Just testing and getting the error:
Calling import-PowerShellSession
ERROR: Unable to create powershell session.
Any recommendation on fix or what to check?

Can you attach the log file?

thanks for the reply,
BVC04.log

Certainly not very exciting.
Let's do this to test and make sure exchange is ready to go.  Make sure you're not running this from the exchange management shell.
Open powershell
$cred = get-credential (use the exchange credential in the same format you specified)
$session = new-pssession -configuration microsoft.exchange -credential $cred -connectionURI https://exchangename/powershell (for example https://webmail.contoso.com/powershell) -authentication Basic -allowRedirection:$TRUE
Import-psSession $session
Does that work.
Tim

Hi Tim, it looks like the issues is due to the -connectionURI using https,
this work
$session = new-pssession -configuration microsoft.exchange -credential $cred -connectionURI http://ex13mb3.xmail.co.uk/powershell -Authentication Kerberos -allowRedirection:$TRUE
but changing to https has an error, is there a switch to use http?

There is not.  Your servers should have self signed certs to the names - and I assume you have at least one server in the environment with a public certificate and a name - like webmail.contoso.com.
Tim
==============================
Timothy J. McMichael
Senior Support Escalation Engineer
<EMAIL_ADDRESS>(980)-776-7465
Hours:  Sunday - Wednesday 08:00 - 16:00 eastern time zone.
Manager:  Tom Roughley<EMAIL_ADDRESS>Premier Support -<PHONE_NUMBER>
Broad Commercial Support -<PHONE_NUMBER>
==============================
From: A-J-A @.>
Sent: Wednesday, March 2, 2022 10:45 AM
To: timmcmic/DLConversionV2 @.>
Cc: Tim McMichael @.>; Comment @.>
Subject: Re: [timmcmic/DLConversionV2] ERROR: Unable to create powershell session. (Issue #39)
Hi Tim, it looks like the issues is due to the -connectionURI using https,
this work
$session = new-pssession -configuration microsoft.exchange -credential $cred -connectionURI http://ex13mb3.xmail.co.uk/powershellhttps://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Fex13mb3.xmail.co.uk%2Fpowershell&data=04|01|timmcmic%40microsoft.com|6f63eb30eddf474247fb08d9fc639cf2|72f988bf86f141af91ab2d7cd011db47|1|0|637818327034902947|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=p3uTRB%2B2uzxKp%2BL6cztLkt%2F2XvE5LoJCoidQdNHERG0%3D&reserved=0 -Authentication Kerberos -allowRedirection:$TRUE
but changing to https has an error, is there a switch to use http?

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftimmcmic%2FDLConversionV2%2Fissues%2F39%23issuecomment-1057075543&data=04|01|timmcmic%40microsoft.com|6f63eb30eddf474247fb08d9fc639cf2|72f988bf86f141af91ab2d7cd011db47|1|0|637818327034902947|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=LTwQQsk%2BO7UHUxn1NrPWBMe3lrFPSg%2BK81EGyc9glO4%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKGTN6NT6RT3PRVOAIFMWGDU56EH3ANCNFSM5PVGRABA&data=04|01|timmcmic%40microsoft.com|6f63eb30eddf474247fb08d9fc639cf2|72f988bf86f141af91ab2d7cd011db47|1|0|637818327034952934|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=MxTFskxicVvYk51VQN6GNaIOZwZMH8QLKBXydm34%2FD8%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|timmcmic%40microsoft.com|6f63eb30eddf474247fb08d9fc639cf2|72f988bf86f141af91ab2d7cd011db47|1|0|637818327034952934|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=cU9hauWOO2TxdhUw664yMVEobDgeXjyfzhTE%2F1XMWM4%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|timmcmic%40microsoft.com|6f63eb30eddf474247fb08d9fc639cf2|72f988bf86f141af91ab2d7cd011db47|1|0|637818327034952934|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=FAcJu%2B%2Bod6c7eeAo6zCNecnXSoC48C0fLW7uvNoYcjo%3D&reserved=0.
You are receiving this because you commented.Message ID<EMAIL_ADDRESS>

thank you for confirming, yes we have self signed and public cert added to Exchange,

If the public cert is added to exchange - and enabled for web services - you should be able to use whatever name would resolve to that.
If the name does not resolve in internal DNS - go ahead and put a host file entry in for it.
Tim
==============================
Timothy J. McMichael
Senior Support Escalation Engineer
<EMAIL_ADDRESS>(980)-776-7465
Hours:  Sunday - Wednesday 08:00 - 16:00 eastern time zone.
Manager:  Tom Roughley<EMAIL_ADDRESS>Premier Support -<PHONE_NUMBER>
Broad Commercial Support -<PHONE_NUMBER>
==============================
From: A-J-A @.>
Sent: Wednesday, March 2, 2022 10:59 AM
To: timmcmic/DLConversionV2 @.>
Cc: Tim McMichael @.>; Comment @.>
Subject: Re: [timmcmic/DLConversionV2] ERROR: Unable to create powershell session. (Issue #39)
thank you for confirming, yes we have self signed and public cert added to Exchange,

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftimmcmic%2FDLConversionV2%2Fissues%2F39%23issuecomment-1057090058&data=04|01|timmcmic%40microsoft.com|c93c425a34ea4192bfe108d9fc659f43|72f988bf86f141af91ab2d7cd011db47|1|0|637818335664496667|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=wfyx1uNvF0DrK9A520kmZOotmrxKU2QkDcgck%2Fd3qcY%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKGTN6KLO7JG6AVM344ECOTU56F5ZANCNFSM5PVGRABA&data=04|01|timmcmic%40microsoft.com|c93c425a34ea4192bfe108d9fc659f43|72f988bf86f141af91ab2d7cd011db47|1|0|637818335664496667|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=aGzE3saJWeNDtH%2FRP3z%2BHFcjXs0H7YrZY8O9QdOkh1A%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|timmcmic%40microsoft.com|c93c425a34ea4192bfe108d9fc659f43|72f988bf86f141af91ab2d7cd011db47|1|0|637818335664546655|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=B%2BiwzB2rk7E9CYZ1txVLJQ14Sgu6JvhkrlUzu4IDtro%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|timmcmic%40microsoft.com|c93c425a34ea4192bfe108d9fc659f43|72f988bf86f141af91ab2d7cd011db47|1|0|637818335664546655|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=Qo6n%2Fs1XPZRjUly0x%2FLtAky%2FwJVLdjw0Vw4OVD%2FGAbU%3D&reserved=0.
You are receiving this because you commented.Message ID<EMAIL_ADDRESS>

Excellent!

From: A-J-A @.>
Sent: Thursday, March 3, 2022 11:58 AM
To: timmcmic/DLConversionV2 @.>
Cc: Tim McMichael @.>; Comment @.>
Subject: Re: [timmcmic/DLConversionV2] ERROR: Unable to create powershell session. (Issue #39)
Hi Tim,
Now working thank you for the assistance, time to start testing 😊
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftimmcmic%2FDLConversionV2%2Fissues%2F39%23issuecomment-1058260890&data=04|01|timmcmic%40microsoft.com|5b741b3ebbcd4711ad1808d9fd3702d5|72f988bf86f141af91ab2d7cd011db47|1|0|637819234996542044|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=qcD4m4GULdUEP2Xcie%2BYXxhc221ULW6KKBKBhQ8ClRo%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKGTN6P53G6RIG4FDHYYGY3U6DVSPANCNFSM5PVGRABA&data=04|01|timmcmic%40microsoft.com|5b741b3ebbcd4711ad1808d9fd3702d5|72f988bf86f141af91ab2d7cd011db47|1|0|637819234996592029|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=AO5uiqRhncTjla3SZSyKomDCa0mW4vCTMIAX4E5r2t4%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|timmcmic%40microsoft.com|5b741b3ebbcd4711ad1808d9fd3702d5|72f988bf86f141af91ab2d7cd011db47|1|0|637819234996592029|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=rsgfPWISTUg5dKbErfYlhYS2EvGJvm3qXMWQVKOoXUU%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|timmcmic%40microsoft.com|5b741b3ebbcd4711ad1808d9fd3702d5|72f988bf86f141af91ab2d7cd011db47|1|0|637819234996592029|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=oG6xSuFCppWTGUXz%2Bw8TX%2Fnog3B9u6uG53qasAsdxvs%3D&reserved=0.
You are receiving this because you commented.Message ID: @.***>


beoran 601 joystick callbacks

merge master from parent repo.
resolve conflicts
change gamepad state callback to use the state.  post empty event after joystick polling to make waitEvents return

The main change I've made here for your implementation was to use the gamepad state struct directly in the callback, rather than forwarding the button and axes states.
The other change I made was to post an empty event if the joystick is polled to make waitEvents work, which was bugging the events test on Mac OS (confirmed, not sure of the downsides of this approach, but it seemed to work for me).
checkout these files for actual changes made

input.c
glfw.h
events.c
window.c

Just noticed this is duplicated


Datepicker for billables filter

'cause at the moment it only works in Chrome

fixed this in #296


Unintuitive error message for missing semicolon

Version
1.10.0
What happened?
I find it quite unintuitive to get this type of output for missing semicolons (;)
For example this input
-- name: GetUserAndPostExample :many
SELECT u.user_id,
       p.post_id
FROM posts as p
         JOIN users as u ON p.post_author_id = u.user_id
WHERE p.post_author_id = ANY (
    SELECT user_id
    FROM users
    ORDER BY users.user_created_at
    LIMIT 1
)

Would be parsed by sqlc and the error I would receive was:
$ db/queries/posts.sql:1:1: syntax error at or near "SELECT"

Given the example above, you can image that in some cases when you have huge queries and a file full of queries, it gets super hard to find the exact causation
Relevant log output
$ db/queries/posts.sql:1:1: syntax error at or near "SELECT"

Database schema
--
-- Name: posts; Type: TABLE; Schema: public; Owner: -
--

CREATE TABLE public.posts (
    post_id uuid NOT NULL,
    post_description character varying(2000) DEFAULT ''::character varying NOT NULL,
    post_title character varying(50) NOT NULL,
    post_price integer NOT NULL,
    post_quantity integer NOT NULL,
    post_targeted_for public.targeted_for NOT NULL,
    post_brand_id uuid NOT NULL,
    post_subcategory_id uuid NOT NULL,
    post_author_id uuid NOT NULL,
    post_created_at timestamp without time zone DEFAULT now() NOT NULL
);

SQL queries
-- name: GetUserAndPostExample :many
SELECT u.user_id,
       p.post_id
FROM posts as p
         JOIN users as u ON p.post_author_id = u.user_id
WHERE p.post_author_id = ANY (
    SELECT user_id
    FROM users
    ORDER BY users.user_created_at
    LIMIT 1
)

Configuration
version: "1"
packages:
  - name: "postgres"
    path: "postgres"
    engine: "postgresql"
    emit_json_tags: false
    emit_exact_table_names: true
    schema: "db/migrations"
    queries: "db/queries"
overrides:
  - go_type: "github.com/gofrs/uuid.UUID"
    db_type: "uuid"
  - go_type: "github.com/gofrs/uuid.NullUUID"
    db_type: "uuid"
    nullable: true

Playground URL
No response
What operating system are you using?
Linux
What database engines are you using?
PostgreSQL
What type of code are you generating?
Go

Thanks to changes in the PostgreSQL parser, we can now give people better error messages.


Allow separate interface ports for multiple Instances

Feature request

I opened a similar issue back in 2020 (#747) about the JSON port, but the issue also applies for all the other interfaces.
As this does not seem to be solved/addressed by #809 I'll give it another try, as I am current working on some new LED projects.
What problem does this feature solve?
Being able to control multiple instances separately by having dedicated ports for them. This would allow using all the companion apps (Android App, any SreenCap, Kodi, ...) to be setup for their specific instance, which is unfortunately still not possible. E.g. using HyperionScreenCap to stream to the second instance only (correct me if there's a way around).
How should this be implemented in your opinion?
Allowing to specify the instance (like in "Remote Control" or "LED Setup" menus) before setting the Port for JSON, PROTO, Flatbuffers.
This could be done in the same dropdown way as in the other menus, which are instance specific.
So this "instance dropdown" could be added to the "Network Services" menu.

Regarding "Token Management" and the API "Network" section: I don't know how independent the separate instances really are and if they allow different settings per instance. If not, they could just stay there with the text saying that they apply for all instances no matter the dropdown setting or the "Network" menu could be split in two subsections: general or API and (instance specific) Port-Settings. What do you think?
Are you willing to work on this yourself?
Unfortunately, I am not familiar with the internals of Hyperion to be able to quickly fix/edit this, but I gladly test some new implementations.
If these hyperion instances really are running quite independant , I hope it shouldn't be much work to allow them to have different ports?
In the end I just want to stream a network grabber to a second instance only (let me know if there's an easy way exept swapping the instances), but I keept the issue more general, so that this could enhance more applications e.g. using the non-official android app to controll instances separatly.
Thanks a lot in advance! Feel free to respond in german if that speeds things up.

, I hope it shouldn't be much work to allow them to have different ports?

Hyperion Instances are to be read as Hyperion LED-Device instances, i.e. you can currently have multiple output instances.
Currently, you can have only one screen-grabber, one USB-grabber, one flat-/probuffer source as input to LED (output) instances.
Only for Bob-light, you could have one port per instance.
Due to this design limitation, you cannot setup a mapping matrix between multiple grabbers/input of the same type (besides Boblight) today.
Your requirement is understood and reasonable, but it is much work, as it would require a redesign of input sources and is not planned currently.

Thanks for the reply! I feared that it's not that separate after all. If you need a Beta-Tester in the future, let me know ;)
Still glad to see Hyperion evolving today since I first started using it in 2014!


Add site contents summary stats

It would be helpful if we could keep summary stats about the samples kept on AmpliconRepository. Namely

how many samples across all public projects on the site?
how many ecDNA in all those public project?

Proposed solution - every time someone creates a project, the site counts number of samples, number of ecDNA, number of BFBs and adds to some variable storing the running total. Home page just displays those variables.

lets discuss how we want this presented as well.  I started roughing this in (just samples and projects) and put it at the bottom right corner

Also need to decide if we want only the public totals or maybe public only for some stats (like you can know how many private projects there are but not how many ecDNAs are in them total)

This is really great. I think we should only report statistics on number of public projects, as that will be conceptually simpler for newcomers to understand (and I do not know if there is any benefit to telling people how many samples there are which they cannot access).
Obviously, we can still keep number of private projects as something reported in the admin view so we have that number on hand for reporting purposes.
I think we should report

number of public projects
number of public samples

then

number of ecDNA
number of BFB
number of other fsCNA (complex non-cyclic + linear)

I think as Forrest pointed out, we should keep a global variable as a running counter in views.py to count number of ecDNA, BFB and other across all samples. Whenever a project is added to the site, it adds the the counter while building.
Whenever the site is restarted, perhaps in the background it can spin up a thread to do that re-counting.

Testing results on 1/12 -

deleting a public project causes uncaught error that crashes site
switching project from private to public does not update stats
uploading a new public project caused site stats to use the project count intstead of sample count for the sample count
should format numbers in tables to have commas (e.g. 1,234 instead of 1234

Fixed bugs with adding/deleting projects, but having issues with getting it work on production. works on dev. Jens will debug further

Added in #219


Support for PHP 8.0

[ ] add support for PHP 8.0 in local env
[ ] add support for PHP 8.0 in CI env
[ ] release new version of the library supporting PHP 8.0

@icewave04 FYI


Add a way to choose the fuse_storage address during runtime

The fuse_storage should not be hardcoded on the config.
I see two possibilities:

[ ] Set a global variable changeable from the client
[ ] Set a per-container storage address

Both will be the best but at least the first one is really important since we need to rebuild the image if the address is wrong.

https://github.com/g8os/core0/commit/e744bfd84044abf0645d21ea20fae56b92909728


adding metainfo to docker image

[x] I have registered the PR changes.

Ⅰ. Describe what this PR did
Adding meta information to docker image,those information will be helpful for locating problems.
docker image inspect registry.cn-hangzhou.aliyuncs.com/truthai/seata-server:latest

Ⅱ. Does this pull request fix one issue?

fixes #5653
Ⅲ. Why don't you add test cases (unit test/integration test)?
I've tested it.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

revert to the docker hub registry.

done


added a block entity and a block entity renderer to the base vanilla …

…minecraft anvil, and added a hammer item to allow for the smithing/flattening of ingots (when the respective sheets/rods or whatever is implemented

I guess I'm going to have to put this in every PR, because ngl, this is getting kinda ridiculous at this point.
The formatting is once again a mess. You just have so many random useless line breaks.
Aside from that the naming is still questionable in places. For example you didn't change the registry name of the block entity. Also why have you appended _BLOCK_ENTITY. Why?
These inconsistencies are a major issue. They may seem like nitpicks, and yes they partially are. But there is a lot of them and it happens in every single PR.
Please if you know a PR is not ready, either don't bother making it, or mark it as a draft.
Other than that. The mixins are a little bit wrong (and maybe you weren't meant to know this).

The MixinHooks class is meant as a place to contain effectively static utility methods that the actual mixins will call. This is useful in multiple ways:

It makes the mixins a lot more concise and therefore easier to read.
It allows much easier debugging of mixins as you can hotswap.
It keeps everything together a little more nicely.

Mixins should not be inside the core package. mixins should always be a package inside the root. It just helps to find mixins a lot easier.

Mixin @Inject methods should be prepended with modid$. So in this case turtychemistry$addIngotToAnvil. This is important as mixin method names must be unique.

I think I may need to change how Intellij reformats the code, because whenever I try to use control + alt+l it ends up adding in those line breaks. It doesn't help that due to the fact that I have a huge side discrepancy between my multiple screens
some line breaks that I think look good on a bigger screen evidently aren't good. Sorry if it's a huge inconvenience for the
formatting issues, I'll get it eventually.

It definitely shouldn't be adding those line breaks. Check your formatting settings.

what formatting settings do you use, because my intellij seems to have some wonk-tastic formatting options for some reason

Default.txt

rename to xml and then you can import

this should be it for this thing that I was doing. Just maybe some formatting help here and there (it still doesn't seem to want to properly format it) and it should be good to go


How to log cookie values in access.log in istio envoy

#22346 , #6695
Hi, Can someone pls share the example for the same. I have set the filter and logs format but in logs, value is still null
Filter yaml
spec:
configPatches:

applyTo: HTTP_FILTER
match:
context: ANY
listener:
filterChain:
filter:
name: envoy.filters.network.http_connection_manager
subFilter:
name: envoy.filters.http.router
patch:
operation: INSERT_BEFORE
value:
name: envoy.filters.http.header_to_metadata
typed_config:
'@type': type.googleapis.com/envoy.extensions.filters.http.header_to_metadata.v3.Config
request_rules:
- cookie: test1_id
on_header_present:
key: test1_id
metadata_namespace: userInfo
type: STRING
- cookie: test2_id
on_header_present:
key: test2_id
metadata_namespace: userInfo
type: STRING

Logs format:
"test1_id": "%DYNAMIC_METADATA(userInfo:test1_id)%",
"test2_id": "%DYNAMIC_METADATA(userInfo:test2_id)%"

it seems that the config you provide has no problem, can you also provide your test request?


Enable or Disable Tasks in your organisations

Section: Enable or Disable Tasks in your organisations
Searching for "Tasks" under Manage app show only 3rd party vendors tasks apps. So currently I still need to search for Planner, to disable/enable it. This is a little confusing.

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

ID: 27c4aab7-6115-ff27-5538-0a0bf73755de
Version Independent ID: 3ff038fc-da4d-fdf4-7b5b-071bd25e3047
Content: Manage the Tasks app for your organization in Microsoft Teams - Microsoft Teams
Content Source: Teams/manage-tasks-app.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@gunterda Thank you for submitting feedback and contributing to the docs. We are currently investigating this.


Control bar does not hide when browser preventsvideo from autoplaying

Describe the bug
The control bar does not close when a video does not autoplay. It works however when a video autoplays.
To Reproduce
Steps to reproduce the behavior:
Example: https://vid.surf/video/T272J (make sure the video does not autoplay)
Expected behavior
The control bar should hide after a few seconds
Screenshots or Links
https://vid.surf/video/T272J (Example, make sure the video does not autoplay)
Desktop (please complete the following information if relevant):

OS: Windows 10
Browser Chrome, Firefox

Affected version
v3.0.0
Additional context
When you click on another video on the site then the player is reloaded and control bar hides as expected

you can add the fade_out class to the bar at
https://github.com/fluid-player/fluid-player/blob/master/src/fluidplayer.js#L641
so that it is always hidden at startup

Thank you for your help!
I made the change in https://vid.surf/fluidhacked.js, however now when a video does not autoplay it won't display the controlbar even while hovering over the video. It is only displayed when the video is paused.
On every subsequent video I click it works flawlessly.
I made a small video demonstrating this issue: https://vid.surf/video/TUFR2

since you don't use fluidplayer anymore, I can't see what happens exactly,
another thing, the autoplay only works when you have already interacted with the page,
otherwise it will not work the first time and anything can happen.
you can mute the video before the "play()" method and see how it behaves.

@biancotv I could not reproduce this, is it still an issue?

@biancotv Is this still and issue? Can I close this? Thank you!