Issue with the callback url from npm package v6

var express = require('express');
var router = express.Router();
require("@shopify/shopify-api/adapters/node")
var shopifyStuff = require("@shopify/shopify-api")
const { shopifyApi, LATEST_API_VERSION } = shopifyStuff
/* GET home page. */
router.get('/', function(req, res, next) {
});
router.get('/auth/callback', async (req, res) => {
// The library will automatically set the appropriate HTTP headers
console.log('hit')
const shopify = shopifyApi({
// The next 4 values are typically read from environment variables for added security
apiKey: "my key",
apiSecretKey: "my key",
scopes: scopes,
hostName: "ff52-147-124-71-250.ngrok.io",
apiVersion: LATEST_API_VERSION,
isEmbeddedApp: false,
});
const callback = await shopify.auth.callback({
rawRequest: req,
rawResponse: res,
});
console.log('callback', callback)
})
router.get('/auth', async (req, res) => {
const shopify = shopifyApi({
// The next 4 values are typically read from environment variables for added security
apiKey: "my key",
apiSecretKey: "my key",
scopes: scopes,
hostName: "ff52-147-124-71-250.ngrok.io",
apiVersion: LATEST_API_VERSION,
isEmbeddedApp: false,
});
await shopify.auth.begin({
shop: shopify.utils.sanitizeShop(req.query.shop, true),
callbackPath: '/auth/callback',
isOnline: false,
rawRequest: req,
rawResponse: res,
});
});
module.exports = router;
const scopes = [
"read_analytics",
"write_assigned_fulfillment_orders",
];
Hello this is my code that i running with the help of ngrok in order to get access to the admin api with v6.
It is returning me
Oauth error invalid_request: The redirect_uri and application url must have matching hosts. I think something is wrong with the oauth2.0 encoding in the package itself please let me know thank you

@Cavmpt it seems that you have not used the Shopify CLI to generate the skeleton of your app. Obvisouly it is not mandatory to do so, but you might want to have a look at this turorial and generate a node app. It should give you an idea of how we configure the use for shopifyApi, the setup for the callback for the auth process. You'll also see how we handle the tunnelling. The CLI takes care of most of it for you but you can customize the skeleton generated for you to your liking.


web socket error on the ui

Getting this error on the ui
ember.prod.js:497 WebSocket connection to 'ws://server.rancher.3b180629.svc.dockerapp.io/v2-beta/projects/1a5/subscribe?eventNames=resource.change&eventNames=service.kubernetes.change&limit=-1&include=hosts&include=services&include=instances&include=instance&include=instanceLinks&include=ipAddresses&sockId=22' failed: Error during WebSocket handshake: Unexpected response code: 200

Useful
Info

Versions
Rancher v1.2.0-pre3 Cattle: v0.169.5 UI: v1.2.11

Access
github admin

Orchestration
Cattle

Route
stacks.index

@mwaaas Is your rancher server behind a firewall or other networking appliance? Typically you see this when said appliance is preventing upgrading connections from http to websocket.

@aemneina I don't think so, I have just run the following docker-compose command:
db:
environment:
- MYSQL_DATABASE=cattle
- 'MYSQL_PASSWORD=9118z29I6&746C6'
- 'MYSQL_ROOT_PASSWORD=9118z29I6&746C6'
- MYSQL_USER=cattle
image: 'mysql:5.5'
server:
environment:
- CATTLE_DB_CATTLE_MYSQL_HOST=db
- CATTLE_DB_CATTLE_MYSQL_NAME=cattle
- CATTLE_DB_CATTLE_MYSQL_PORT=3306
- 'CATTLE_DB_CATTLE_PASSWORD=9118z2*9I6&746C6'
- CATTLE_DB_CATTLE_USERNAME=cattle
image: 'rancher/server:v1.2.0-pre3'
links:
- db
ports:
- '80:8080'
volumes:
- '/var/run/docker.sock:/var/run/docker.sock'

what web browser were you using when you noticed this?

@aemneina chrome

@mwaaas Can you try again with the rancher/server:v1.2.0 release?

@nut-abctech A 200 status means the Connection and/or Upgrade headers aren't making it to the backend, so it just returns OK when sent an otherwise normal looking GET request instead of 101 Switching Protocols.  Your config does not match our current suggested nginx config, it looks like a variant of a very old one (because x-api-request-url is present).


CodeCop Rule AA0218 - Should not warn in RoleCenter, HeadlinePart, Media CardPart etc.

Describe the bug
It is unnecessary to warn about missing tooltip for fields and actions where the tooltip is not shown in the client.
These are the places I have come across. There can be other places as well.

RoleCenter page - area Embedding - Actions
RoleCenter page - area Sections - Actions
HeadlinePart page - Fields
NavigatePage (wizard) - Actions where InFooterBar = true (Back, Next, Finish)
Media Field in CardPard - Media can only be shown in the client via a CardPart page. The tooltip for a media field is not visible in the client.

To Reproduce
Create pages, fields, and actions as described above without the tooltip property.
Expected behavior
I expect not to be warned about missing tooltip when the tooltip is unnecessary because it is not shown in the client.
Screenshots

5. Versions:

AL Language: 5.0.254558

Business Central:

@MarcHansenMicrosoft can you take a look at this issue?

fixed in master


README.md - complete path to tslint.json for „extends“

According to my tests with --print-config option of tslint the linter config ist not correctly extended unless full path to tslint.json is given.

@ml-eds you have to update the version number in package.json


I can't start dreamview

computer: intel core i3
os: ubuntu 16.04   64 bit
have installed nvidia driver
but can't start dreamview
log as follows:
[.....monitor_manager.cc:52] No HMIStatus was received
that's why???

@wanggujin could you post your terminal error messages here?
Also, which version of Apollo are you working on?

@wanggujin could you post your terminal error messages here?
Also, which version of Apollo are you working on?

apollo 3.5

@wanggujin we would need a screenshot of your error messages. Thanks!

and what do you mean by cannot start dreamview? Did the output of './script/bootstrap.sh' tell you there is a core dump, or the webpage is blank when you visit 'localhost:8888'?

@unacao excuse me, when I visit 'localhost:8888' the webpage is blank, what should I do then?thanks!

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


Throttle for max request limit.

I am using the https://pub.dev/packages/geocoding geocoding package. I have a list of coordinates but it says throttle for max requests. How can i prevent the limit reach error. I like this package but i can't.

The Apple documentation states that you should only send one request per minute. So I used this limit for both platforms. You can use a Timer or you work with timestamps. It depends on your case. I am using it in an applicationloop which starts, stops, restarts etc. So in my case I used the timestamp option.
Example:
var lastTimestamp = 0;
static const durationInterval = 60000; // in ms
some Loop {
.....
// check if adress is older than 60 seconds
if (getTimeDifference(lastTimestamp, DateTime.now().millisecondsSinceEpoch)) {
getAddress(lastPosition);
}
......
}
bool getTimeDifference(int oldTime, int newTime) {
if (newTime - oldTime > durationInterval) {
lastTimestamp = newTime;
return true;
}
return false;
}
Future getAddress(GPSLocation? position) async {
if (position != null) {
try {
List placemarks = await placemarkFromCoordinates(position.lat, position.lon);
Placemark place = placemarks[0];
setAddress(place.country, place.locality, place.postalCode, place.street);
......
But if you need the List instant converted, you should consider making an API-key and talk to the API directly. Check pricing before!

Hi @anilenepal! Thank you for reaching out with the issue you are experiencing.
As mentioned by @Iron-Mike-Tyson, geocoding functionality is rate-limited for both Android and iOS.
If you need to resolve many locations at once, you cannot make use of the free iOS and Android geocoding functionality. You should instead rely on a different API that is likely subject to service fees.


Change how webview is resized

Chrome 43 has changed how webview's WebContents is resized according to the webview element in embedder, this PR updates our webview implementation for that.
Fixes #1804.

Awesome :+1:


Small Documentation Change

Great work on this -- short and saves me a few files.
Looks like the "logging" property wrapper isn't required if you use fromJson instead of fromFile (in fact it causes problems).  It is clear from the source and tests, but could update the documentation to make this clear? (The readme implies you always need "logging").
Let me know if you'd like me to make the change and submit a pull request.

Sorry for the really really late reply, but I definitly prefer "solutions" meaning pull requests instead of an issue only ;-)


.buildkite: Fix CI pipeline race condition

It seems that running multiple Buildkite agents in parallel (after https://github.com/oasisprotocol/oasis-core/pull/5258) occasionally triggers a race condition if two Buildkite agents simultaneously attempt to write the same file to the local cache:
+mv -f /tmp/artifacts_tmp/oasis-node.test /tmp/artifacts/oasis-node.test
mv: cannot create regular file '/tmp/artifacts/oasis-node.test': File exists

This PR simplifies the Buildkite artifact downloading logic and prevents this race condition from causing a failed CI job.

The name of the changelog file 5282.trivial.md for #5283 is incorrect :)


SPI Interface Error

Hi,
I'm having the exact same error "SPI Interface Error" as https://github.com/ArduCAM/Arduino/issues/141
Please find my circuit and code setup:

Kindly help.

Thank you @BDCowboy, I shall try that and get back to you asap.

@prasuchit
Please refer to this schematic diagram

You should connect the SPI signal to here .

@UCTRONICS , thanks for the quick response, however, is this the schematic of the arducam board or arduino, I'm confused. The ICSP pin seems to be among those 6 pins at the middle of the arduino mega board, but that wasn't connected in this picture: http://www.arducam.com/wp-content/uploads/2017/12/ArduinoConnection-.jpg

@UCTRONICS , also I found this link about arduino mega connections, but it is very incoherent, it doesn't mention which sketch it is talking about, doesn't talk about the hardware clearly, and is hard to follow in general: http://www.arducam.com/how-to-connect-arducam-shield-to-mega-2560/
Can you please explain the circuit for arduino mega and any changes in the program? I'm using Arducam_mini_lowpowermode example code from the library.

Ok, I figured it out, for anyone who is having this trouble in the future:
(This was done with Arduino Mega, but I guess the principle is the same for all boards)
Connect Vcc and Ground of the arducam board to the +5V and Ground on arduino.  The CS pin from the arducam board to any digital pin on arduino and look for "const int SPI_CS = 49;" in the code that you are running and change that to that pin number. The rest of the pins, refer to the image:

This worked for me after searching for a really long time. I don't know why arducam wouldn't have a diagram for mega. This worked for me with the 2MP Plus using MEGA

Thanks for your great suggestions. We will update our diagram for mega as soon as possible.

Hello,
Arducam has update document and add diagram for mega2560+ Arducam MINI Camera
https://www.arducam.com/docs/spi-cameras-for-arduino/hardware-connection-for-mega2560/

how do i change the clock speed when using the arduino uno r3 and arducam mini 5mp plus


fix: 6759 add oidc issuer alias

Tips:

Maybe add you organization to USERS.md.
check.
Your PR needs to pass the required checks before it can be approved. If the check is not required (e.g. E2E tests) it does not need to pass
ran e2e tests.
Sign-off your commits to pass the DCO check: git commit --signoff.
check.
Run make pre-commit -B to fix codegen or lint problems.
check.
Say how how you tested your changes. If you changed the UI, attach screenshots.
Tested using dex on the local k3d cluster.
make start PROFILE=sso
Notice the original dex issuer is set up as http://dex:5556/dex.
Update this configmap to configure dex with a different issuer, http://mydex:5556/dex
Without the code change, this should display an error when logging in with SSO, such as
oidc: id token issued by a different provider, expected %q got %q"
With the code change:
Configure the workflow-controller-configmap with the new IssuerAlias field, to match the new issuer above ( http://mydex:5556/dex)
set /etc/hosts to have
<IP_ADDRESS> dex mydex (so that it resolves)
Observe that SSO succeeds, because the code uses the alias as an alternate issuer URL.
If changes were requested, and you've made them, then dismis the review to get it looked at again.
You can ask for help!

@alexec  can you review. this PR?

needed to sync my go.mod with upstream and ran go mod tidy
should be fixed now.


Support Server-Side Client Authentication with custom Cert Validation Callback

This PR adds Server-Side logic for TLS Client Authentication (also called Mutual Authentication) to s2n using a flexible Cert Validation callback. It partially addresses Issue https://github.com/awslabs/s2n/issues/184
Specifically the Cert Validation callback API is int verify_cert_chain(struct s2n_blob *cert_chain, struct s2n_cert_public_key *public_key, void *context) and can be set for verifying either the Server's Cert or the Client's Cert. This callback is meant to provide a base for future Certificate Validation additions to s2n (Eg with WATCHList) or for consumers of s2n to specify custom Certificate Validation logic.
To enable Client Auth, consumers of s2n need only to make 2 calls:

Tell s2n server to request a certificate from the client:
s2n_connection_set_client_cert_auth_type(server_conn, S2N_CERT_AUTH_REQUIRED);
Provide a pointer to a function that will validate the Client's Certificate and copy the Certificates Public Key into a s2n_cert_public_key struct:
s2n_connection_set_client_cert_verify_callback(server_conn, &cert_verification_func, &context);

Changes that are not included in this PR:

Full support for s2n to be the Client in a Mutually Authenticated Handshake. This PR is getting pretty big, I'll include that functionality in a following PR. This mainly consists of implementing s2n_client_cert_verify_send(struct s2n_connection *conn) and adding Integration tests with OpenSSL to test it.

The following changes to existing code were also made to aid with the Mutual Auth Changes:

Existing Unit Tests are migrated to use the callback and explicitly provide a dummy accept_all_rsa_certs function rather than have the s2n_client silently perform this functionality as it was previously
The Connection Handshake Hash updates were changed to take place after calling the state handler for the current state in the state machine. This is because the cert_verify state needed to access the hash of the TLS Handshake up to (but not including) the cert_verify Handshake Message.
-m option has been added to the s2nd server daemon to enable Mutual Authentication
The Signature and Hash Algorithm Parsing in s2n_client_extensions.c  has been broken up and refactored into smaller functions since the Mutual Auth code needed to use a subset of the same functionality from the Client Extension Code.
SHA 224 and SHA 512 Hashes of the TLS Handshake are now also kept track of since they may be negotiated by a client and used in the Certificate Verify State.

Besides the changes necessary for Client Authentication there are also a few clean up changes:

Whitespace was added to static struct s2n_handshake_action state_machine[] to form columns that are easier to read
static message_type_t handshakes list of State Handlers was reformatted so that every time the Handshake IO direction changes between Client and Server, a new line is inserted for better readability.
PEM formated values in source code that were not properly split onto multiple lines are split into multiple lines
Several Enums that listed many variants in a single line were split onto one per line for better readability
Typo in s2n.mk  for Fuzzing Debug CFLAGS was fixed so that if a Fuzz test were to fail, it would include a readable stack trace as was the intention.
Bugfix in s2n_config_set_protocol_preferences that incorrectly attempted to return an S2N_ERROR was fixed. This bug is only able to be reached at Server start up time if a server attempted to configure an application_protocol that was longer than 255 characters or the total length of all application protocols was longer than 64k.

I think this PR has become too large to review. What do you think about incorporating the comments and opening new, smaller PRs for the different portions?

Going to close this PR in favor of multiple smaller ones


Switch rospy_message_converter to iron branch

Ooops. Now there is an "iron" branch! I've closed and reopened the PR to trigger the CI again.


Cannot run AWS FreeRTOS MQTT demo on ESP32-DevkitC : nothing is displaying on Serial Monitor after " "I (927) WIFI: SYSTEM_EVENT_STA_START""

Describe the bug
I am trying to run the MQTT demo on ESP32. I followed this tutorial : https://docs.aws.amazon.com/freertos/latest/userguide/getting_started_espressif.html
Its configuration is done and build files are successfully generated. But when i run this command to flash and run FreeRTOS : " ./vendors/espressif/esp-idf/tools/idf.py flash monitor -p /dev/ttyUSB0" on serial monitor it shows "Starting Demo" and gets stuck at
"I (927) WIFI: SYSTEM_EVENT_STA_START"
and there is no output after that, nothing is displayed on the monitor after that..not even an error. I cannot figure out what kind of error it could be.
There's is another error :  E (297) PKCS11: failed nvs get file size 4354 0
System information

Hardware : ESP32- DevkitC
Operating System Linux-Ubuntu
Version of FreeRTOS : 202002.00

Console Output
ets Jun  8 2016 00:22:57
rst:0x1 (POWERON_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0018,len:4
load:0x3fff001c,len:7216
ho 0 tail 12 room 4
load:0x40078000,len:12620
load:0x40080400,len:6708
entry 0x40080778
I (75) boot: Chip Revision: 1
I (76) boot_comm: chip revision: 1, min. bootloader chip revision: 0
I (41) boot: ESP-IDF v3.3-163-g601a03e 2nd stage bootloader
I (41) boot: compile time 00:33:37
I (41) boot: Enabling RNG early entropy source...
I (46) boot: SPI Speed      : 40MHz
I (51) boot: SPI Mode       : DIO
I (55) boot: SPI Flash Size : 4MB
I (59) boot: Partition Table:
I (62) boot: ## Label            Usage          Type ST Offset   Length
I (69) boot:  0 nvs              WiFi data        01 02 00010000 00006000
I (77) boot:  1 otadata          OTA data         01 00 00016000 00002000
I (84) boot:  2 phy_init         RF data          01 01 00018000 00001000
I (92) boot:  3 ota_0            OTA app          00 10 00020000 00177000
I (99) boot:  4 ota_1            OTA app          00 11 001a0000 00177000
I (107) boot:  5 storage          WiFi data        01 02 00317000 00010000
I (114) boot: End of partition table
I (119) boot: ota rollback check done
I (123) boot_comm: chip revision: 1, min. application chip revision: 0
I (130) esp_image: segment 0: paddr=0x00020020 vaddr=0x3f400020 size=0x23504 (144644) map
I (190) esp_image: segment 1: paddr=0x0004352c vaddr=0x3ffbdb60 size=0x030e0 ( 12512) load
I (195) esp_image: segment 2: paddr=0x00046614 vaddr=0x40080000 size=0x00400 (  1024) load
0x40080000: _WindowOverflow4 at /home/maliha/RGS/freertos-1592506346911/FreeRTOS/freertos_kernel/portable/ThirdParty/GCC/Xtensa_ESP32/xtensa_vectors.S:1713
I (197) esp_image: segment 3: paddr=0x00046a1c vaddr=0x40080400 size=0x095f4 ( 38388) load
I (222) esp_image: segment 4: paddr=0x00050018 vaddr=0x400d0018 size=0xa70d0 (684240) map
0x400d0018: _flash_cache_start at ??:?
I (462) esp_image: segment 5: paddr=0x000f70f0 vaddr=0x400899f4 size=0x0ce70 ( 52848) load
0x400899f4: r_rwble_isr at ??:?
I (497) boot: Loaded app from partition at offset 0x20000
I (497) boot: Disabling RNG early entropy source...
I (498) cpu_start: Pro cpu up.
I (501) cpu_start: Application information:
I (506) cpu_start: Project name:     esp-idf
I (511) cpu_start: App version:      1
I (516) cpu_start: Compile time:     Jun 29 2020 00:29:05
I (522) cpu_start: ELF file SHA256:  32b3a7452d4b58e6...
I (528) cpu_start: ESP-IDF:          v3.3-163-g601a03e
I (534) cpu_start: Single core mode
I (538) heap_init: Initializing. RAM available for dynamic allocation:
I (545) heap_init: At 3FFAFF10 len 000000F0 (0 KiB): DRAM
I (551) heap_init: At 3FFB6388 len 00001C78 (7 KiB): DRAM
I (557) heap_init: At 3FFB9A20 len 00004108 (16 KiB): DRAM
I (563) heap_init: At 3FFBDB5C len 00000004 (0 KiB): DRAM
I (569) heap_init: At 3FFCCC68 len 00013398 (76 KiB): DRAM
I (575) heap_init: At 3FFE0440 len 0001FBC0 (126 KiB): D/IRAM
I (582) heap_init: At 40078000 len 00008000 (32 KiB): IRAM
I (588) heap_init: At 40096864 len 0000979C (37 KiB): IRAM
I (594) cpu_start: Pro cpu start user code
I (277) cpu_start: Starting scheduler on PRO CPU.
I (287) uart: queue free spaces: 10
0 1 [main] Initializing FreeRTOS TCP stack
I (297) PKCS11: Initializing NVS partition: "storage"
E (297) PKCS11: failed nvs get file size 4354 0
1 1 [main] Write certificate...
2 18 [iot_thread] [INFO ][DEMO][180] ---------STARTING DEMO---------
I (467) BTDM_INIT: BT controller compile version [9891bc6]
3 18 [iot_thread] [INFO ][INIT][180] SDK successfully initialized.
I (477) system_api: Base MAC address is not set, read default base MAC address from BLK0 of EFUSE
I (567) phy: phy_version: 4102, 2fa7a43, Jul 15 2019, 13:06:06, 0, 0
GAP procedure initiated: stop advertising.
GAP procedure initiated: advertise; disc_mode=2 adv_channel_map=0 own_addr_type=0 adv_filter_policy=0 adv_itvl_min=480 adv_itvl_max=960
I (867) wifi: wifi driver task: 3ffd8fb0, prio:23, stack:3584, core=0
I (867) system_api: Base MAC address is not set, read default base MAC address from BLK0 of EFUSE
I (867) system_api: Base MAC address is not set, read default base MAC address from BLK0 of EFUSE
I (887) wifi: wifi firmware version: c94f3e6
I (887) wifi: config NVS flash: enabled
I (887) wifi: config nano formating: disabled
I (887) wifi: Init dynamic tx buffer num: 32
I (897) wifi: Init data frame dynamic rx buffer num: 32
I (897) wifi: Init management frame dynamic rx buffer num: 32
I (907) wifi: Init management short buffer num: 32
I (907) wifi: Init static rx buffer size: 1600
I (907) wifi: Init static rx buffer num: 10
I (917) wifi: Init dynamic rx buffer num: 32
I (927) wifi: mode : sta (24:0a:c4:c6:39:a0)
I (927) WIFI: SYSTEM_EVENT_STA_START
Its my first time using AWS FreeRTOS...Can anyone help?
Thank you!

Would it be possible to step the code and see where it got stuck?
(May need to refer to the debug setup in the getting started guide you've linked.)

@Maliha03, This happens when WiFi credentials are not configured.
To fix the issue, you need to configure aws_clientcredential.h and aws_clientcredential_keys.h.

Hello @Maliha03,
I am going to close this issue as there has been no response for some time. Please let us know if we can assist you further by re-opening this issue.


Adds OSS Search Functionality to VS Code Extension

Package.json
a. Added an activation event
b. Added a command and title
extension.ts
a. Added async function and changed it to include Promise since there was an error without it
b. Registered the command under the async function. I have an error that it needs a void operator

Ah thanks for reviewing TJ! I think he meant to send this as a draft PR as
we were planning to go over this and review it live in our next pair
programming session :) I’ll make sure we fix your comments!
--
Follow me on twitter @slimsag https://twitter.com/slimsag.

Whoops, apologies TJ! This is my first real PR, so I'm still learning the ropes. I'll coordinate with Stephen and find the right solution Wednesday during our 1:1

Updated: We followed up on all the feedback here and sent+merged a new PR #200


feat(ngMock.$componentController): add helper to instantiate controllers for components

Closes #13683

Wouldn't it be nice to have that for plain directives, too?

There will be some slight inconsistency between using controller: 'SomeCtrl as identifier' and controller: 'SomeCtrl' or controller: SomeCtrl. In the former case, the controller will be added as an alias on the scope, whereas in other two cases, it won't (when usign $componentController, I mean). Likewise, failing to provide a $scope in locals will throw an error in the former case (because it will try to assign the controller instance on the scope), but not in the other two cases.

@Narretz - it is much more complicated to do it for plain directives because we don't actually call the directive factory function until the directive is matched in some HTML. This means that the controller is not easily available. Components are simpler because the controller is not wrapped in a factory function.

There will be some slight inconsistency between using controller: 'SomeCtrl as identifier' and controller: 'SomeCtrl' or controller: SomeCtrl. In the former case, the controller will be added as an alias on the scope, whereas in other two cases, it won't (when usign $componentController, I mean). Likewise, failing to provide a $scope in locals will throw an error in the former case (because it will try to assign the controller instance on the scope), but not in the other two cases.

@gkalpak this is also true of ngMock.$controller right now.

@petebacondarwin, true, but things are more explicit there. You are are the one passing the argument to $controller() so the outcome should be less surprizing. In the case of $componentController() this happens indirectly; I would expect the same outcome when instantiating a component controller.
I wouldn't go to great lengths trying to "fix" this. If there's no easy fix (e.g. extracting the controller name and ignoring the alias or something), I would at least document that when using controller: 'SomeCtrl as identifier', you need to supply a $scope in locals.
No biggie anyway - it only affects test code. Feel free to leave as is, if you think it's OK.
LGTM otherwise :+1:

@gkalpak - OK, I see what you mean. But since this is for people testing controllers that (hopefully) they have actually written, then it shouldn't be too surprising!

Actually in light of the latest discussions about controller as we should probably provide the ident param, if it is not explicitly provided, to be in line with the values of the component definition. I.E. if none is given the controller is attached to the scope via $scope.$ctrl.

Now it LRGTM ('R' for "really") :smiley:


small typo in resolve_udp ext_socket.connect ?

Hi Frank, maybe a small typo ?

That shouldn't make any difference, but why not !

Indeed, I'm learning Rust, thank you.
Do you think having addition upstream_addr for cases with split-dns and clients with vpn would be useful ? When vpn is off client comes with public IP, and upstream_addr resolver gives public IPs, when vpn goes on then client transparently switches to private IP, and this is where it gets different upstream_addr pointing to another split-dns resolver. I added a new configuration param "upstream_addr_private" in a branch here: https://github.com/paukstis/encrypted-dns-server/tree/upstream_private , would love to hear your opinion, thanks!


[HUDI-4093] fix NPE when insert records that partition column is null…

… into partition hudi table
Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

high level question/thoughts.
is this about partition column value being null or partition path field config value set to null?
also, can we ensure we look at all write paths? i.e. spark-sql, spark-datasource as well and ensure we are consistent.

@watermelon12138 : can you respond to my question above.

high level question/thoughts. is this about partition column value being null or partition path field config value set to null? also, can we ensure we look at all write paths? i.e. spark-sql, spark-datasource as well and ensure we are consistent.

@nsivabalan I am so sorry，I didn't respond in time. This problem is about partition column value being null. The solution is that when the value of the partition column is null, the default partition is created.

@watermelon12138 I suggest you to try out latest master as #7349 has been merged. It should fix the NPE.

@watermelon12138 : If this is not an issue in current master, can you kindly close it ?


Graph detection

Feature branch for graph detection
Related work:
https://www.electricmonk.nl/docs/dependency_resolving_algorithm/dependency_resolving_algorithm.html
https://en.wikipedia.org/wiki/Topological_sorting
https://www.embeddedrelated.com/showarticle/799.php

I've introduced some code to detect circular deps. The next question is how I integrate this with Brap in a clean way.

@onebit0fme The code is now able to detect cycles. I can ship brap2.0 when I'm happy with an API for developers to play with.
In BrapBrap I'm expecting to do something such as:
src
├── classes
├── injection
└── framework

(not the actual folder structure, just the idea-space of where things are isolated)
Injection might look like:
    from brap import Container

    from classes import ERBFileParser, TemplateEngine

    container = Container()
    container.set('file_parser', ERBFileParser)
    container.set('template_engine', TemplateEngine, ['file_parser'])

And the framework might do this:
from brapbrap import App
from injection import container

container.verify_no_undefined_values()
container.verify_no_circular_dependencies()

app = App(container)


Is this library available?

> Could not find com.github.Qusion:android-pin-dot-view:0.3.0.
     Searched in the following locations:
       - https://jcenter.bintray.com/com/github/Qusion/android-pin-dot-view/0.3.0/android-pin-dot-view-0.3.0.pom
       - https://dl.google.com/dl/android/maven2/com/github/Qusion/android-pin-dot-view/0.3.0/android-pin-dot-view-0.3.0.pom

Hello Ihalegria!
I'm so sorry for the late reply.
Unfortunately, I've mismatched the library names in readme files. Thanks for reaching out, it's all fixed now.
Hope you'll like it!


MAX aggregation

Closes #4

@ajnavarro @erizocosmico Now NULL values are handled in the MAX node although when the #33 will be implemented, this logic could be useless.

Rebase needed


Add SmartFinder Latest

A GUI tools for managing content of Smartisan OS phone.

Build failure is unrelated (#13783).
That said, the cask should be called smartfinder, not smart-finder, pursuant to the token reference.

Thank you for the contribution. It needed some fixes, so they were made in b435164f1bbaac8f83afa0a480b9a58a1247a123. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

@jawshooah Thanks, I will pay attention to travis build result next time.


Failed to install generator-ng2-seed-advanced

For installing generator-ng2-seed-advanced, additiona gulp packages are requied:
npm install -g gulp gulp-cli

Suggest to add above to installation steps, otherwise the installation will fail. Thanks!

After installing gulp dependencies, I also got following errors. Are there any steps missing? Thank you.
[10:32:00] Failed to load external module ts-node/register
[10:32:00] Failed to load external module typescript-node/register
[10:32:00] Failed to load external module typescript-register
[10:32:00] Failed to load external module typescript-require
/Users/lyon/.nvm/versions/node/v6.9.4/lib/node_modules/generator-ng2-seed-advanced/node_modules/angular2-seed-advanced/gulpfile.ts:1
(function (exports, require, module, __filename, __dirname) { import * as gulp from 'gulp';
                                                              ^^^^^^
SyntaxError: Unexpected token import
    at Object.exports.runInThisContext (vm.js:76:16)
    at Module._compile (module.js:542:28)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Liftoff.handleArguments (/Users/lyon/.nvm/versions/node/v6.9.4/lib/node_modules/gulp/bin/gulp.js:116:3)
    at Liftoff.<anonymous> (/Users/lyon/.nvm/versions/node/v6.9.4/lib/node_modules/gulp/node_modules/liftoff/index.js:198:16)
/Users/lyon/.nvm/versions/node/v6.9.4/lib
└── (empty)

npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm ERR! Darwin 16.4.0
npm ERR! argv "/Users/lyon/.nvm/versions/node/v6.9.4/bin/node" "/Users/lyon/.nvm/versions/node/v6.9.4/bin/npm" "install" "-g" "generator-ng2-seed-advanced"
npm ERR! node v6.9.4
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>postinstall: `typings install && gulp check.versions && npm prune && gulp webdriver && node tools/install.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'typings install && gulp check.versions && npm prune && gulp webdriver && node tools/install.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the angular2-seed-advanced package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     typings install && gulp check.versions && npm prune && gulp webdriver && node tools/install.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs angular2-seed-advanced
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls angular2-seed-advanced
npm ERR! There is likely additional logging output above.

thanks the documentation will be updated with this

fixed

Hi, I still see this error when I try to install (even after installing all the global dependencies):
(function (exports, require, module, __filename, __dirname) { import * as gulp from 'gulp';
^^^^^^
SyntaxError: Unexpected token import
at createScript (vm.js:80:10)
at Object.runInThisContext (vm.js:139:10)
at Module._compile (module.js:588:28)
at Object.Module._extensions..js (module.js:635:10)
at Module.load (module.js:545:32)
at tryModuleLoad (module.js:508:12)
at Function.Module._load (module.js:500:3)
at Module.require (module.js:568:17)
at require (internal/module.js:11:18)
.....
.....
Am I missing anything?


add council pallet back

Codecov Report

Merging #199 (5b9b944) into master (3decbe7) will increase coverage by 0.18%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #199      +/-   ##
==========================================
+ Coverage   78.85%   79.04%   +0.18%
==========================================
  Files          60       60
  Lines       17151    17530     +379
==========================================
+ Hits        13524    13856     +332
- Misses       3627     3674      +47

Impacted Files
Coverage Δ

runtime/src/lib.rs
21.99% <33.33%> (ø)

testing/src/genesis.rs
100.00% <100.00%> (ø)

pallets/credit/src/benchmarking.rs
30.98% <0.00%> (-22.68%)
:arrow_down:

pallets/credit/src/mock.rs
96.88% <0.00%> (-3.12%)
:arrow_down:

pallets/credit/src/lib.rs
71.61% <0.00%> (-0.23%)
:arrow_down:

pallets/tips/src/tests.rs
98.83% <0.00%> (ø)

pallets/credit/src/tests.rs
100.00% <0.00%> (ø)

pallets/staking/src/mock.rs
94.60% <0.00%> (ø)

pallets/credit/src/weights.rs
0.00% <0.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

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


Self-banning feature to prevent lurking

This adds the self-banning feature. It's main purpose is to make sure that one stops lurking on the server if needed (e. g. for work).
A user can use .selfban with an optional duration in hours as an argument.
If no argument is given, the ban can only be lifted by moderaders (inifite ban).
Selfbans can't be unlifted by the user to avoid that the user can make himself lurk again.
This implementation takes care of the following details:

Moderaders can still override selfbans, effectively converting the selfban to a normal ban
Users can not troll moderadors by selfbanning themselves immediately after they did shit on the server
Loading and unloading is possible due to using UTC timestamps
There's a caveat though: bans will only be lifted/checked every minute due to simpler implementation
@NullDev can never ban himself for security reasons
It added ids.banned_channel_id which will also being used in the private message sent to the user

@NullDev is responsible for implementing the loading/saving of exports.bans in ban.js.
Please fill the stubs exports.saveBans and exports.loadBans accordingly in ban.js.
Please don't forget to add the mentioned config entry (see config.template.js).

Fixed ESLint problems, you can recheck now.

Fixed a logic bug, oops.

Fixed a logic bug, oops.

lmao


unison: update from 2.40.102 to 2.48.3

Not sure if this breaks backwards compatibility somehow.
cc @viric

manually merged here

Thanks!


Proguard

I am trying to use proguard but I am getting error with the ormlite
09-16 19:31:27.477 13462 13462 E AndroidRuntime: java.lang.RuntimeException: Unable to create application br.com.calculadora.v2.CalculadoraApplication: java.lang.IllegalArgumentException: Foreign field class br.com.calculadora.v2.e.a does not have id field
I am using these two dependencies:
    compile 'com.j256.ormlite:ormlite-core:4.48'
    compile 'com.j256.ormlite:ormlite-android:4.48'

and my proguard rules has ( I looked the issue #44 ):
-keepattributes *DatabaseField*
-keepattributes *DatabaseTable*
-keepattributes *SerializedName*
-keep class com.j256.**
-keepclassmembers class com.j256.** { *; }
-keep enum com.j256.**
-keepclassmembers enum com.j256.** { *; }
-keep interface com.j256.**
-keepclassmembers interface com.j256.** { *; }
-dontwarn com.j256.ormlite.android.**
-dontwarn com.j256.ormlite.logger.**
-dontwarn com.j256.ormlite.misc.**

Do you guys have any idea?

have you tried
-keepclassmembers class <your model package>.** {
	@com.j256.ormlite.field.DatabaseField <fields>;
	@com.j256.ormlite.field.ForeignCollectionField <fields>;
}


build: Ensure clean go build environment for goreleaser and ko

Previous builds were overriding the GOOS and GOARCH enviroment
variables, which is fine to do when using ko directly, but appears to be
broken when building via ko using goreleaser.
This commit ensures that GOOS and GOARCH are always unset when
running builds by undefining them via make undefine. The override
keyword ensures that the env vars are always undefined regardless of the
source (e.g. enviroment, make variables, etc) to ensure a clean build
environment.
Instead of using GOARCH env var for building for the local build
architecure, this commit uses goreleaser's {{ .Runtime.Goarch }}
variable available via templating which contains the architecture of the
build machine.
This commit also removes the ldflags config from the ko
configuration in goreleaser config as this is automatically inherited
from the build config referenced in the ko config.

The error on Nutanix is to do with reserving the IPs for the CP endpoint:
failed to wait for task completion: failed to wait for task ZXJnb24=:2c4527f5-30b4-4562-98d7-c99345cb9410 to complete: context deadline exceeded

I think we may have a leak but this is blocking PR merges so we need to fix this ASAP.


Issue #1684 Merge pull request #3 from OpenRefine/master

Fixed issue with description: Unable to create a SQL export for a project creating numeric data if I mark the column as numeric #1684

@tcbuzor I don't see any changed files on this pull request.  Did you pull from master first ?  This PR looks weird from Github view.  Perhaps you should delete it and create a fresh one ?


[tsp-client] Allow specifying additional directories during init

Services that require additional directories need to specify them in order to be able to perform a complete client library initialization

These should be specified in the tspconfig.yaml


signing key incorrectly added using apt-key add instead of being dropped in trusted.gpg.d

The package should not be using apt-key add to add keys, but drop a key file in trusted.gpg.d. We APT developers seem to be very bad at getting people to stop using apt-key, arguably we should have added stronger deprecation messages.
Anyway, you should drop a .gpg blob into your directory somewhere and copy it to trusted.gpg.d, and use apt-key del to remove the key from trusted.gpg. Or base64 encode it and decode it, or just drop the ASCII-encoded version with .asc extension (but this only works with APT 1.4 and newer).
Expect the current way to break by 2022 (well, it's even not correctly working right now - you don't depend on gnupg, but apt-key add is not functional without it).

Same for chromium: https://bugs.chromium.org/p/chromium/issues/detail?id=1106309


Notebook-view doesn't work with jupyter installed in $HOME

I followed the installation instructions for linux but I don't get it to use the notebook interface when I open .ipynb files. Instead they are opened in JSON mode.
It might be important to know that I have installed almost all my python packages in my home directory. So the jupyter script is in $HOME/.local/bin. I'm new to atom and have no idea if it will be able to look in $HOME/.local/bin as well for executables.

Toggle the Developer Tools (CTRL+ALT+I) and watch the console as you try to open an ipynb file. It sounds like jupyter-notebook is exiting due to an exception...

Failed to require the main module of 'jupyter-notebook' because it requires an incompatible native module.
Run apm rebuild in the package directory to resolve.
Window load time: 1835ms
/deep/ combinator is deprecated. See https://www.chromestatus.com/features/6750456638341120 for more details.
::shadow pseudo-element is deprecated. See https://www.chromestatus.com/features/6750456638341120 for more details.

Looks like the extension isn't even loaded. Ok I could fix that by clicking on the red bug icon in the lower right corner (running apm rebuild in the package folder didn't help, but that has been reported elsewhere)
Good thing the notebook is now rendered correctly. But it complains that jupyter couldn't be spawned. This is likely because I have installed it at $HOME and atom doesn't pick up my personal configuration for $PATH (I set it as well in $HOME/.profile which should be picked up by most processes started by me).
A fix for me would be if you add a config option to tell the extension where I have installed jupyter.

I added an option to set jupyter binary path: https://github.com/jupyter/atom-notebook/commit/8e616efa100b254e9c931b17249b5b377c0334df

Thanks for adding the option. But I still have the same problem that jupyter couldn't be spawned.
I entered the path (/heom/max/.local/bin) that is returned by which jupyter.  I'm not sure if this matters but I'm running Linux.

What is the exact error message that you're seeing?
Open the Developer Tools console and type process.env.PATH to confirm
that your jupyter path is in there. If you don't see it, then add it and
try opening a notebook. Report back what you find...
On Tue, Jan 5, 2016 at 1:59 PM, kain88-de<EMAIL_ADDRESS>wrote:

Thanks for adding the option. But I still have the same problem that
jupyter couldn't be spawned.
I entered the path (/heom/max/.local/bin) that is returned by which
jupyter. I'm not sure if this matters but I'm running Linux.
—
Reply to this email directly or view it on GitHub
https://github.com/jupyter/atom-notebook/issues/25#issuecomment-169146914
.

I get the following error message

'jupyter' could not be spawned. Is it installed and on your path? If so please open an issue on the package spawning the process.

Checking the path in the console gives.

process.env.PATH
"/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games:~/.local/bin"

As well what is the command that you use to start/connect to the jupyter notebook? It could still be that my jupyter installation isn't working correct.

It sounds like your jupyter path is /heme/max/.local/bi, so try this in the
console: process.env.path = process.env.path + ':/heme/max/.local/bi'and
then try to open a notebook. If that doesn't work, try copying and pasting
that to your init script and reloading Atom (command palette > Reload) and
then try opening a notebook.

On Jan 9 2016, at 12:06 pm, kain88-de<EMAIL_ADDRESS>wrote:

I get the following error message

'jupyter' could not be spawned. Is it installed and on your path? If so
please open an issue on the package spawning the process.

Checking the path in the console gives.

process.env.PATH
"/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games:~/.local/bin"

As well what is the command that you use to start/connect to the jupyter
notebook? It could still be that my jupyter installation isn't working
correct.

—
Reply to this email directly or [view it on GitHub](https://github.com/jupyter
/atom-notebook/issues/25#issuecomment-170275239).![](https://github.com/notifi
cations/beacon/AAfRYhfEXl0SVeNJQtX2WqkUFYK14TSOks5pYV-ugaJpZM4G73qr.gif)

@kain88-de Any luck?

@gnestor no luck. I can't even update the package currently because node-gyp rebuild fails. It is missing headers for to compile some c-functions (I'm a little bit surprised that I have to compile C-code to install some javascript).   I'll reported the issue with the atom team.

See: https://github.com/jupyter/atom-notebook/issues/23
There is an issue with node-gyp and Python 3. Try explicitly setting the Python version PYTHON=python2 apm install.

@kain88-de Still having problems?

After updating atom and jupyter the error has changed but still no luck installing the extension.
It now complains about an error with jupyter-js-utils.

Ok good at least that correlates with another current issue: https://github.com/jupyter/atom-notebook/issues/32#issuecomment-184922403

I'll close this since the original issue has been resolved. Thanks


update the extensions.js

Only included the change to extensions.js in my previous PR.

Please schedule an adhoc build for linux to make sure this doesn't trigger EMFILE.


add support for 'only_if_executable' and handle predefined OSS configs differently

Explain the reasoning for this pull request. For instance, is it for a new feature, bug fix, code style/cleanup, or something else? If fixing an open issue, please link to it here.
Any additional comments/context?

Hi Jesse.
Seems like this PR somehow obsoletes my later PR #140 ... :grin:  ...and also my later PR #141 ...! :joy:
But not completely, since I also changed the PostgreSQL configuration in #140.
So the PR's last merged must be updated before merge to reflect the changes in the first merged PR...!
I had a quick look, and as I understand your change, you want to always deploy the OSS configurations previously found in the samples directory and let the Service Commander check for the executable before start and stop of the service. Right?
One question then: How do you manage any user changes to the configs? That they are not overwritten in future updates of Service Commander? Or do the users have to copy the config files to another directory to keep their changes - a directory that Service Commander will read before its own "oss-common" directory?


New model

https://github.com/GSeanCDAT/GIMM-VFI
Use this as new model because it focuses on flow control and is faster than GMFSS

I will review the project and will @ you in this issue once the integration is complete.

@2994dd42d75b0 Hi, I have integrated Gimm. You can install and configure the Gimm environment after updating the code. Then, use python interpolate_video_forward_anyfps.py -i ... -o ... -m gimm ... to call it. recommend to use -scale 0.5 to reduce the optical flow scale, otherwise, it will consume a large amount of GPU memory (>17GB) on 1080p videos.

Thanks

Loaded model
97% 139/143.0 [15:11<00:24,  6.16s/it]Wait for all frames to be exported...
Stuck on this only for model gimm


undefined method `update' for #Mustache::Context:0x000000053d34e8 stash 1.0.3, rails 4.2.4

Hello,
I was trying to use Stash 1.0.3 with rails 4.2.4, when I try to render the page I get the error below
Started GET "/bikes/new" for ::1 at 2015-10-06 23:30:06 -0500
Processing by BikesController#new as HTML
Rendered app/templates/bikes/new.mustache within layouts/application (7.4ms)
Completed 500 Internal Server Error in 9ms (ActiveRecord: 0.0ms)
ActionView::Template::Error (undefined method update' for #<Mustache::Context:0x00000004c7d4c8>):   app/templates/bikes/new.mustache:18:in _app_templates_bikes_new_mustache___1745009738163216398_70103334426860'
I've tried to use rails 4.2.4 with stash 1.1.1, I don't get an error however the page does not render.
Any help will be appreciated.

Turns out it was an error on my part.  Rails 4.2.4 and stash 1.1.1 works, the problem was that my views ruby file name was not correct.
module Bikes
class New < Stache::Mustache::View
def planet
Bike.planet
end
end
end
The name of the file was apps/views/bikes/bike.rb and it would not render correctly, once I changed it to apps/views/bikes/new.rb the page rendered correctly... newbie mistake.


Match generated files by frontmatter

When writing new highlights to markdown files, check if an existing file with the article URL in its frontmatter already exists. Create this frontmatter for new files. Releasing this will require people to reset & delete all their existing files, so we may want to add a migration path in another PR.
New files created after this change merges can be moved around or renamed freely.
The markdown files now somewhat act as the synchedFiles state which was kept manually before. Thus I deleted that state and rewrote the logic of hypothesis-resync-deleted to match the Hypothesis API response to this local state.
@weichenw what's your feedback on this?
References:

obsidian-kindle-plugin implementation of frontmatter matching https://github.com/hadynz/obsidian-kindle-plugin/blob/master/src/fileManager/index.ts
Obsidian MetadataCache API docs: https://github.com/obsidianmd/obsidian-api/blob/master/obsidian.d.ts#L2033

I dont use it as extensively as some other users. But I am ok with the replacement of the delete logic

Thanks again mate. I think the group tagging is broken. I haven't got the time to test them properly.


Problem with Reactive Forms in edit scenario (setting default value for control)

I get persian date string from the API, and set it as the value of the FormControl:
this.projectForm.controls['startDate'].setValue(response.startDate); // '1397/03/10'
And it works fine. However, when I get the form value, the date passes as Miladi date to the server:
this.prjService.editProject(this.projectForm.value).subscribe(); //  startDate: 'its Mildai equivalent'
I also have set a default format for control via config:
datePickerConfig = {         format: 'jYYYY/jMM/jDD'     };
And the control declaration:
dp-date-picker dir="rtl"                                      formControlName="startDate"                                      mode="day"                                      placeholder="تاریخ شروع"                                      theme="dp-material"                                     [config]="datePickerConfig">                                 </dp-date-picker>
What's wrong?

This behavior occurs only if I set the default value (as mentioned above) and NOT change the date by the control.
When I change the dates by the control itself, the problem disappears and works fine!

i have this problem too , exactly as @aminsaghi-ir said .

i'm using 2.2.6.
in my case the value it's not moment object , it's just a String and i think that i can't format it before send it to server.

@fingerpich  what is wrong ? i have to use the moment format ?

I think the problem is in using jalali date directly.
Please use the following code or make a demo in plunker to make the problem clear.
import moment from 'jalali-moment'
const jm = moment.from(response.startDate, 'fa', 'YYYY/MM/DD');
this.projectForm.controls['startDate'].setValue(jm);

you tell that is set value with 2 way binding and ngModel is incorrect ? i think a nice plugin like this must have this feature !!

No I only passed a moment object instead of string

as you know i cant use moment as type , so what is the type of this property in your model ?


RANK() OVER (ORDER BY x) reversed in recent update

Context: The "x" column is a scoring column. The higher the score, the rank is closer to 0.
After building a new version of my application (but not updating the version of the sqlite package), the version was and still is v1.14.22, I noticed that I am getting opposite query ordering as before my application update.
I reverted to the old code and noticed the issue still persisted, but it did not occur in production.
I pulled down the production binary and ran it locally, noticing that the the production version, when ran locally, gave the correct result. The two versions have the exact same query.
I am guessing this is to do with a portion of the C libraries. I notice in the sqlite3 command line, the output matches how it is now.
Was this a bug fix? Is this the correct behavior?
Since even with the old version of the code, I cannot replicate the older behavior, but I can with the older production binary, what on my disk changed to make the change to ORDER BY x DESC rather than ORDER BY x?

Please provide the full query, sample data set, expected output, and actual output.
Also are you compiling with any build tags?

@rittneje
		SELECT rank, elo FROM (
		SELECT RANK() OVER (ORDER BY elo) rank, elo, tg_id
		FROM user
		) x
		WHERE tg_id = xyz;

This is the query. The previous functionality was as if there was as DESC inside the ORDER BY.
Example data: (tg_id column and elo respectively)
1846306122|1455
6170115934|1545

Output when query was run in version compiled in June for user id<PHONE_NUMBER>:
2|1455
Output when query was run by version compiled today for user id<PHONE_NUMBER>:
1|1455
I think the output is correct now, it was supposed to have the DESC inside it from the beginning, but I cannot figure out how recompiling brought about this change.

If it helps, I can provide the binaries, both compiled from identical source code and go mod projects, exhibiting the different outcomes, but they come with a lot of setup to get running.


TypeError: catppuccin is not a function

I followed the readme to configure tailwind.config.js, but I got an error "TypeError: catppuccin is not a function"
my tailwind.config.js is set up like this:
import daisyui from 'daisyui'
import catppuccin from '@catppuccin/daisyui'

/** @type {import('tailwindcss').Config} */
export default {
  content: ['./index.html', './src/**/*.{vue,js,ts,jsx,tsx}'],
  theme: {
    extend: {},
  },
  plugins: [daisyui],
  daisyui: {
    themes: [catppuccin('latte'), 'light'],
  },
}

project dependencies like this:
dependencies:
@vercel/analytics 1.4.0
chart.js 4.4.6
pinia 2.2.6
vue 3.5.13
vue-router 4.4.5

devDependencies:
@catppuccin/daisyui 1.2.1
@vitejs/plugin-vue 5.2.0
autoprefixer 10.4.20
daisyui 4.12.14
postcss 8.4.49
tailwindcss 3.4.15
vite 5.4.11

pnpm version is 9.13.2
node version is 23.1.0
error message

Since the reproduction hasn't been attached, I forked your repo and created this code sandbox.
I'm not sure what other changes you made, but I didn't encounter any issues when adding dependencies and configuring them. And maybe you can just console.log(catppuccin) to gather more information!
You can refer to my commit, and I can submit a PR for you if needed.

Sorry for not providing more information about the repo. I want to replace vuetify with daisyui in the original repo.
 But when I downgrade the node version from 23.1.0 to 22.11.0, the problem is solved.
I will close this issue now.
Thank you very much for your answer :v: .


Kong 0.14.1 - path match results in "no route and no API found with those values"

NOTE: GitHub issues are reserved for bug reports only. For anything else,
please join the conversation in Kong Nation https://discuss.konghq.com/.
Please read the CONTRIBUTING.md guidelines to learn on which channels you can
seek for help and ask general questions:
https://github.com/Kong/kong/blob/master/CONTRIBUTING.md#where-to-seek-for-help
Summary
Longest path not being evaluated/matched causing "no route and no API found with those values"
When defining two paths with different methods, the longer path is not evaluating/matching the correct resource.
Steps To Reproduce

Create a Service
resource "kong_service" "test" {
name              = "apitest"
protocol          = "http"
host              = "someazureinstance"
port              = 23456
path              = "/"
}
Create base route with only GET method
resource "kong_route" "test" {
protocols         = ["http"]
methods           = ["GET"]
hosts             = ["somehost.net"]
paths             = ["/test"]
strip_path        = false
preserve_host     = true
service_id        = "${kong_service.test.id}"
}
Create longer route with GET and DELETE methods
resource "kong_route" "testById" {
protocols         = ["http"]
methods           = ["GET", "DELETE"]
hosts             = ["somehost.net"]
paths             = ["/test/{id}"]
strip_path        = false
preserve_host     = true
service_id        = "${kong_service.test.id}"
}
Call longer route using DELETE - results in "no route and no API found with those values"
DELETE http://somehost.net/test/123

Additional Details & Logs

Kong version (0.14.1)
Kong debug-level startup logs ($ kong start --vv)
Kong error logs (<KONG_PREFIX>/logs/error.log)
Kong configuration (the output of a GET request to Kong's Admin port - see
https://getkong.org/docs/latest/admin-api/#endpoint)
Operating system

/test/{id} is not a valid PCRE regex. Kong will thus only attempt to match it as a plain path:
curl -g http://somehost.net/test/{id}

Note the -g option:
 -g, --globoff       Disable URL sequences and ranges using {} and []

Please refer to the Proxy Guide to get guidance on how to define regex paths.


Fixed squished button bug on landing page for mobile

Changelog:
•Made the buttons stay the same width for mobile
(src/components/LandingInfo.scss)
•Corrected the format of the page so that when you shrink the size of the screen the paragraphs and their headers are in order (src/components/LandingInfo.tsx)

Nice!


[role]: confusing service error caused by invalid --scope argument

Describe the bug
az ad sp create-for-rbac use wrong apiVersion.
To Reproduce
az ad sp create-for-rbac --name autotools-master-sp --role contributor --scope /subscriptions/<subscript id>/myRG

Output:
Role assignment creation failed.

role assignment response headers: {'Cache-Control': 'no-cache', 'Pragma': 'no-cache', 'Content-Type': 'application/json; charset=utf-8', 'Expires': '-1', 'x-ms-failure-cause': 'gateway', 'x-ms-request-id': '297b982c-922a-4940-91b4-fcd6478c87f3', 'x-ms-correlation-request-id': '297b982c-922a-4940-91b4-fcd6478c87f3', 'x-ms-routing-request-id': 'WESTUS:20180928T211950Z:297b982c-922a-4940-91b4-fcd6478c87f3', 'Strict-Transport-Security': 'max-age=31536000; includeSubDomains', 'X-Content-Type-Options': 'nosniff', 'Date': 'Fri, 28 Sep 2018 21:19:49 GMT', 'Content-Length': '407'}

The api-version '2018-01-01-preview' is invalid. The supported versions are '2018-08-01,2018-07-01,2018-06-01,2018-05-01,2018-02-01,2018-01-01,2017-12-01,2017-08-01,2017-06-01,2017-05-10,2017-05-01,2017-03-01,2016-09-01,2016-07-01,2016-06-01,2016-02-01,2015-11-01,2015-01-01,2014-04-01-preview,2014-04-01,2014-01-01,2013-03-01,2014-02-26,2014-04'.

Expected behavior
The service principal been created an role been assigned properly.
Environment summary
Azure Cloud Shell Bash
$ az --version
azure-cli (2.0.46)

acr (2.1.5)
acs (2.3.4)
advisor (0.6.0)
ams (0.2.3)
appservice (0.2.4)
backup (1.2.1)
batch (3.4.0)
batchai (0.4.3)
billing (0.2.0)
botservice (0.1.1)
cdn (0.1.1)
cloud (2.1.0)
cognitiveservices (0.2.3)
command-modules-nspkg (2.0.2)
configure (2.0.18)
consumption (0.4.0)
container (0.3.4)
core (2.0.46)
cosmosdb (0.2.1)
dla (0.2.3)
dls (0.1.3)
dms (0.1.1)
eventgrid (0.2.0)
eventhubs (0.2.4)
extension (0.2.1)
feedback (2.1.4)
find (0.2.12)
interactive (0.3.30)
iot (0.3.2)
iotcentral (0.1.2)
keyvault (2.2.3)
lab (0.1.1)
maps (0.3.2)
monitor (0.2.3)
network (2.2.5)
nspkg (3.0.3)
policyinsights (0.1.0)
profile (2.1.1)
rdbms (0.3.2)
redis (0.3.2)
relay (0.1.2)
reservations (0.4.0)
resource (2.1.4)
role (2.1.5)
search (0.1.1)
servicebus (0.2.3)
servicefabric (0.1.3)
signalr (1.0.0)
sql (2.1.4)
storage (2.2.2)
telemetry (1.0.0)
vm (2.2.3)

Python location '/opt/az/bin/python3'
Extensions directory '/home/myname/.azure/cliextensions'

Python (Linux) 3.6.5 (default, Sep 18 2018, 21:58:22)
[GCC 5.4.0 20160609]

Legal docs and information: aka.ms/AzureCliLegal

**Additional context**
Add any other context about the problem here.

Are you in azure stack?

@yugangw-msft I don't think so. How can I verify this?

It's AzureCloud.
az assignment list  return error az: 'assignment' is not in the 'az' command group. See 'az --help'..
I guess you want me to run az role assignment list. All roles have been printed without any error after running this command.

correct, i meant az role assignment list
to isolate, can you just run az ad sp create-for-rbac w/o any argument? does that work?

i believe your scope might be wrong /subscriptions/<subscript id>/myRG, you missed a resourceGroups in the middle. Should be /subscriptions/<subscript id>/resourceGroups/myRG

@yugangw-msft you are right, I missed a resourceGroups in the middle. After added it, I created the service principal successfully.
Maybe you can change this ticket to something like misleading error when wrong a scope been specified.

add to S164.


Initialize all Attribute fields for Server

While playing around with subscriptions I noticed that we only initialize the attributes that are defined with default values for a cluster. Fields that are not in the "attributesInitialValues" are potentially never initialized correctly.
To fix this I now iterate over all attribute fields. But this means that we now could have undefined as value which basically is better then mitting attributeservers. For this I added filter in the relevant places to filter out undefined values when sending  read responses or DataReports on subscriptions.

Typescript compiler forces to define all mandatory attributes in attributesInitialValues.
Optional attributes can optionally also be defined in attributesInitialValues.
So I think initializing only attributesInitialValues defined attributes is correct, since this is:

all mandatory attributes
optional attributes that this implementation chooses to use

Commands only have access to mandatory attributes, so they are all initialized.
Can you point out which are the case were an attribute might not be initialized correctly?

OptionalWritable attribute that is written later? Would be missed now and return an error. And it felt wrong to then "initialize" it dynamically.
And I'm not that sure that a command handler would never need/try to access an optionalAttribute - I did not yet read all specs. I assume we would then search a while ;-)
And last but not least the ClusterClient for use
As controller initialize all attributes (yes he needs to, but so behaves slightly different).
But also to prepare a consistent high level api it would be hard when the available get methods differ based on the fact which data were set with initial values.

I checked again and yes I stumbled over this topci while implementing the "WriteRequests" - not "TS compiler warning wise" but semantically.
The "getAttribute" in IntractionServer.ts would not find such an attribute because it iterates over the attributes that got initialized by CLusterServer ... so all thee would be missing for write requests. At minimum there it is unexpected because we would return an error about an unsupported Write.
So I have decided to not add "special handling and dynamic delayed Attribute cluster initialization" on WriteRequests but fix the underlying issue directly ... This PR is for that.

I still don't understand what you are trying to fix, can you walk me through the flow that creates the problem?
Here is how I implemented it.
Let's say, we have a cluster C with attributes:

A: mandatory
B: optional
C: optional

When initializing the cluster, A initial values needs to be provided, and optionally B / C.
The command handlers can only access A which is always defined.
If a command needs to access the optional attribute B, and can be redeclared as mandatory, using UseOptionalAttributes
Let's say that A and B are initialized on the cluster.
read A => valueA (mandatory and always defined)
read B => valueB (optional but always defined since initialized at cluster creation)
read C => nothing (optional and not present on the cluster)
write A => works
write B => works since the AttributeServer has been created
write C => error since C doesn't exist (the optional AttributeServer was not created)
Optional attributes can't be initialized dynamically: they are either initialized at the creation of the cluster or they don't exist


[GPT-35] 267 footer image cannot be seen

changed footer color to black

Looks like this now

@gajinkee how about the "Powered by Ghost" part lol

Oh my bad did not see it haha. Fixed it here.


Update poetry & poetry-dynamic-versioning in gh deploy action

Most importantly, pin a version of MarkupSafe that fixes the failure that was making poetry-dynamic-versioning to be skipped
The "Publish to PyPI" step is failing like this at the moment:
Publishing rovio-ingest (0.0.1) to PyPI
Error processing line 1 of /opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/zzz_poetry_dynamic_versioning.pth:

  Traceback (most recent call last):
    File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site.py", line 168, in addpackage
      exec(line)
    File "<string>", line 1, in <module>
    File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/poetry_dynamic_versioning/__init__.py", line 7, in <module>
      import jinja2
    File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/jinja2/__init__.py", line 12, in <module>
      from .environment import Environment
    File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/jinja2/environment.py", line 25, in <module>
      from .defaults import BLOCK_END_STRING
    File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/jinja2/defaults.py", line 3, in <module>
      from .filters import FILTERS as DEFAULT_FILTERS  # noqa: F401
    File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/jinja2/filters.py", line 13, in <module>
      from markupsafe import soft_unicode
  ImportError: cannot import name 'soft_unicode' from 'markupsafe' (/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/site-packages/markupsafe/__init__.py)

Pinning MarkupSafe==2.0.1 fixes that error.
As a bonus, minor update to poetry version.

👍


Fix review issues for adding into rancher charts

Related pr: https://github.com/rancher/charts/pull/1595

@yaocw2020 can u please help to update the PR of https://github.com/rancher/charts/pull/1595 and backport the changes to the master branch, thanks.

@yaocw2020 can u please help to update the PR of rancher/charts#1595 and backport the changes to the master branch, thanks.

Updated


chore: fixes and updates

coverage: 93.952% (-1.2%) from 95.112%
when pulling 63c2aeb3978b18192449c8a2d129e9f3610e0630 on 0-fixes-and-updates
into 09c5ecc25adc1e5e0888f5abe3c5ceb167821743 on main.


build.gradle ERROR - Android {...} line 35, column 10

THE ERROR
PS C:\Users\...\notage_finish> flutter build apk
Das System kann den angegebenen Pfad nicht finden.

 Building with sound null safety

FAILURE: Build failed with an exception.

* Where:
Build file 'C:\Users\...\notage_finish\android\app\build.gradle' line: 34

* What went wrong:
Could not compile build file 'C:\Users\...\notage_finish\android\app\build.gradle'.
> startup failed:
  build file 'C:\Users\...\notage_finish\android\app\build.gradle': 34: unexpected token: ... @ line 34, column 10.
     android {...
              ^

  1 error

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 1s
Running Gradle task 'assembleRelease'...
Running Gradle task 'assembleRelease'... Done                    2.058ms
Gradle task assembleRelease failed with exit code 1
PS C:\Users\...\notage_finish> `

THE build.gradle
def localProperties = new Properties()
def localPropertiesFile = rootProject.file('local.properties')
if (localPropertiesFile.exists()) {
    localPropertiesFile.withReader('UTF-8') { reader ->
        localProperties.load(reader)
    }
}

def flutterRoot = localProperties.getProperty('flutter.sdk')
if (flutterRoot == null) {
    throw new GradleException("Flutter SDK not found. Define location with flutter.sdk in the local.properties file.")
}

def flutterVersionCode = localProperties.getProperty('flutter.versionCode')
if (flutterVersionCode == null) {
    flutterVersionCode = '1'
}

def flutterVersionName = localProperties.getProperty('flutter.versionName')
if (flutterVersionName == null) {
    flutterVersionName = '1.0'
}

apply plugin: 'com.android.application'
apply plugin: 'kotlin-android'
apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"

def keystoreProperties = new Properties()
   def keystorePropertiesFile = rootProject.file('key.properties')
   if (keystorePropertiesFile.exists()) {
       keystoreProperties.load(new FileInputStream(keystorePropertiesFile))
   }

android {...
}

   compileSdkVersion 30

    sourceSets {
        main.java.srcDirs += 'src/main/kotlin'
    }

    defaultConfig {
        // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html).
        applicationId "com.blagrenivi.notage"
        minSdkVersion 16
        targetSdkVersion 30
        versionCode flutterVersionCode.toInteger()
        versionName flutterVersionName
    }

    signingConfigs {
       release {
           keyAlias keystoreProperties['keyAlias']
           keyPassword keystoreProperties['keyPassword']
           storeFile keystoreProperties['storeFile'] ? file(keystoreProperties['storeFile']) : null
           storePassword keystoreProperties['storePassword']
       }
   }
   buildTypes {
       release {
           signingConfig signingConfigs.release
       }
   }
}

flutter {
    source '../..'
}

dependencies {
    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
}
`

What is false?

Hi @Blackjack386
There is an error in your build.gradle

build file 'C:\Users...\notage_finish\android\app\build.gradle': 34: unexpected token: ... @ line 34, column 10.
android {...
^

Please fix your build.gradle and run the app again. you can check if the there are any errors by opening the android folder in Android Studio and check out gradle, you will see what errros are there
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


feat: Support setting userId and sessionId manually

I am working on adding CloudWatch RUM to a web app. Our users will authenticate with AWS Identity Center (IDC) which assigns each user a UUID already. I'd like to use that same UUID as the userId RUM instead of letting RUM generate a new UUID for me. Having consistent userIds will make it easier to connect RUM data with our backend events. We could pass in the IDC user ID as event metadata but it makes sifting through data more confusing and error prone.
Passing in user and session IDs could look like:
const config: AwsRumConfig = {
  userId: 'bd33b2a2-4b14-4ea6-aaeb-ee85c2062f5d',
  sessionId: '40af9b9a-6fa2-45f8-a437-67d2566380d9'
};

Hi @crowecawcaw thanks for this; I like your proposal. Would you ever dynamically change the sessionId, or would it be sufficient to set the session Id only on initializing AwsRum?

For our specific use case, we would only want to set the sessionId at initialization time.

For a different use case, we need to dynamically set both sessionId and userId.

To summarize, we need the following to close out:

[ ] Statically set user id at initialization time
[ ] Statically set session id at initialization time

In addition, I have also received feedback for

[ ] Dynamically set user id at runtime
[ ] Dynamically set session id at runtime
[ ] Dynamically start, stop, and get current session at runtime


=, not =<

@xhlwill Only webpack@3 and lower contain 0.4.6 version

Isn't webpack>=3.0.0 <4 contain 0.4.6

@xhlwill agree some misleading, let's rewrite to webpack < v4.0.0

@xhlwill also please accept CLA

Please rebase with current master

Is it right...

@xhlwill can you rebase again?


INVALID ARGUMENT

I tried the example you provided gives the same error

Are you trying to fetch reply chat? If so, set isReplyChat true:
https://github.com/holodata/masterchat/blob/9280689e76aa1e4e6578523ac5292f72c67bc842/packages/core/src/chat.ts#L889-L892

But a clearer error message would certainly be better.

could you give an example on hw to fetc livechats for a certain ideo iid at a cirtain time? would help a lot thx

Just released a new version with a whole bunch of breaking changes and improvements.
I also wrote some example codes in the README please check it out.
https://github.com/holodata/masterchat
Fetching chat at a certain time of the video is not supported and I don't know how. Please consider sending PR if are interested!


Bug: Only one account can be created

Trying to create two accounts returns a conflict

I had a unique constraint on the verify token, and it was triggered when I set it to an empty string to signify verified. Solution is to make it nullable, since null doesn't trigger uniqueness constraint.

Confirmed fixed


Bugfix: Stop throwing a missing-class-property-error when only @dataMember is specified

This should prevent throwing 'source object must be an object' error for cases
when @dataMember() is specified and @required() is NOT specified for the property that links to another class.

@santee it is better to have CI configured for this project


Restrict to show 2 characters in avatar

Is there any option to restrict it to show just 2 characters in avatar?

Hello,
The current version of ngx-avatar component does not have this feature. However, I think that it makes sense to add this restriction as input.
I will implement this feature and release a new version asap

This feature has been added in version 2.8.0.
I just added a new property initialsSize  to fix the number of characters for initials avatars.
Here's an example that restricts the avatar to render 2 characters :
<ngx-avatar  initialsSize="2"  name="John Johnson Doe"></ngx-avatar>

Thanks.


Moved menu to top, refactored some parts of the interface.

There is a lot in this change:

Removed filtering in the main dashboard
Main dashboard has a list of apps & services as most important
A lot of componentization (Badges, Buttons, ContextMenu, SVGs etc) to start making all screens look more similar and demand less css work and avoid code repetition.
A rewrite of getStatus to have a centralized logic ($lib/api/status)

Still pending:

[ ] rewriting foundUnconfigured...
[x] ensuring cleanup works and is available
[ ] I have not understood what otherGitSources and similar variables meant, removed them for now.
[x] Making LeftSidebar and RightSidebar mobile friendly

Wow, thanks for the PR! Really appreciate it!
But I have some concerns:

The main navbar should stay on the left side because on the top it looks a bit ugly with the huge icons. Maybe if blend it into the main background color, using smaller fonts + text would be better.
Why did you remove the dashboard filtering feature? It is useful for quickly filtering through applications.

There are lots of changes here and it is hard to follow them tbh. Can you please separate them into separate PR's please?


Add Support for Awaitable XHP

Adds support for generators inside XHP rendering. This makes one very important change to basic XHP principles, and that is that the rendering order can no longer be guaranteed. Previously, XHP would follow a breadth-first rendering from :x:element to :x:primitive (concurrently), but with this change, XHP will now render depth-first in search of WaitHandles, batching together as many as it can. Additionally, all :x:elements will be rendered before the :x:primitive pass will begin. In theory this should have no affect because XHP does not store or encourage storing of references to other XHP elements (besides the parent, which still will be guaranteed to be rendered first). However, it is still possible to manually keep references and a variable rendering order has the potential to affect code in unpredictable ways.
In terms of implementation, I wanted to just have render() without the need for genRender(), then an interface would be all that was needed, but the return type of render() would need to be different in each case. I could get away without using the XHPAwaitable interface and instead check for the XHPGenerator trait using class_uses(), but this doesn’t check parent traits, so I would have to loop over each element’s inheritance chain and that felt very ugly to me.
Also fixed some 80 cols stuff cause I took this from my personal project and I'd made those changes awhile ago.
fixes #93

edited message to mention the issue

https://github.com/Swahvay/xhp-lib/pull/1


Compare function URL construction error.

Describe the bug
Compare function is not correctly set. Loading a run list shows the following pop-up windown message.
Compare function failed
TypeError: URL constructor: http://(ids, token) => 'http://acme.com:3030/report/foobar?q=' + ids.join('&q=') + "&token="+token is not a valid URL.

The above has been obfuscated slightly but demonstrates the point.
To Reproduce
Open this run list /run/list/20 to see the UI reporting the construction error.
Version
0.12.1

Java
What is the version of Java ?
17

The compare function changed with release 0.12. It is now and Horreum will append id=<runId> for each selected run


GithubBrowserSample extend login function

I have applied Android Architecture Component the same as GithubBrowserSample.
Now, I want to develop the login function in my app. I want to use username and password to get a token and save it. When I call API, I will attach a token.
Anybody can give me some suggestion about technical.
Thanks and best regards,

if you known MVVM archi and Dagger, you can try this
https://github.com/MindorksOpenSource/android-mvvm-architecture/blob/master/app/src/main/java/com/mindorks/framework/mvvm/data/remote/ApiHeader.java

thanks, @nguyenvanminhfptpoly


The webpage css overrides the extension css

I was building the extension using tailwind CSS. But, the extension UI is rendering differently on different websites. I mean, on some of the sites, it is behaving as we expected, But some of the sites, it intakes the css property of that particular website.
Any idea of how to fix that ?
I also tried injectCSS. That is also not working

@Rooban19  If you are displaying something in the web page, then move your UI to an iframe and load css in this iframe to prevent this issue.

I'm using a different framework but also have the same problem. I tried isolation with custom element and shadow DOM of web components, but no luck either.
Adding an iframe won't always work because sometimes I need to insert element, say a button, into the host page's DOM, and that button alone can introduce conflict of style.
@Rooban19, have you figured out a solution?

I have used the shadow DOM to render the css that I need. And also I wrote the css for all the tags with unique css selectors name which won't affect the css of host's. And this solution worked for me  @felixswang


Add select mode

When select mode is active, user can select several files and add (or remove) tags to these files, remove them.

The context menu must change when select mode is turned on:

Options Rename, Update description must be disabled.
Update tags, and Delete must affect all selected files.
Download must download zip archive, which contains all selected files.

It's left to test and check bugs.


Datadog Lambda Extension does not respect Tags

Expected Behavior
env, service and version tagged appropriately in Datadog
Actual Behavior
env === none
service === aws.lambda
Steps to Reproduce the Problem
import { Runtime } from "@aws-cdk/aws-lambda";
import { PythonFunction } from "@aws-cdk/aws-lambda-python";
import * as cdk from "@aws-cdk/core";
import { Tags } from "@aws-cdk/core";
import { Datadog } from "datadog-cdk-constructs";

export class DdtestStack extends cdk.Stack {
  constructor(scope: cdk.Construct, id: string, props?: cdk.StackProps) {
    super(scope, id, props);

    Tags.of(this).add("env", "my-test-env");
    Tags.of(this).add("service", "my-test-service");

    const dd = new Datadog(this, "Datadog", {
      pythonLayerVersion: 38,
      // forwarderArn: this.formatArn({
      //   service: "lambda",
      //   resource: "function",
      //   resourceName: "datadog-forwarder",
      //   sep: ":",
      // }),
      extensionLayerVersion: 8,
      apiKey: process.env.DD_API_KEY!,
    });

    const fn = new PythonFunction(this, "Function", {
      entry: "./lib/handler",
      runtime: Runtime.PYTHON_3_8,
    });

    dd.addLambdaFunctions([fn]);
  }
}

The handler
from ddtrace import tracer

@tracer.wrap()
def another():
    return "another"

@tracer.wrap("MyHandler")
def handler(event, context):
    print(another())
    return "hi"

Specifications

Datadog Lambda Layer version: 8
Node version: v14.15.4

Screenshots

The top trace is using the forwarder.
The second trace is using the extension.

This may belong in https://github.com/datadog/datadog-agent

@liveFreeOrCode thanks for the bug report, it looks like related to a backend issue (no update needed on your side once fixed) that we are fixing right now.

@tianchu - Would using DD_ENV, DD_SERVICE and DD_VERSION be an acceptable workaround do you think in the meantime?

@liveFreeOrCode we are pushing out the fix this week, but yes setting those DD_XXX env vars might help.

👋 Hi @tianchu , just checking in on this.

@liveFreeOrCode sorry for the late reply, the team was focusing on the actual fix and forgot to update on this issue. I'm closing the issue, since it should have been fixed for everyone. Please reopen if that's not the case.


Async checking

Take a look at the async cost handling and the processing of the results data sent back

resolved and update the func to handle result and then use mutex lock to protect the append


IOU - LHN for IOU report still shows the requested amount when IOU is deleted

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

Version Number: v1.4.44-0
Reproducible in staging?: y
Reproducible in production?: y
Issue reported by: Applause - Internal Team
Issue found when executing PR: https://github.com/Expensify/App/pull/36557
Action Performed:

Go to staging.new.expensify.com
Go to 1:1 DM.
Create a manual request.
Go to request details page.
Send a message.
Click 3-dot menu > Delete request.
Return to IOU report.

Expected Result:
LHN for IOU report will show [Deleted request].
Actual Result:
LHN for IOU report still shows the requested amount when IOU is deleted.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/bc35c5fa-714a-4f9c-99bb-dde16d9d01d8
View all open jobs on GitHub

@stephanieelliott I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-bills
CC @davidcardoza

I don't really think this is a bug, as long as the X owes Y 0.00 stays current, it's not unexpected that the LHN shows the most recent request made (even if it was deleted afterward). I'm not sure if it was a conscious UI decision or something that wasn't given consideration, but I don't think we need to prioritize updating this behavior.


ci: do not limit feature powerset check by depth

Motivation
Help us to catch errors like https://github.com/alloy-rs/alloy/commit/9eb1409dfd4d4af5f87aa3cef73f2bdb7a99208b
Solution
Check full feature powerset without limiting by depth.

That's not even gonna compute because it goes OOM lol

Depth 2 is ok but could take too long


Organize tutorials sections

Description
This PR continues some of the improvements to the organization of our tutorials proposed in #4077, but adding an extra level of organization.
I have gotten something wrong though, as I don't see the extra level and what I thought would be a link is not a link

I've also gotten some warnings
/Users/nsofroniew/GitHub/napari/docs/howtos/layers/points.md:28: WARNING: 'myst' reference target not found: ../../tutorials/applications/annotate_points
/Users/nsofroniew/GitHub/napari/docs/howtos/layers/points.md:92: WARNING: 'myst' reference target not found: ../../tutorials/applications/annotate_points
/Users/nsofroniew/GitHub/napari/docs/tutorials/annotation/index.md:5: WARNING: 'myst' reference target not found: annotate_points
/Users/nsofroniew/GitHub/napari/docs/tutorials/processing/index.md:5: WARNING: 'myst' reference target not found: dask
/Users/nsofroniew/GitHub/napari/docs/tutorials/segmentation/index.md:5: WARNING: 'myst' reference target not found: annotate_segmentation
/Users/nsofroniew/GitHub/napari/docs/tutorials/tracking/index.md:5: WARNING: 'myst' reference target not found: cell_tracking

So clearly there is a bit more work to do before this is done, but if anyone had any tips on how to fix the nesting & links that would be great, thanks!

Looks great! One thing that might be helpful is open documentation issues with content requests for each of those sections, so that people know how to contribute.

Looks great! One thing that might be helpful is open documentation issues with content requests for each of those sections, so that people know how to contribute.

Nice I like that idea.
Also @melissawm I'm running into the following myst reference target not found errors, but i'm not sure why - are they typos? the wrong link formatting?
/Users/nsofroniew/GitHub/napari/docs/howtos/layers/points.md:28: WARNING: 'myst' reference target not found: ../../tutorials/annotation/annotate_points
/Users/nsofroniew/GitHub/napari/docs/howtos/layers/points.md:92: WARNING: 'myst' reference target not found: ../../tutorials/annotation/annotate_points
/Users/nsofroniew/GitHub/napari/docs/tutorials/annotation/index.md:5: WARNING: 'myst' reference target not found: annotate_points
/Users/nsofroniew/GitHub/napari/docs/tutorials/processing/index.md:5: WARNING: 'myst' reference target not found: dask
/Users/nsofroniew/GitHub/napari/docs/tutorials/segmentation/index.md:5: WARNING: 'myst' reference target not found: annotate_segmentation
/Users/nsofroniew/GitHub/napari/docs/tutorials/tracking/index.md:5: WARNING: 'myst' reference target not found: cell_tracking

The things aren't links too. Do you have any idea how to fix? Maybe @jni @Carreau or @tlambert03 knows too - sorry if this is super simple thing, i just can't seem to figure it out!!

This is because the default for the current ToC setup is to ignore files that are not included in the ToC - so links to them aren't generated. I'll note the fix below.

This is because the default for the current ToC setup is to ignore files that are not included in the ToC - so links to them aren't generated. I'll note the fix below.

Ah ok great - that makes sense. Maybe worth capturing in our docs contributing guide - cc @DragaDoncila

This is now working great - thanks for help @melissawm. PR is ready for review


Update scheduling-email.md

having a look here i think is send_at ->> https://sendgrid.com/docs/API_Reference/SMTP_API/scheduling_parameters.html
Description of the change:
Reason for the change:
Link to original source:

Closes #

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

Hello @marmureanuweb,Thanks again for the PR! We want to show our appreciation by sending                                              you some swag. Could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Update Key Vault configuration provider docs

Fixes #26443
Fixes #26714

/cc @JoshLove-MSFT

@heaths  much appreciated.


More clarity on the README

Hi, Can you please me understand this sentence better: "Please note: Since version 6.3 Hybris has built-in recipes with Docker support and it's recommended solution." Could you please share me the recipes with docker support?

Hi,
Please note, this project is not official Hybris repository. It's just a PoC. If you're looking for the support, please write to support guys or on the Hybris Experts Forum. I'm not sure if mentioned feature was already released, but there gonna be recipes with "dockerized" in their name. They should be available in the snapshot releases. I'm not maintaining these recipes, they're more production ready than this PoC and they're doing slightly different work than a project in this repo.


Docker: Allow to skip Talos and CLI build and use it in integration test

The Talos build and CLI tools take more time combined than Agora, and are not used in integration test.
The also represent a major slowdown when a developer wants to try out something locally.
This brings the timing on building on my machine from:
 => [builder 4/8] RUN if [ -z ${AGORA_STANDALONE+x} ]; then npm ci && npm run build; else mkdir -p build; fi                    364.2s
 => [builder 5/8] WORKDIR /root/agora/                                                                                0.0s
 => [builder 6/8] RUN AGORA_VERSION=HEAD dub build --skip-registry=all --compiler=ldc2 ${DUB_OPTIONS}                           298.0s
 => [builder 7/8] RUN if [ -z ${AGORA_STANDALONE+x} ]; then dub build --skip-registry=all --compiler=ldc2 -c client;     else c  33.5s
 => [builder 8/8] RUN if [ -z ${AGORA_STANDALONE+x} ]; then dub build --skip-registry=all --compiler=ldc2 -c config-dumper;     263.2s

to:
 => [builder 4/8] RUN if [ -z ${AGORA_STANDALONE+x} ]; then npm ci && npm run build; else mkdir -p build; fi                      0.5s
 => [builder 5/8] WORKDIR /root/agora/                                                                                0.0s
 => [builder 6/8] RUN AGORA_VERSION=HEAD dub build --skip-registry=all --compiler=ldc2 ${DUB_OPTIONS}                           376.4s
 => [builder 7/8] RUN if [ -z ${AGORA_STANDALONE+x} ]; then dub build --skip-registry=all --compiler=ldc2 -c client;     else cp  0.7s
 => [builder 8/8] RUN if [ -z ${AGORA_STANDALONE+x} ]; then dub build --skip-registry=all --compiler=ldc2 -c config-dumper;       0.4s

Which is saving over 10 minutes of build time.

Current pipeline takes 22 to 24 minutes, with the bulk of it (~20 minutes) dedicated to "test".
The actual test run starts at the 14 to 15 minutes mark.

Fixed, improved, ready to go.


DCOS-21638 use the mediator api on repository settings

This implements the mediator pattern for the Settings page using recompose + rxjs. In the end we had to implement 0 lines of code on the data-service 🎆 .
To test this just play with Settings > Package Repositories, add, remove and filter packages.
Come to me if you need valid repos to test, or just remove first and then you can add again.

[x] adequate with the API here #2841

Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

@jaxesn. At first, I planned to implement componentFromStream myself. That I would type the observables so you would pass an Observable<React.Component>. Also, the handler functions could also have better-defined types.
Now that I got everything from recomposing, and had to implement nothing, I might check the @ts/recompose and see if adds values for us.
I think types would be really helpful to implement the dcos-labs/data-service package that is going to be extracted from packages/data-service folder, but as for now, I have implemented nothing 💃

Oh okay, so we already had tests for that piece, I wasn't aware of that. Will review tomorrow 👍

Thanks, @DanielMSchmidt. I started the efforts on TypeScript tonight.

@Fabs please start also some efforts in the direction of sleeping :trollface:

@DanielMSchmidt. I have many clones :D

@DanielMSchmidt I tried to use enzyme yesterday to test this. I ran into an interesting problem @orlandohohmeier keep reminds us about. I added this ticket next sprint so we can address testing the integration between the React UI components and the streams.
https://jira.mesosphere.com/browse/DCOS-21792

@Fabs Ok, seems fine, let's pair on it next time 👍

@DanielMSchmidt. Thanks for for the pairing offer. if it is ok can you approve?
Just to add some more info. The ability to test the mediator properly depends on us being able to inject data dependencies here. Which is something that @orlandohohmeier and @nLight are looking at with inversify.js and etc. That is why It was hard to test (and not because of enzyme).
Regarding TS I think we are lucky we don't have it here, because it will still take a while to get it merged. (webbbpack -> ts -> tslint).

@Fabs I think we need to make sure our review cycles are faster. Taking them as a reason to not add type checking just doesn't feel right to me. Therefore I think I should not approve this PR, nor reject it :) Waiting for others to air their opinion :)

@DanielMSchmidt typescript is a nice to have for the data layer, not a must-have. I decided to add it because it looked like a quick win and it would be beneficial to the entire team. I would see it more like as a choice I would have to make something better, and less as an obligation to deliver it on typescript. Especially given the goals that this project has to achieve to enable MT.
The types we will have to describe in grpahql schemas, is the real must have for the Data Layer.

@Fabs Agreed, I didn't saw this as testing the mediator but more as the first step and I didn't want us to just start with JS when we could use TS instead. I see your point with fast iteration cycles, I just hope we can really leverage TS here in the long term

@DanielMSchmidt. I think we are aligned on this wish. I would love if the data-service (the package we will extract the data-layer implementation too) is entirely written in .ts.
Thanks for challenging and for the help :D.

Apparently the fail is legit, will look into it.

This PR provided a great discussion. We have a second version already that supplements this one, so this PR is no longer necessary:
https://github.com/dcos/dcos-ui/pull/2842


Correcting the example scenario because v1 does not exist anymore

fixes #360

Removed all the lines mentioning mode

..including mode: managed smile
Done . But i observed that it is present  in almost all the scenarios that we currently have. So somebody needs to bring it to notice.

..including mode: managed smile

Done . But i observed that it is present in almost all the scenarios that we currently have. So somebody needs to bring it to notice.

It's a deprecated option - it doesn't hurt to state it, but it's also not necessary. Feel free to open an issue for the options removal!