Major change in stats percentages

Describe the bug
hampus_sethfors on Slack:

I just noticed that the iOS dark mode stats on appt.org has dropped from 35% a couple of months ago, to 18% now. Feels strange, anyone know what’s happened?

To reproduce
Visit https://appt.org/en/stats/dark-mode
Expected behavior
The stats shouldn't change this much.
Screenshots
If applicable, provide screenshots, they help us understand the bug.

System
Any browser.
Additional context
Aug 23 is when the drop happened to 17.59%
On Aug 22 it was 35.17%
{"name":"Ios - Preference ui style - Dark","feature_key":"preference_ui_style","property_key":"dark","platform":"ios","amount":1597286,"percentage":17.59}

vs
{"name":"Ios - Preference ui style - Dark","feature_key":"preference_ui_style","property_key":"dark","platform":"ios","amount":1597105,"percentage":35.17}

The percentage seems to have halved: 35.17 / 2 = 17,585 which would round to 17.59
Other stats seem to be unaffected based on 1 check of bold text
iOS bold text on Aug 23
{"name":"Ios - Accessibility is bold text enabled - True","feature_key":"accessibility_is_bold_text_enabled","property_key":"true","platform":"ios","amount":632002,"percentage":9.92}

iOS bold text on Aug 22
{"name":"Ios - Accessibility is bold text enabled - True","feature_key":"accessibility_is_bold_text_enabled","property_key":"true","platform":"ios","amount":631864,"percentage":9.92}

If 631864 is 9.92% then total amount of datapoints is 6.369.597
If dark mode has 1597286 then percentage of 6,3M = 25% That doesn't match 17,59 nor 35,17 :thinking_face:
And if 1597286 is 35,17% then that gives 4.541.615 datapoints, with 17.59% it's double so around 9M
When investigating this bug, I would like to request a new feature of the total amount that's used for division to get the percentage to be included in the JSON files. The addition of this field won't break any of the current appt.org code.

Hi! Johan requested for me to look into this! Thanks for the detailed reporting. That's a helpful start!
It would appear that one of the organisations didn't propagate any occurrence of the dark value for the field Preference_UI_style in the past. Since 21-08-2024 we're starting to see this data coming through which explains why we're seeing the stat for preference_ui_style change. The reason why this is changing so much is because this value competes with a lot of occurrences of the light value from the past.
In our real world example the data we were first seeing was contributed to only by org 1 and thus the percentage would be ~35%. Since the addition of data from org 2 this percentage is effectively being halved since it started with such a low amount of data. This amount is slowly increasing as the update with support for dark mode is becoming more common. The number of dark mode users is still low though on the side of org 2. I believe this is because the current source for this data is the iOS app settings instead of the iOS general accessibility settings.
This table contains all usage of the dark value both in absolute terms and as a fraction of the total usage of the Preference_UI_style field for the org that just worked on this. It shows that the fraction has increased since the 21st of August but since the total occurrences of the dark value is still very small in historic context we're not seeing this having an effect on the total fraction.
So three main takeaways:

The dark value was recently implemented and this is why we see the sudden drop
The preference ui style field is likely not implemented correctly by the same org that just implemented the dark field
Since the dark value is recently implemented it has a long way to go to impact the historic statistics. This can be considered a weakness of the chosen aggregation. We've previously discussed a rolling window of a year to clean up older data, which would alleviate this issue somewhat.

I'm going to discuss with Johan what the most suitable solution is both short term and long term. I'll also discuss the proposed feature to add as a feature: the total amount that's used for division

Thanks for the update @jorisbr ! I think the biggest problem is the usage of app settings instead of system settings. For these statistics, we want to track global user settings and not app specific user settings. I think the new org should change the source of their dark mode % to the Android/iOS system setting field.
Plus the other things you mentioned would also increase the accuracy of the stats. Thanks and have a nice weekend :)

Hi @jorisbr , the iOS stat is back to 36% now, but Android has dropped to 14%. Maybe the same org has now adopted the stats libary in their Android app?


Update logos for Frigg and HubSpot

Fresher Frigg logo and new format for HubSpot logo url

Task linked: FRI-428 Update logo in CFA frontend to be the more recent Frigg logo


Added the ability to set table and rows flags

Currently, generating migrations using the CLI isn't all that helpful as it uses the migration name as the table name. This PR fixes that by adding a --table flag which will set the table name. If it's not defined it will revert to it's default behaviour of using the migration name. This feature was requested by @cannap in gitter.
This PR also adds an additional flag, --rows, which accepts a comma separated list of rows to be added and their properties. The structure should look something like this
--rows rowName:rowType:[optional additional attributes]

The attributes that have been added are required, nullable, index, primary, and unique. I'd like to find a way to do default as well, but I couldn't think of a good way to format it.

Coverage increased (+0.5%) to 46.365% when pulling 9f159a47c7f4478bb04c7784c78d6e4d8e8d3c37 on watzon:develop into 9d6de967250e9da3bd129fcdfda8953b827b5e62 on adonisjs:develop.

Coverage increased (+0.5%) to 46.365% when pulling 9f159a47c7f4478bb04c7784c78d6e4d8e8d3c37 on watzon:develop into 9d6de967250e9da3bd129fcdfda8953b827b5e62 on adonisjs:develop.

I am okay with the table flag but not the --rows flag, since it is very limited but will open a channel to keep on making it complex as everyone's requirements will evolve.

@thetutlage my thought was to just support the types that knex.js offers and leave it with a very simple API. This is a feature that Rails has that I've missed in both Laravel and Adonis

I believe having a separate package that adds a new command will be nice.
When I don't fully believe in something (doesn't mean bad) I suggest users to create separate packages for them and simply offer it as a community addon.
It not only helps other but helps me too, since I don't have to maintain the code, I never believed in, but doing it for the sake for stability.


Update 2.json

@kristinaleko This PR has a conflict in 2.json with the last PR you provided. Which of the two is correct?

@daniel-eder yes right sorry i have corrected now on my fork for pull this one i'll delete. The mistake was mine, i created changes on the mail on json 2 and not of fork. Sorry for this.

Will be resolved in #85


Update ReadME

Hi Anson, ChatGPT thought it is necessary for your repository to have a readme file. so i made one!

！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！

ok i will use kimi ai

please merge my pull request
it fixed lots of problems

Why not WTFPL
I love this LICENSE

I'm sad because u reject my pr :(

I'm sorry about that


Support seeding a new repository from an existing one

This feature would allow a user to initialize a new Corso repository and seed it with an existing backup from a different repository.
Seeding it with an existing backup would allow new backups to continue as incrementals.

If we added a generic backup migrate command to support this use case - it would also enable other use cases that require backup copies (moving storage providers, etc.)


add new AuthConfig/discoveryDoc behavior for urls

This will add a new option (overwriteDiscoveryDocumentUrls) to change behavior how the settable URLs from AuthConfig will be handled in relation to the discovery document.
maybe fixes #1161 too

Thx for the PR! I will mention I still have my reservations about this approach but it's up to the maintainer to make a judgement call here I think.

My use case differs form #1161:
I want to overwrite the logoutUrl, because my sidecar oAuth Proxy will listen for e.g. '/oauth/logout/' and then forward logout request to OIDC-Provider by its own. This must be done to dispose session store within oAuth Proxy. An interception of the direct logout path for OIDC-Provider will not work because of different routing paths.

Cheers, makes sense.
My main worry is that this library already has to many "custom scenario xyz" style flags and ternary/if-else bits of logic. This would add one more such feature, and there's a tradeoff as it introduces quite a bit of extra code and possible bugs (not necessarily yours, but more likely in future changes that build on this).
This is not to invalidate your use case, or the need to handle that situation.
I'd prefer that the library (or Angular) introduces interception points at the right places, for example in the way you can inject your own OAuthStorage implementation. So instead of a flag, something like a interface DiscoveryDocumentParser injectable and a default implementation that you could tweak. This is harder to write a PR for, but I do feel it leads to a more stable library while at the same time enabling more use cases than just the one in front of us.
But that's just my 2 cents of course!

... This is harder to write a PR for ...
Would prefer such an solution too, but that would be too big for me at my first PR for this project 😅

But maybe my PR is suitable for an intermediate solution for these two use cases.

No feedback from maintainer @manfredsteyer since one year, will close this PR within next few days...

closed because of no feedback from maintainers


Add Pattern Match and Rewrite Library For Extensions

Signed-off-by: silverstar195<EMAIL_ADDRESS>Commit Message:
Add Pattern Match and Rewrite Library For Extensions Additional Description:
Additional Description:
Added extension library for implementation of url pattern matching and rewrite.
This library will be called by both extensions as needed.
Full set of changes to be merged is mostly complete and tested. It does still need cleaning up. I'll be splitting PRs off of it here:
https://github.com/envoyproxy/envoy/pull/22207
Risk Level:
No changes to existing functionality within this PR.
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/assign @stevenzzzz @htuch @lizan @mattklein123 @edin44

/assign @alyssawilk

/wait on CI.
 I think you're going to want to run format checks locally before pushing upstream - it'll save you a bunch of cycles

All presubmits are passing except. Tracking 2nd owner now
ERROR: Extensions require at least 2 owners in CODEOWNERS:
/*/extensions/path/pattern_template_lib @alyssawilk

Should we close this off in favor of the larger PR?
/wait

Closing out for full PR: https://github.com/envoyproxy/envoy/pull/22207


Unable to minify code from discount_lodash.js

While trying to build the react project inside a docker container, there seems to be a bug which fails to minify code from discount_lodash.js
Steps to reproduce.

Create a docker container with React project with react-dnd dependency and build react project.

This seems related to #1389, can you provide some more details on the errors you're seeing?


[BUG] Wrong label for first set of advanced moves

Describe the bug
The Advanced Moves should be available from level 2-5 (which has been written in the past) but it is currently written from level 1-5.
To Reproduce
Steps to reproduce the behavior:

Go to Battle screen, or create custom class view
Tap on '+' to add move at the appropriate location
The level titles are '1-5' and not '2-5'

Expected behavior
Should show 2-5.
Screenshots

Smartphone (please complete the following information):

Device: N/A
OS: N/A
Version: At least 1.1.4 and above

Additional context
Reported via email.
References https://github.com/DungeonPaper/dungeon_world_data/issues/4

Fixed in 3d38f7ff1efeb6d1b6ddd1d50ba15331a923ee50.


[docker] remove circleci base image, point dependabot at gha base image

Motivation
remove circleci base image, point dependabot at gha base image
Have you read the [Contributing Guidelines on pull requests]
Y
Test Plan
CI
Related PRs
None

Thank you for your submission. We require all contributors to
sign the CLA before we can accept your contribution.
Have you signed the CLA already, but your status is still pending? Recheck CLA

CLA bot broken?

/land


Fix: core js need to be v2 after update

corejs is upgraded to v3, need to add a fix version for v2 build

Coverage remained the same at 89.583% when pulling 98aa81f661d41e915658dc72f6ac4b506c4774cf on j0eii:fix/201909-core-js-dependancies into<PHONE_NUMBER>bbab308eeb4ea7e89e84db99c14ee8 on aichbauer:master.

I'll take a look into it...

i tried, it need a fix version @2, otherwise it will throw error @3..


Conditional Formatting is breaking existing formats

Nothing was being shown, and I can't close dialog after making changes.
No obvious related messages in the console

Duplicate of gh-4632


mediainfo-gui: 23.04 -> 23.06

Description of changes
https://mediaarea.net/rss/mediainfo_updates.xml
Related：
https://github.com/NixOS/nixpkgs/pull/242372
https://github.com/NixOS/nixpkgs/pull/242105

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

23.07 is already out again 😁


adds #find and #findBy methods for collection

Coverage increased (+0.003%) to 98.758% when pulling 0c2d61ba0ad807570820c9f45c1f783ea001f7ed on yratanov:add-find-method-to-collection into 1dac34d70d7125f88ebdbc0a4278d57a58f1dade on san650:master.

Thanks for the PR @yratanov!
Yes, I think currently we miss an easy way to find a single elemement for collections. There were some discussions in this area already. When new collections API was developed, @pzuraq raised an idea of findOne:
-New Collections API - https://github.com/san650/ember-cli-page-object/issues/352

findOne possible design - https://github.com/san650/ember-cli-page-object/pull/347#discussion_r160061630

I feel, in scope of page object collections, it may make sense to provide a bit more clever form of find, which finds exactly one item or throws error if only 0 more than 1 items was found.
What do you think? does it make sense for you?

@ro0gr I've added asserts for:

finds exactly one item or throws error if only 0 more than 1 items was found.

Not sure about findOne though. All other methods are native, so it will be confusing having filterBy that accepts field, value and findOne/findAll with completely different API. They should all use same arguments IMO.
What do you think?

findAll was supposed to behave the same as filter does, but the idea was to give it a power to accept query objects, like:
  myCollection.findAll({
    someProp: 'some text'
  })

As for me, even if it seems a bit more convenient, this query objects are quite limited:

what about nesting
"greater(less) than" or anything more advanced  than equality check
larger api surface

In other words, in my opinion, findAll is not a necessary part of collections, since filter is more flexible and well known for the most of the devs already.
findOne on the other hand, is a bit different from find for a good reason, it allows developer to get a quick feedback if something goes wrong. If we let find throw, it could confuse a developer, cause that's not how regular find works. That's why I think highlighting the new behavior via a different name is a good idea.

Also I'm not sure about the findBy...
I understand we already have filterBy and mapBy, but:

afaik there are no such methods in native JS arrays. These are methods which are added by Ember to the Array prototype. For instance, if someone installs ember-disable-prototype-extensions in their app or addon our mapBy/filterBy would not work.
After we got arrow functions it does not really make more convenience to me to use ***By(key, value) extensions:

myList.findBy('id', 3);
// vs
myList.find(i => i.id === 3);

maybe that's just a matter of taste or I miss something, but in my opinion, a single findOne(or something like this) would be sufficient.

In other words, in my opinion, findAll is not a necessary part of collections, since filter is more flexible and well known for the most of the devs already.

👍

findOne on the other hand, is a bit different from find for a good reason, it allows developer to get a quick feedback if something goes wrong. If we let find throw, it could confuse a developer, cause that's not how regular find works. That's why I think highlighting the new behavior via a different name is a good idea.

I don't think assert is such a big deal so that we need to type three more characters. find/findBy are also instantly recognisable, so users will know what args to pass without looking into docs.

After we got arrow functions it does not really make more convenience to me to use ***By(key, value) extensions:

I use ***By methods a lot and find it more convenient... I think it's pretty common in Ember to know/use these methods.

I don't think assert is such a big deal so that we need to type three more characters.

Sorry me for being that guy, but I can't agree. Though find and findOne may feel similar, they are different. It introduces 2 new edge cases.
With findOne we:

throw if no elements found. I suspect I could accept it mentally under the find name, if it were the only case, but
throw when there are multiple matching elements on the page. On the other hand, selecting the first element from many is completelly fine for a native find. I think this is even less intuitive than throwing when no elements found.

Because of these ^ edge cases, I think findOne can't be considered substitutable with find.

so users will know what args to pass without looking into docs.

We can improve learnability by fisnishing typings effort. Also findOne and find have the same interface but exceptions.
Hope it makes sense to you.

@ro0gr ok, then what do we do? I really would like to keep ability to find by property, value.
findOne and findOneBy?

@yratanov yes. let's do it, if that's fine for you!
/cc @san650

@ro0gr renamed :)

@ro0gr applied your feedback 👍

Thanks!

@yratanov sorry for asking so late, but would you mind to add yourself to the AUTHORS please?


Add newsletter links

Signed-off-by: James Cole<EMAIL_ADDRESS>Adds newsletter links to nav and footer.

We should think about how to give the newsletter signup a bit more attention in the near future, but think this is a good start! Merging


LGTM and Appveyor should not run when only /doc path is touched on PR

On a recent Draft Pull Request, I noticed that LGTM and Appveyor both ran, even though only files under /docs where touched.
We need to tweak our CI/CD workflows to exclude the /doc path as documented here: https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#example-ignoring-paths
It also might make sense or be easier if we split up the CI/CD into multiple workflow files and then reference them as needed.  GitHub presents this as a solution for advanced job/step handling, instead of trying to do everything in 1 workflow file.
We might have a "update-docs.yml" file, along with others as needed.
To Reproduce
Steps to reproduce the behavior:

Submit a PR or Draft PR with only change a file under /docs and notice LGTM and Appveyor fail with errors, but Netlify then continues.

Current Results
LGTM and Appveyor are not skipped
Expected Behavior
LGTM and Appveyor steps/jobs should be skipped entirely when only /docs path is touched.
Screenshots

Versions

Operating System:
Browser Version:
JRE or JDK Version:
OpenRefine:

Datasets

Additional context

I don't think we should tag CI-related issues as "good first issue" or "help wanted" since it's not so easy to fix them when you are new to the project (people could need some privileges to access the corresponding services and configure them).

This seems to be solved already, for instance see https://github.com/OpenRefine/OpenRefine/pull/4280


There is a typo in the config.php

https://github.com/SzuniSOFT/laravel-szamlazzhu/blob/master/config/config.php#L11

Pull request with the fix: #3

Thanks.
Missed that one. Sorry!


where is manjaro's main issue tracker?

I'm new user of manjaro Since 16.06, want to report new bug while the main site https://manjaro.github.io/ is not telling where is entry of bug tracker? if just using github, which repo is the main entrance? because I'm not sure which repo should it belong to under this github organization

For now we are using github and users should post the depending repositories. If the package is not listed in one of our packages then it is maitained by Archlinux itself. What issue would you like to post?

have filed manjaro/manjaro-tools-iso-profiles#164 but there are more issues with this distro
relying on github issue tracker's main problem is there are multiple entrances under this github org, even worse is some repos got deprecated (like manjaro/manjaroiso#48 why not delete); this keeps confusing people don't know where to file issues
More issues I found is with displaying, maybe in driver, because some specific portion of the screen is flashing, intermittently I compared same hardware running Ubuntu 16.04 doesn't have the problem, should I record a small video to describe ? I am running live iso mode, more details I have written in https://github.com/manjaro/manjaro-tools-iso-profiles/issues/164#issuecomment-229496352

I see https://manjaro.org/2016/07/24/manjaro-has-now-a-new-bugtracker/ and https://bugs.manjaro.org/ is a nice move; this can be closed now.


cant type in search box

right after running the windows10debloater i restarted my pc then i couldnt write in the search bar and inside the file explorer (quick access top right)?
any help i have tried everything

@b3ns-creator - Can you please try running the Revert Registry Changes button? It may resolve the issue. Also, at that time I believe there was an issue with a Microsoft Windows update.

For me the fix was enabling and starting the service "Touch Keyboard and Handwriting Panel Service".


DragTarget.onLeave not called when inside scrollable

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
It seems DragTarget.onLeave only calls if pointer moves, but scrolling can also make pointer leave a dragTarget.
On desktop, one can scroll without moving the pointer.

I have a scrollable ListView containing DragTarget
Drag and hold a draggable on top of DragTarget.
Scroll the mouse wheel without moving the pointer, so that DragTarget is not under pointer. onLeave is not called.
If release the mouse button, onLeave still not called; If move the pointer a little, onLeave is called.

sample code:
Code Sample

import 'package:flutter/material.dart';

void main() {
  runApp(const TestWindow());
}

class TestWindow extends StatelessWidget {
  const TestWindow({super.key});

  @override
  Widget build(BuildContext context) => MaterialApp(
    title: 'test dragtarget',
    theme: ThemeData(
      useMaterial3: true,
      colorScheme: ColorScheme.fromSeed(seedColor: Colors.green),
    ),
    debugShowCheckedModeBanner: false,
    home: Scaffold(body: _buildBody()),
  );

  Widget _buildBody() => ListView(
    children: [
      Draggable<int>(
        data: 100,
        feedback: Container(
          width: 200,
          height: 100,
          color: Colors.purple,
        ),
        child: Container(
          height: 100,
          color: Colors.red,
        ),
      ),
      DragTarget(
        onMove: (DragTargetDetails<int> details) {
          print('moving at ${details.offset}');
        },
        onLeave: (_) {
          print('left target');
        },
        builder: (BuildContext context, List<dynamic> accepts, List<dynamic> rejects) => Container(
          height: 200,
          color: Colors.green,
        ),
      ),
      Container(
        height: 300,
        color: Colors.yellow,
      ),
    ],
  );
}

Proposal
I think onLeave should trigger as soon as pointer is no longer above DragTarget, no matter of whether pointer moves or not.

Hi @songziming. Thanks for proposing this. I would appreciate it if you could provide a screen record of this behaviour as I can't seem to reproduce it with the description above.

@dam-ease This is a screen record on my machine.
In the video, I paused a little bit after scrolling, pointer not moved, and you can see onLeave was not called. Once I move the pointer, onLeave was called.
Note that default flutter window size is 1280x720, which is too big for this sample. You have to resize it smaller first, then you can scroll it.

Hi @songziming. Thanks for your response. Checking this on macOS, I can't seem to reproduce this.
https://github.com/flutter/flutter/assets/53122008/0d4dbe81-fb2d-4911-adca-f81acf7d0cb2
Can you upgrade to the latest version of Flutter to check this on your end as well?

Hi @dam-ease, I'm using flutter latest stable channel:
Flutter is already up to date on channel stable
Flutter 3.13.6 • channel stable • https://github.com/flutter/flutter.git
Framework • revision ead455963c (7 days ago) • 2023-09-26 18:28:17 -0700
Engine • revision a794cf2681
Tools • Dart 3.1.3 • DevTools 2.25.0

In your video at 0:07, you moved the pointer downward, from green to yellow.
To reproduce, you need to keep pointer position (relative to window) not change, and use the mouse wheel to scroll the list.
Using a physical mouse helps.

Happens on my machine as well (macOS, Flutter 3.22.2).

I encountered a similar problem, here is my solution:
Copy flutter/src/widgets/drag_target.dart to project
// Add a Controller to manually trigger hitTest
class DraggableController {
  _DragAvatar? _dragAvatar;

  void _attach(_DragAvatar dragAvatar) {
    _dragAvatar = dragAvatar;
  }

  void dispose() {
    _dragAvatar = null;
  }

  void hit(Offset position) {
    _dragAvatar?.updateDrag(position);
  }
}

// Create a callback that returns the controller

typedef OnDragStartedCallback = void Function(DraggableController controller);

// Modify the _startDrag method of _DraggableState

  _DragAvatar<T>? _startDrag(Offset position) {
    // ...
        widget.onDragStarted?.call(DraggableController().._attach(avatar));
    return avatar;
  }

your_list.dart
     // Save the Controller and draggingOffset in the callback, and call it later when the list is scrolled.
     _draggableController?.hit(_draggingOffset!);


Update pod-security-policy.yaml remove formatting issues

When we try to apply this file with kubectl apply it shows some formatting errors, this fixes them.

Reason for Change:

Requirements

[x] squashed commits
[ ] included documentation
[ ] added unit tests and e2e tests (if applicable).

Issue Fixed:

Does this change contain code from or inspired by another project?

[ ] Yes
[ ] No

If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers:

@616b2f Thanks for opening PR. Could you add e2e test for this? Currently we run e2e test by installing provider via helm charts.
We can modify e2e_test_suite to install provider from deployment dir and run entire suite.


[notification hubs] SDK cannot parse valid JSON response

Package Name: @azure/notification-hubs
Package Version: 1.0.0-beta.7

Describe the bug
When retrieving an installation via getInstallation, a RestError is thrown stating the response body could not be parsed. The response body is valid JSON format. After debugging, I found the response headers contain a Content-Type of application/xml even though the response itself is JSON. I confirmed the outgoing request headers also contain the Content-Type of application/json.
To Reproduce
Steps to reproduce the behavior:

Create an installation
Use the getInstallation method to get the installation.

Expected behavior
The SDK is able to parse a JSON reponse and not throw a RestError
Screenshots
Request:

Response:

Additional context
Add any other context about the problem here.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Cosmos:0.08663254,Storage:0.06025839,KeyVault:0.057102457'
Label prediction was below confidence level 0.6 for Model:CategoryLabels: 'Client:0.59070516,Mgmt:0.3959275,Service:0.009578549'

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Cosmos:0.08663254,Storage:0.06025839,KeyVault:0.057102457'
Label prediction was below confidence level 0.6 for Model:CategoryLabels: 'Client:0.59070516,Mgmt:0.3959275,Service:0.009578549'

@bm77525-kr The Azure Notification Hubs service team is aware of the issue and will sync back here once we have an answer.  cc @marstr

@bm77525-kr The Azure Notification Hubs team has been able to reproduce the issue and are working on a fix.  Update here when the fixed has been rolled out

Closing as fixed per customer


Output added to TestRun should show the last value appended and not the first one

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.71.1
OS Version: Windows 10

The issue is that when a user does a test launch and multiple outputs are appended to a TestRun in the same line, the first one will always be shown, when the correct would be showing the last value (see image attached).
Steps to Reproduce:

Make a run which has multiple contents added to the test output (i.e.: TestRun.appendOutput()) in the same line/test
Check that when multiple contents are added to the same line the first one added is always shown (when it should be the last one).

The image below shows the issue:

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.71.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

The issue persists in the latest stable release.

Thanks for the fix ;)


[ALLUXIO-2867] rename HelpCommandTest to HelpCommandIntegrationTest

https://alluxio.atlassian.net/browse/ALLUXIO-2867

alluxio-bot, check this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15852/
Test PASSed.


Add groups to IAM user

Is your feature request related to a problem? Please describe.
Feature request to enable a resource to add groups to a user.
We are currently using the dynatrace_iam_user to create users and associate their groups.
We are also using SCIM for SSO authentication and have 2x challenges:

User logs in before we run the terraform - causes the TF to fail (duplicate)
Pipeline taking 30+ mins to run with around 250 users.

As such we are looking for an alternative approach which:

We allow the SCIM process to create the DT user automatically on login
We change our TF to NOT create the user, but to 'add groups to existing users'

As such we'd use our config data file as the source of email addresses, but we'd need a way of adding a list of groups (which are terraformed vi dynatrace_iam_group) to these users.
Describe the solution you'd like
A new IAM resource like dynatrace_iam_user_group which would utilize this DT API: https://docs.dynatrace.com/docs/dynatrace-api/account-management-api/user-management-api/put-user-groups
Something like:
resource "dynatrace_iam_user_group" "iam_user_group" {
    for_each = local.users

    email = each.value.email
    groups = local.groups[email]
}

Hello @edfenergy-stevebowerman,
What you've described makes perfectly sense. With the next release the resource dynatrace_iam_user will no longer error out in case a user with that email already exists. Instead it will just update the group memberships.
The fact that your pipeline is taking 30+ mins to run is likely the result of a rate limiter we had to introduce recently, when api.dynatrace.com is being accessed. We've noticed an unusual amount of 504 Gateway Timeout responses coming back from that endpoint and the only way to cope with that was to ensure that the provider doesn't send more than one request per second.
I've been told, that the issues with the gateway should be resolved meanwhile, but I didn't have time yet to try that out.

Brilliant, thanks again Reinhard. Awesome work


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
cde7189 -> 00d2055

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
minor
v1.26.3 -> v1.27.1

terraform-ibm-modules/common-pipeline-assets
action
minor
v1.19.0 -> v1.20.0

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.27.1
Compare Source
Bug Fixes

deps: update module github.com/ibm/platform-services-go-sdk to v0.56.2 (#747) (d6dbbcb)

v1.27.0
Compare Source
Features

delete iam policy (#745) (82603f4)

v1.26.5
Compare Source
Bug Fixes

deps: update gomod (#743) (1ab2e92)

v1.26.4
Compare Source
Bug Fixes

deps: update gomod (#738) (a9275e6)

terraform-ibm-modules/common-pipeline-assets (terraform-ibm-modules/common-pipeline-assets)
v1.20.0
Compare Source
Features

exposed the ability to pass TOOLCHAIN_ID in cra-config.yaml (#635) (d128eca)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

:tada: This PR is included in version 1.0.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Difference between Polymer and this mod?

Just curious, seems the same goals as polymer. https://modrinth.com/mod/polymer Maybe add this question to your FAQ or the pinned issue as well?

Copy of #2
This mod is on temporary hold until I get more free time. But a lot of the stuff mentioned in #2 apply here. Polymer is targeting modpacks while Vanillify is suppose to be more of a "Server-Side Library" that allows modded items server-side without any other targeted environments. Its still extremely early, but I eventually intend it to be a fully-fledged API similar to Spigot/Bukkit but within the FabricMC Ecosystem of course.

Hmm, I think the goal might be more similar than you think. Polymer isn't for modpack devs or players, it's also a serverside library mods can use if they want to support vanilla clients. Perhaps some combining of efforts might be in order, or at least some closer inspection of that mod. It also has your 2 future plans bullet points implemented already.

Hmm, I think the goal might be more similar than you think. Polymer isn't for modpack devs or players, it's also a serverside library mods can use if they want to support vanilla clients. Perhaps some combining of efforts might be in order, or at least some closer inspection of that mod. It also has your 2 future plans bullet points implemented already.

Ive had some issues with the author of that mod personally in the past, so this one was started with the intent to implement features that polymer was missing. I guess you're somewhat right: Polymer does do a lot of the stuff now that Vanillify was planning to do. Im not really sure what to do with the project currently.


Can I ask which version of ldc2 you last built with?

I ran into a few errors when trying to build with the build.sh script and ldc2 version 2.109.1 (llvm-18.1.8). The first one was:
Building reng 0.20.0-beta3: building configuration [lib-standard]
source/re/core.d(50,36): Error: undefined identifier `InspectorOverlay`
Error ldc2 failed with exit code 1.

Then when I just imported the file where InspectorOverlay was defined, I ran into some other errors, unfortunately. Do you think this is a newer compiler error? Or perhaps something wasn't pushed with the last updates, that affected building?

Oh, hmm, it appears to be a 'release' problem. Things build and the celtest demo runs in debug mode with both dmd and ldc2. Not sure why, but figured you may have an idea why this doesn't build in release mode ;)

I know what the issue is, probably accidentally wrapped some stuff in a debug block. I can fix it soon

Fixed in https://github.com/bmchtech/rengfx/releases/tag/v0.20.0-beta4 with https://github.com/bmchtech/rengfx/commit/075df457f4c391002d752acb9052394cb5ef3c63

Great, thanks for the quick fix, works now ;)


Update globalnet related doc

With https://github.com/submariner-io/submariner-operator/pull/323
merged, user can now specify global cidrs for each of clusters.
Update related docs to reflect this info.
Closes: #98
Signed-off-by: Janki Chhatbar<EMAIL_ADDRESS>

@Jaanki please check the comments (about the --globalnet-cidr parameter on quickstart) and conflict

@Jaanki can you check the comments?. Other than that I think it's ready to merge.
Thank you! :)


Relations between buckets expressed with pointers ?

Disclaimer: I'm throwing an idea here, no clue if it is possible / relevant... anyway:
When using a DB such as storm as a persitance layer, you inevitably find yourself in a situation where you want to have relations between your various structs.
I think it would be nice to be able to declare a field of a struct as foreign (given it is a valid storm struct), and when saving it storm would store that field in a separate bucket.
type Line struct {
	ID   int `storm:"increment"`
	Text string
}

type File struct {
	ID    int     `storm:"increment"`
	Lines []*Line `storm:"foreign"`
}

func main() {
	db, err := storm.Open("stuff.db")
	if err != nil {
		log.Fatal(err)
	}

	l1 := Line{Text: "first line"}
	l2 := Line{Text: "second line"}
	file := File{Lines: []*Line{&l1, &l2}}
	db.Save(&file)
	// would store l1 and l2 in a "Line" bucket

	var l3 Line
	db.One("ID", 1, &l3)
	// l3 == {1,"first line"}
	l3.Text = "modified first line"
	db.Save(&l3)

	var file2 File
	db.One("ID", 1, &file2)
	// file2 == {1, {"modified first line", "second line"}}
}

Another suggestion along that line: if a field of a struct is itself a struct and has an ID field, then automatically store that separately, and store the type + ID when serializing the database.


Provide a flag to break builds when publication has dynamic versions

One of the nice features from Gradle 5.1+ is the availability of resolution result for publications:
publishing {
    publications {
        withType(MavenPublication) {
            versionMapping {
                allVariants {
                    fromResolutionResult()
                }
            }
        }
    }

However, there are cases where plugins could modify or create a new publication, for example https://github.com/gradle/gradle/issues/10684.
It would be great if gradle provided a flag to fail builds if a publication has dynamic dependencies or unresolved dependencies before trying to upload artifacts.

I am not sure we should support specific use cases. However it might make sense to provide an API to validate publications before they are published but after all logic has been applied.

That would be great, too!

Hi @ljacomet , are there any plans for API to validate publications?


Update download-sql-server-management-studio-ssms.md

Docs Build status updates of commit 451b886:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/ssms/download-sql-server-management-studio-ssms.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Single dash long options if no shorthand defined.

I have had issues migrating from https://github.com/koding/multiconfig or other https://golang.org/pkg/flag/ based configuration to pflag. This is because existing users were using the long options with a single dash (-option vs --option). The pflag package does not seem to support this and just errors trying to interpret these as shorthand options (e.g., -option => -o -p -t -i -o -n).
To allow easier migration when not using the shorthand option feature, I made a small change to interpret single dash options as double dash options when no shorthand options are defined. This seemed the least intrusive and did not seem to mess up anyone using shorthand options while truly making this a drop-in replacement for the flag package which it was not really before due to this.

Just a ping on this. Any comments?

Your PR literally breaks the whole point of this package, and I can't imagine why the maintainers would merge it.  Drop-in replacement for the flag package is talking about the API, not the argument parsing algorithm.

I agree with @moorereason that this is not a desired feature.


Fix image animation for memberspec when there are children to the cluster

Previously, there was an issue on the memberspec node's animation when a cluster had more than one child. This fixes it for both Chrome and Firefox
Before

After

LGTM


Question:  How to add a separate .jar as a Java Agent and modify Docker image ENTRYPOINT args

I've successfully built and run the Kubernetes "flavor" of the Spring Pet Clinic Microservices project.  I'd like to add a 3rd party JAR file to the output image, but I am failing to see where I can the directives to download the desired Java Agent .jar file and modify the default ENTRYPOINT directive to add the -javaagent flag to the ENTRYPOINT.
After reviewing the parent pom.xml and researching the spring-boot-maven-plugin docs, it appears that all I would need to do would be to modify the Dockerfile that's the basis for each service's Docker image, but it doesn't appear that a Dockerfile is used when this command is invoked:
./mvnw spring-boot:build-image -Pk8s -DREPOSITORY_PREFIX=${REPOSITORY_PREFIX} && ./scripts/pushImages.sh

Based on how I understand the project is designed, the Dockerfile located at <project root path>/docker/Dockerfile is only used when this command is executed:
./mvnw clean install -P buildDocker

I think I'm losing the trail of breadcrumbs to understand how all the pieces fit together.  Any guidance on where I can define my additional requirements of adding an additional .jar file to the output image and modify the ENTRYPOINT directive in that image would be greatly appreciated.
I have been able to discover a means to download and place the Java Agent .jar file into a specific module's target path with the following addition to the module's pom.xml.  This case, I added to the spring-petclinic-customers-service module with the following xml snippet:
<plugin>
                <groupId>com.googlecode.maven-download-plugin</groupId>
                <artifactId>maven-download-plugin</artifactId>
                <version>1.0.0</version>
                <executions>
                    <execution>
                        <id>fetch-java-agent</id>
                        <phase>generate-resources</phase>
                        <goals>
                            <goal>wget</goal>
                        </goals>
                        <configuration>
                            <url>path-to-my-third-party-jar</url>
                            <unpack>false</unpack>
                            <outputDirectory>${project.build.directory}</outputDirectory>
                        </configuration>
                    </execution>
                </executions>
            </plugin>

I feel like this might be part of the overall solution, but I am not sure how this would be copied into the image, nor how to modify the ENTRYPOINT command.

Hi @LoPan455
It looks like your question is related to the https://github.com/spring-petclinic/spring-petclinic-cloud/ project and not this spring-petclinic-microservice project that does not provide k8s support. If you confirm, I propose you copy your question in a new issue
Regards

@arey You are correct, my mistake.  Thank you for the response.


feat: showcaseview for at_backup_keys

- What I did - added showcase view tooltip in backup Dialog
ticket link - https://github.com/atsign-foundation/apps/issues/600

@Sardar1208
We have validated backup key help text tooltip with at_widgets feature/tooltip branch on bellow configurations. Help text tool tip is working as expected.
Device Details
Device: iPhone XR
OS: iOS 15.5


table updated

Aren't this changes not also in  #117 @dustinsommerfeld ?
We could then close this and only use #117

Aren't this changes not also in #117 @dustinsommerfeld ? We could then close this and only use #117

Ohh thanks! I did not see that ☺


chore(main): release 1.21.0

:robot: I have created a release beep boop
1.21.0 (2024-12-13)
Features

math: add min and max functions (9ad149b)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.21.0
:sunflower:


Up 5017 - fix contextOverrides path not applied

…nd adding a logger to show in debug from which path bean are loaded

Checklist

[x] the individual contributor license agreement is signed
[x] commit message follows commit guidelines
[ ] tests are included
[ ] documentation is changed or added
[ ] message properties have been updated with new phrases
[ ] view conforms with WCAG 2.0 AA

Description of change

Fix the problem in gradle tasks that doesn't load overridden beans needed in some case like for groups using the smartLDAPGroupStore.

Hey @jgribonvald! :wave:
Rest assured this PR has not been forgotten. :relieved:
Many of the uPortal committers are focused on preparation for Open Apereo 2018 right now. :soon:
More folks should be available to help with reviews after June 6.

Evidence suggests this change is needed to run SmartLdapGroupStore in uP5.

If you watch the code I added a debug to list all beans discovered/loaded, I can remove it or move from debug to trace. But I think it could be an help to check if a customized bean is used.

Thanks @laurenra7! :bow:


Does not work, nothing happens when saving

Hi
I'm trying to make this plugin work with the following example :
test.uml
@startuml
Alice -> Bob: Authentication Request
Bob --> Alice: Authentication Response

Alice -> Bob: Another authentication Request
Alice <-- Bob: another authentication Response
@enduml

When I save the file, nothing happens.
For information, when I run the following command in the terminal :
java -Dapple.awt.UIElement=true -splash: -jar .vim/plugged/vim-slumlord/plantuml.jar -charset UTF-8 -tutxt test.uml
This generates test.utxt file correctly containing the ascii output of the diagram.
More information about my environment :
Mac OS High Sierra 10.13.1
~$ java --version
java 9.0.1
Java(TM) SE Runtime Environment (build 9.0.1+11)
Java HotSpot(TM) 64-Bit Server VM (build 9.0.1+11, mixed mode)

~$ vim --version
VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Nov 13 2017 14:41:09)
macOS version
Included patches: 1-1250
Compiled by Homebrew
Huge version without GUI.  Features included (+) or not (-):
+acl             +file_in_path    +mouse_sgr       +tag_old_static
+arabic          +find_in_path    -mouse_sysmouse  -tag_any_white
+autocmd         +float           +mouse_urxvt     -tcl
-balloon_eval    +folding         +mouse_xterm     +termguicolors
-browse          -footer          +multi_byte      +terminal
++builtin_terms  +fork()          +multi_lang      +terminfo
+byte_offset     +gettext         -mzscheme        +termresponse
+channel         -hangul_input    +netbeans_intg   +textobjects
+cindent         +iconv           +num64           +timers
-clientserver    +insert_expand   +packages        +title
+clipboard       +job             +path_extra      -toolbar
+cmdline_compl   +jumplist        +perl            +user_commands
+cmdline_hist    +keymap          +persistent_undo +vertsplit
+cmdline_info    +lambda          +postscript      +virtualedit
+comments        +langmap         +printer         +visual
+conceal         +libcall         +profile         +visualextra
+cryptv          +linebreak       +python          +viminfo
+cscope          +lispindent      -python3         +vreplace
+cursorbind      +listcmds        +quickfix        +wildignore
+cursorshape     +localmap        +reltime         +wildmenu
+dialog_con      +lua             +rightleft       +windows
+diff            +menu            +ruby            +writebackup
+digraphs        +mksession       +scrollbind      -X11
-dnd             +modify_fname    +signs           -xfontset
-ebcdic          +mouse           +smartindent     -xim
+emacs_tags      -mouseshape      +startuptime     -xpm
+eval            +mouse_dec       +statusline      -xsmp
+ex_extra        -mouse_gpm       -sun_workshop    -xterm_clipboard
+extra_search    -mouse_jsbterm   +syntax          -xterm_save
+farsi           +mouse_netterm   +tag_binary
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
  fall-back for $VIM: "/usr/local/share/vim"
Compilation: clang -c -I. -Iproto -DHAVE_CONFIG_H   -DMACOS_X -DMACOS_X_DARWIN  -g -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: clang   -L.             -L /BuildRoot/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.Internal.sdk/usr/local/libressl/lib -L/BuildRoot/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.Internal.sdk/usr/local/lib  -L/usr/local/lib -o vim        -lm  -lncurses -liconv -lintl -framework AppKit  -L/usr/local/lib -llua -fstack-protector  -L/System/Library/Perl/5.18/darwin-thread-multi-2level/CORE -lperl -F/usr/local/opt/python/Frameworks -framework Python   -lruby.2.3.0 -lobjc

Do you have an idea what could be the problem ?

For information, when I try to save an empty file in test.uml, I get the following error
Error detected while processing function slumlord#updatePreview[6]..<SNR>202_mungeDiagramInTmpFile[2]..<SNR>202_convertNonAsciiSupportedSyntax:
line    3:
"test.uml" 0L, 0C written

This means that the plugin is well installed and reacts to file saves, but it just can't inline the output inside the file :/

Nothing happened as well when I save the file, and I have set     let s:jar_path = expand("<sfile>:p:h") . "/../plantuml.jar" to the right path, but still it does not work , I am using Uubuntu 16.04

Can confirm @jon301  's results. @scrooloose  Are you still alive?

Loooks like this plugging stopped working all the sudden with lastest vim

Not working either :(   @scrooloose - thoughts please?

With empty file I see:

With a valid, populated file nothing happens & no messages available in nvim.

Well, it seems to be creaking along, semi-working & throwing errors at the same time for some diagrams:

Regression of this issue I think.

Hmm, the error messages have stopped appearing. It seems that once the temporary file under eg. /var/folders/w_/lkyx406x43v1_507sn2j31jc0000gn/T/nvimWdTnjz/1.utxt  has been created, it becomes stable thereafter.

@kinowarrior  how did you fix it ? how did you create the above file ?

@Slyces doesn't work for me

Same for me, nothing happens when saving a .uml file (either empty or with valid markup). Also no change when replacing the plantuml.jar file with one from the PlantUML website.

the syntax i was using must have been too complex,
i just copied an example from the plantuml website
i replaced it with this one: https://plantuml.com/sequence-diagram
@startuml
Alice -> Bob: Authentication Request
Bob --> Alice: Authentication Response

Alice -> Bob: Another authentication Request
Alice <-- Bob: Another authentication Response
@enduml

and it worked on save as expected

Hello, I also had this problem, but modifying slumlord.vim and changing the way the plantuml jar is invoked solved the problem for me - more precisely, invoking the jar in headless mode (adding "-Djava.awt.headless=true" option). I'll create a pull request and let the author decide whether I missed anything there.

I encount ths same problem.

@kdurant
For the sake of trying it out, can you please try the forked repo or to apply the linked PR? Hopefully that will solve your problem, too.


External blogpost comment

Hey team! Please add your planning poker estimate with Zenhub @BeierKevin @FloBaumi @roman533


Address in mailbox given does not comply with RFC 2822

Q
A

Bug report?
yes

Feature request?
no

RFC?
no

How used?
Symfony

Swiftmailer version
symfony/swiftmailer-bundle: ^2.6.4

PHP version
7.2.19

Observed behaviour
Hello, I am using Monolog to send emails when an error occurs:
        critical:
            type: fingers_crossed
            action_level: critical
            handler: grouped
        grouped:
            type: group
            members: [streamed, deduplicated]
        streamed:
            type: rotating_file
            path: '%kernel.logs_dir%/critical.%kernel.environment%.log'
            level: critical
            max_files: 15
        deduplicated:
            type: deduplication
            time: 120
            handler: swift
        swift:
            type: swift_mailer
            from_email: '%mailer_from_email_address% <%mailer_from_email_address%>'
            to_email: '%mailer_internal_email_address%'
            subject: '%mailer_error_subject%'
            level: info
            formatter: monolog.formatter.html
            content_type: text/html

As you can see I am using the format `Full Name<EMAIL_ADDRESS>in order to get a sender name but I receive this error:
PHP Fatal error:  Uncaught Swift_RfcComplianceException: Address in mailbox given [Full Name<EMAIL_ADDRESS>does not comply with RFC 2822, 3.6.2. in /home/luca/Sites/never_alone/vendor/swiftmailer/swiftmailer/lib/classes/Swift/Mime/Headers/MailboxHeader.php:345

Expected behaviour
The format is really valid, if you use a validator:
http://sphinx.mythic-beasts.com/~pdw/cgi-bin/emailvalidate
"Full Name<EMAIL_ADDRESS>is a valid email address.
Thanks!

The main reason this error is throw, your email given does respect the
email with comment, the system do expecte an add_spec token, so the error
message should be clearer
The error, specificaly reference the RFC 2822 Section 3.6.2 which should
accept an angl-addr like you provided.
the exception should reference addr-spec as defined in section 3.4.1 of
RFC-2822
Regarding monolog, i suspect you can have from_email and from_label (or
from name or comment).
On Fri, Jul 5, 2019 at 3:50 PM lukepass<EMAIL_ADDRESS>wrote:

Q A
Bug report? yes
Feature request? no
RFC? no
How used? Symfony
Swiftmailer version symfony/swiftmailer-bundle: ^2.6.4
PHP version 7.2.19 Observed behaviour
Hello, I am using Monolog to send emails when an error occurs:
    critical:
        type: fingers_crossed
        action_level: critical
        handler: grouped
    grouped:
        type: group
        members: [streamed, deduplicated]
    streamed:
        type: rotating_file
        path: '%kernel.logs_dir%/critical.%kernel.environment%.log'
        level: critical
        max_files: 15
    deduplicated:
        type: deduplication
        time: 120
        handler: swift
    swift:
        type: swift_mailer
        from_email: '%mailer_from_email_address% <%mailer_from_email_address%>'
        to_email: '%mailer_internal_email_address%'
        subject: '%mailer_error_subject%'
        level: info
        formatter: monolog.formatter.html
        content_type: text/html

As you can see I am using the format `Full Name<EMAIL_ADDRESS>in
order to get a sender name but I receive this error:
PHP Fatal error:  Uncaught Swift_RfcComplianceException: Address in mailbox given [Full Name<EMAIL_ADDRESS>does not comply with RFC 2822, 3.6.2. in /home/luca/Sites/never_alone/vendor/swiftmailer/swiftmailer/lib/classes/Swift/Mime/Headers/MailboxHeader.php:345
Expected behaviour
The format is really valid, if you use a validator:
http://sphinx.mythic-beasts.com/~pdw/cgi-bin/emailvalidate
"Full Name<EMAIL_ADDRESS>is a valid email address.
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/swiftmailer/swiftmailer/issues/1205?email_source=notifications&email_token=AAAME7OX2RSUHW4HXY7PCBTP55GSPA5CNFSM4H6LP3IKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G5SAB5A,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAME7IVD5I5TBMWDRMH6BLP55GSPANCNFSM4H6LP3IA
.

--
Xavier De Cock
GPG Fingerprint: 93CA EE3F 9F57 5BE1 AE4A  794D 3C74 CA9E E7A5 0C1B
GPG Id: 0xE7A50C1B

Unfortunately Monolog doesn't offer an option to set the from name. These are all the available parameters:
swift:
    type: swift_mailer
    from_email<EMAIL_ADDRESS>    to_email:
        -<EMAIL_ADDRESS>    subject: Test subject
    level: info
    formatter: monolog.formatter.html
    content_type: text/html
    priority: 0
    bubble: true
    app_name: null
    include_stacktraces: false
    process_psr_3_messages: null
    path: /var/logs/prod.log
    file_permission: null
    use_locking: false
    filename_format: '{filename}-{date}'
    date_format: Y-m-d
    ident: false
    logopts: 1
    facility: user
    max_files: 0
    action_level: WARNING
    activation_strategy: null
    stop_buffering: true
    passthru_level: null
    excluded_404s: {  }
    excluded_http_codes: {  }
    accepted_levels: {  }
    min_level: DEBUG
    max_level: EMERGENCY
    buffer_size: 0
    flush_on_overflow: false
    exchange_name: log
    message_format: text
    api_version: null
    channel: null
    bot_name: Monolog
    use_attachment: true
    use_short_attachment: false
    include_extra: false
    icon_emoji: null
    notify: false
    nickname: Monolog
    use_ssl: true
    title: null
    host: null
    port: 514
    index: monolog
    document_type: logs
    ignore_error: false
    config: {  }
    members: {  }
    headers: {  }
    mailer: mailer
    lazy: true
    time: 60
    deduplication_level: 400
    store: null
    client_id: null
    auto_log_stacks: false
    release: null
    environment: null
    message_type: 0
    tags: {  }
    console_formater_options: {  }
    nested: false

Maybe the request should be done there because your suggestion will break api.
And could "botname/nickname" be what you're searching?

Address in mailbox given [] does not comply with RFC 2822, 3.6.2.


Bug with gossip sync

Description
Weird channels names appeared after udpate OBW to v0.2.0-11.

Normal channel open with a clearnet node Eternal.Sovereigm, few weeks ago
OBW vers. 0.1.8
Also opened one with Jiraiya and one with SATM
using clearnet, never Tor

After update OBW to 0.2.0-11 some weird gossip sync happen:

Jiraiya HC appeared to be etleneum,com (I never open with etleneum.com and is also another IP)

Eternal.Sovereign node appeared now as [onion] with displaying his old onion address (no longer available)

Eternal said that only changed the alias few days ago and other wallets / channels are connected well.
Notice in Amboss its old onion address

Steps I did:

removed HC etleneum and added again Jiraiya as HC. Worked, all ok, channel online
Switched to Tor, but the normal channel with Eternal still shows the old address and stays offline.

Will wait few days more to see if the gossip will sync correctly and will report back here.

@Darth-Coin if you can do me a favor, please download the apk from https://github.com/nbd-wtf/obw/actions/runs/3810252555 and live with it for a while so we can see if this problem keeps happening. A proper fix would require doing a big refactor, but this should be enough if the problem is just the ACINQ node replaying very old gossip messages on us.

@fiatjaf I install it but is coming as separate new app not overwrite the old one.
The thing is that with the original OBW I can't make export of the channels to import in any other version. See issue #35
So I can't really test this dev version.
Let me know how should I proceed now. If I can make the backup in OBW I could import into this dev version and see how is doing. But I can't trigger the backup (I have 2 HC + 1 normal chan).
Yes, I could FC that chan (is with Eternalkr, no problem to communicate) but then we can't test to see this fix in v0.2.0 is ok.

I'll do that other thing.

Actually if you can just use this dev version with a random key and get an HC from Jiraiya and never use it for anything, just wait to see if it changes from Jiraiya to Etleneum eventually.

Ah OK then, I misunderstood. I can do that sure. testing it for a while.

I updated to new v0.2.1
made the backup channels
imported the backup + seed into an other device with v0.2.1
SATM HC channel OK
Jiraiya HC channel, imported as etleneum. Removed and add Jiraiya again, worked. OK
Eternal.Sovereign normal channel is still synced wrong as described in the beginning and is still offline.

Actually if you can just use this dev version with a random key and get an HC from Jiraiya and never use it for anything, just wait to see if it changes from Jiraiya to Etleneum eventually.

I check almost twice per day using OBW_DEV version, to see if Jiraiya is changing the name. It is not updating.
If you are still looking into fix this issue let me know. I can keep that broken channel open until you find a solution.
Maybe this will help others with the same issue.
But if you think there's no fix for this, also let me know so I can FC the channel and move on.

Well, if it is not changing back to Etleneum that is a good thing. Does the channel appear as online? I think you can stop checking now. Thank you very much for your help. Is the channel working in the main OBW too or has that one changed to Etleneum?

yes, the Jiraiya chan is active online and working on both versions (0.2.1 and dev)
Only the normal channel with Eternal node is still offline and showing onion address.

@fiatjaf this morning I tried again to use OBW_DEV version.
As usual, I open OBW, wait to sync the channel (I have only one HC with Jiraiya) and sent the payment.
I get this error, even that I wait several minutes and no sign on top of sync.

Then I tried from another device, the normal OBW (latest version) and payment worked.


Eliminate "libtool" buildInput for Crypto++

It appears that the libtool derivation pulled in previously is invalid. It lacks the -static flag that Crypto++ uses.
See #6150.

Apparently this doesn't work on all platforms... Though I'm not sure how to debug that build log.

Closing as this issue seems to be resolved in #7756.


Handling Logouts with Form auth

After some research, there is not a good way to do logouts on Quarkus form authentication, as they aimed the develoment towards OIDC, there are some workarounds, like for example here https://github.com/quarkusio/quarkus/issues/27389#issuecomment-1492970763 and https://stackoverflow.com/questions/68640209/quarkus-http-authentication-logout,
after searching the Vaadin documentation, the best bet would be to execute the javascript code with this
https://vaadin.com/docs/latest/advanced/browser-access#executing-javascript-in-the-browser and using the answer on the stack overflow post, i haven't tried it yet, but it shows some promise, another way would be to invalidate the cookie using vertx while tapping into the vaadin sesion or the post requests

Now I get the point. What we are doing so far is, when a user presses the logout button, to call rest endpoint which indeed expires the cookie.
Roughly something like this:
@ConfigProperty(name = "quarkus.http.auth.form.cookie-name")
	Optional<String> formAuthCookieName;

@GET
	@Produces(MediaType.APPLICATION_JSON)
	@Path(URI_LOGOUT)
	@Authenticated
	public Response logout(@Context SecurityContext security) {
NewCookie cookie = new NewCookie.Builder(formAuthCookieName.get())
				.value("")
				.path("/")
				.domain(null)
				.maxAge(0)
				.expiry(Date.from(Instant.EPOCH))
				.build();
			clientResponse.cookie(cookie);
			return clientResponse.build();
	}

Quarkus does not hold local session but restores the session upon request from the encrypted cookie.
That beeing said, we did currently not look into if vaadin creates an hold an extra user session which must be invalidated. If you know something about that, please feel free to share.

From what i can see Vaadin can close the seesion, but i dont know if is necesary to do so, i tested like you mentioned and indeed it deletes the quarkus token, how do you call the endpoint? with navigation?

Pretty simple
Button logout = new Button(new Icon(VaadinIcon.SIGN_OUT),
                e -> UI.getCurrent().getPage().setLocation(Paths.LOGOUT)
        );

I'll see if we can enhance the starter with an example

Thanks, i was doing with navigate and it was not working properly, that would be great! i would try to add the basic JPA auth too, is not hard to implement but would be a bit more clear to new people trying it out, also there could be some hackery that could be done to implement OIDC , as you can tap into the requests vaadin makes, but i do not know how hard it would be to implement

We added logout example to the starter recently


Implement the DynamicMemberLookupProtocol proposal.

This is the second cut at implementing the DynamicMemberLookupProtocol proposal.  Changes from PR13076 include:

Incorporating @slavapestov's awesome code review feedback.
Retroactive conformance to DynamicMemberLookupProtocol now passes.

This also passes all tests for me locally, but the other patch apparently failed some test on linux.  the logs are gone, so I can't investigate it.  If this patch fails again then I'll investigate more promptly.

@swift-ci please test

Thank you for the feedback everyone.  I've put the updated patch over in PR 14546


Unable to use with Angular 16

Hi Team,
Am upgrading my angular project from 13 to 16 and facing some issue with ngx-virtual-scroller can someone please help.
This likely means that the library (ngx-virtual-scroller) which declares VirtualScrollerModule is not compatible with Angular Ivy. Check if a newer version of the library is available, and update if so. Also consider checking with the library's authors to see if the library is expected to be compatible with Ivy.

Hi Team, Am upgrading my angular project from 13 to 16 and facing some issue with ngx-virtual-scroller can someone please help.
This likely means that the library (ngx-virtual-scroller) which declares VirtualScrollerModule is not compatible with Angular Ivy. Check if a newer version of the library is available, and update if so. Also consider checking with the library's authors to see if the library is expected to be compatible with Ivy.

@gora-v2  I hope you can use this module [ @iharbeck/ngx-virtual-scroller ] as an alternative. Which is an extended module from the [ngx-virtual-scroller]. You can check this out!!!

Hi @RajeshMunirathinam,
Thanks for pointing to the right alternative.


SEO Friendly Url and Route in ASP.NET Core

Hi,
I am doing research to create URLs for search engine friendly using asp.net core MVC route and however, I could not find any material that could help me to achieve using  https://docs.microsoft.com/en-us/aspnet/core/mvc/controllers/routing?view=aspnetcore-3.1.
I would thank you if you could create a sub-section under routes to implement it.
Thanks

Hello! Thanks for contacting us. This sounds like a general question about using ASP.NET Core. While we try to look at and respond to all issues, for questions like this we recommend posting to a community support group like Stack Overflow with the asp.net-core tag.


Cancel signal ineffective when the Resolver thread is busy

Example LocalResolver run when a function is modified to spin forever:
~/work/sematic$ bazel run sematic/examples/add
INFO: Analyzed target //sematic/examples/add:add (1 packages loaded, 28 targets configured).
INFO: Found 1 target...
Target //sematic/examples/add:add up-to-date:
  bazel-bin/sematic/examples/add/add
INFO: Elapsed time: 1.476s, Critical Path: 0.50s
INFO: 3 processes: 3 internal.
INFO: Build completed successfully, 3 total actions
INFO: Build completed successfully, 3 total actions
INFO:sematic.resolvers.state_machine_resolver:Starting resolution 0414b92c15d149b59c1a0d0f1ecb99af
INFO:sematic.resolvers.state_machine_resolver:Running inline sematic.examples.add.pipeline.pipeline
INFO:sematic.resolvers.state_machine_resolver:Scheduling sematic.examples.add.pipeline.add31
INFO:sematic.resolvers.state_machine_resolver:Scheduling sematic.examples.add.pipeline.add31
INFO:sematic.resolvers.state_machine_resolver:Scheduling sematic.examples.add.pipeline.add2
^C^C^C^C^C^C^C^C^C^C^C^CWARNING:sematic.resolvers.state_machine_resolver:Received SIGINT, canceling resolution...  <- first Ctrl^C
^C^CWARNING:sematic.resolvers.state_machine_resolver:Received SIGINT, canceling resolution...
^CWARNING:sematic.resolvers.state_machine_resolver:Received SIGINT, canceling resolution...
^CWARNING:sematic.resolvers.state_machine_resolver:Received SIGINT, canceling resolution...
^CWARNING:sematic.resolvers.state_machine_resolver:Received SIGINT, canceling resolution...
^CWARNING:sematic.resolvers.state_machine_resolver:Received SIGINT, canceling resolution...
^CWARNING:sematic.resolvers.state_machine_resolver:Received SIGINT, canceling resolution...  <- several minutes later
^C^C^C^C^C^C^C^C^CKilled: 9    <- this is an external `kill -9`
~/work/sematic$

This will theoretically also happen when the CloudResolver is busy with an inline function.
This will theoretically also happen with the UI cancel button press.
All these cases should be validated.

At this time there is no way to kill the thread. We do not know the PID of the process so we can't kill it. We would need to store that in the resolution object. Not really worth it IMO since the user can Ctrl+C from the console.

Not really worth it IMO since the user can Ctrl+C from the console.

@neutralino1 This issue says that Ctrl+C does not work from the console in these cases.


Build different format of the docs

Is it possible to build epub or mobi version of the docs. That would be quite useful to have.

Thanks for the feedback @ppoliani - we'll look into this. I've created https://pegasys1.atlassian.net/browse/PAN-3109 for tracking this.


update workflow.GoNamed() doc

 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.

Coverage remained the same at 69.06% when pulling 7f6c0cbe89de84b90e9ea68fe6b42dcb1e46d0a9 on mikolajb:master into d4d8e2f5e540814e1374fc52a67508a25ff8e0ed on uber-go:master.


Issues with Coinbase wallet

When you are logged in with Coinbase wallet there is no "Sign Out" button, just "Switch Wallets".
It's currently not working with Kovan, even if Coinbase is set to Kovan in the ETH network settings.

@bconfortin testnet isn't supported with WalletLink at this time. https://github.com/walletlink/walletlink/issues/41#issue-656016136
Noticed another issues though with CoinbaseWallet, i have a PR to address that here


Arrow keys not working when using R in a VSCode terminal

I seem to have stumbled into a key binding issue when running R under a VSCode terminal. The keys work just fine when R.exe is used under a basic Windows 10 terminal or RStudio. Under a VSCode terminal, they do nothing, not even the weird characters that show up when the key strokes are not mapped correctly (typically correct with an .inputrc file).

VSCode Version: 1.12.2
OS Version: Windows 10

Steps to Reproduce:

Download and install R from https://cran.r-project.org/bin/windows/base/R-3.4.0-win.exe
Open a terminal in VSCode
Run "C:\Program Files\R\R-3.4.0\bin\x64\R.exe"
Attempt using the tab, left, right, up or down arrow keys

I have confirmed that the keys work when using R under a cmd.exe terminal. Any help that you can provide is helpful.
Thanks for all the great work!

Does it work if you set your shell to use R.exe? Adding this to your settings.json:
"terminal.integrated.shell.windows": "C:\Program Files\R\R-3.4.0\bin\x64\R.exe"

Hi there,
I added the proposed line to my settings.json. Started a terminal using Ctrl+` which took me directly into an R console session. I typed in part of an R command and then tried to use the arrow or tab keys without success. I should point out that other keys like Backspace work just fine so I can sort-of kind-of operate.
Thanks for getting back to me. I am happy to try anything else that you recommend.

The recent update to 1.13.0 has fixed the issue for me. I can open an R-3.4.0 session in the VS Code Integrated Terminal and navigate using the arrow keys! I have not modified anything on my computer other than the VS Code 1.13.0 update.

VS Code = 1.13.0
OS = Windows 10, Build 1703

Thanks for all the wonderful work!


from_items is deprecated

pd.DataFrame.from_items is deprecated and from_dict should be used instead
added a .gitignore with pycache in it

thx for making this pull-request... i am so annoyed by this warning as well, i was about to do the pull-request myself. please merge @meteomatics


Add execute method to providers and jstp API

Landed in https://github.com/metarhia/globalstorage/commit/e4f9fe14fecd70d3815f07ab285372db8e9d0f4a.


Fake prediction does not enter Proc function

linkstatic = False in bazel BUILD files causes components based on TimerComponent to never enter Proc() function.
To reproduce:

Run Fake prediction module: mainboard -d modules/tools/prediction/fake_prediction/fake_prediction.dag
In cyber_monitor the chanel /prediction/ostacles/ will be empty.

To avoid the bug:

chank static linking to True in this line:
https://github.com/ApolloAuto/apollo/blob/b00c030ac02c54f98579d3d3e879dbf52f0d1d53/modules/tools/prediction/fake_prediction/BUILD#L18

rebuild this package

the messages will appear.

Similarly, if you change linkstatic to False in modules such as Control or Monitor, they will stop execute Proc function.
Version:
This bug appeared only in Apollo 8.0. In Apollo 6.0 the modules without static linking work well. (didn't test in Apollo 7.0).
Do you have ideas why this happens?

Yes, it's not a bug, this is due to 8.0 changing the compilation method.

Before 8.0, each library is compiled as a dynamic library, and a module may have many dynamic libraries
After 8.0, all library is compiled as a dynamic library, and then linked as one dynamic library

The advantage of this is that there is no need to package so many dynamic libraries when release, and it need a trick in bazel like use both linkshared and linkstatic to achieve this
cc_binary(
    name = "libcontrol_component.so",
    linkshared = True,
    linkstatic = True,
    deps = [":control_component_lib"],
)

Does it mean that all binaries should be compiled with linkstatic = True? There are few such targets with False in prediction and planning modules.
As I understand, this flag only defines a mode for linking libraries and is not supposed to brake modules. With linkstatic = False the Init() function of TimerComponent is executed but not the Proc() function. Or there are some other changes for new compilation method except of linkstatic flag?

Yes, this a trick for bazel, but not highly recommended. Bazel recommends static links when publishing, but we publish so files.


Fix Flask async issue when starting a teleport server locally

It seems that Flask alone cannot handle properly asynchronous calls. It especially happens when combining a teleport webpack frontend template with this one.
To manage the network stack we need uwsgi (even on local development) on top of Flask.
Nb: the --py-autoreload option let uwsgi reload the flask server automatically on changes; useful for dev.
@pocketjoso @ClemDelp @Ledoux

We should test that this resolves the problem for everyone involved.


Wallet creation is generating wrong addresses from xpub

The wallet lib method used to get addresses expects the xpub already in the last derivation path, however we use the root xpub.
We must change to send the root xpub and a derivation path.
This change must be done in the wallet service and in the wallet lib.

@pedroferreira1 I don't understand why we should do it.
Quick note: Even though the change is simple, it feels like a critical change that requires a lot of testing.


Improve self-signed certificate generation:

When starting, the code generates a new ephemeral private key and a corresponding certificate to go along with it. This process can take time and, while this is unlikely to matter for normal server operations, it can have a significant impact for unit testing and development. Profiling data suggests that ~20% of the time needed for a unit test run can be attributed to this.
This commit does several things:

It restructures the code so that a new self-signed certificate and its corresponding private key are only initialized once at startup; this has minimal impact on the operation of a regular server.
It provides new default DH parameters. This doesn't impact the security of the connection, but those who compile from scratch can generate new parameters if they so choose.

High Level Overview of Change

Context of Change

Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (non-breaking change that only restructures code)
[ ] Tests (You added tests for code that already exists, or your new feature included in this PR)
[ ] Documentation Updates
[ ] Release

Generally, I think it might be a good idea to remove this automatic certificate generation from within the code. It's a convenient option to be sure, but it's also limited; for example, it will only generate RSA certificates.
with options like LetsEncrypt for those whose servers have an associated domain, it is now trivially easy to get a "real" certificate from a well-known registrar, and it would be better if server operators did that instead.
Those who can't get a certificate from a CA could still generate a self-signed one. Information on how to do it on the web is plentiful but if necessary, we could include instructions on how to do it and it might even be something that could happen during initial package installation by invoking the right incantation.

Maybe it is a naive question, because I didn't look at the code, but assuming most unit tests don't need the certificate, would it be possible to generate it the first time it is needed?

Maybe it is a naive question, because I didn't look at the code, but assuming most unit tests don't need the certificate, would it be possible to generate it the first time it is needed?

We would need to (a) restructure the startup sequence; and (b) make it possible to avoid instantiating the Overlay inside ApplicationImp. Both of these are good ideas and worth doing even if this is accepted & merged, but they are both much more complicated than this simple fix.
I folded a commit into this that addresses the concern by @donovanhide and fixes #4007. Apologies, I hadn't realized you had already reviewed, @ximinez, @greg7mdp.

Merged as 0ecfc7cb1a958b731e5f184876ea89ae2d4214ee


Update documentation to limit indexed fields for import

Pull Request
Related issue
Fixes #134
What does this PR do?

Updates import script documentation to indicate how to limit fields to be indexed when running said script

PR checklist
Please check if your PR fulfills the following requirements:

[X] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[X] Have you read the contributing guidelines?
[X] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

Hey @rkeppner thanks a lot for contributing 🔥. As per this comment, I'm closing this PR in favor of this PR #139

This message is sent automatically
Thanks again for contributing to Meilisearch :heart:
If you are participating in Hacktoberfest, and you would like to receive some gift from Meilisearch too, please complete this form.


UTF-8 chars in title break analysis

First detected by @sjanssen2.
Basically, if your title has a UTF8 char, the analysis breaks during validation (my guess is that this will be fine for Q2).

Can you provide more information about the error?

Yup, if your study is named: Antonio’s study vs. Antonio's study you will get something like this:
https://qiita.ucsd.edu/analysis/description/14608/ (only owners and admins can see it)


[SUGGESTION] Ask to join feature

Is your feature request related to a problem? Please describe.
no
Describe the solution you'd like
add a join party thing on the plus icon heres an ex https://media.discordapp.net/attachments/688765745139679292/800645672386166795/unknown.png?width=293&height=94
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
ee

This is completely unrelated but
is there a different RP that can do that?

There was i think..
(https://github.com/thelennylord/rblx_rich_presence)
but it's now broken because it used auth ticket to join the game


cant start xfce after update

when i run sudo startxfce4
i get

Update your chroot see https://github.com/dnschneid/crouton/wiki/Common-issues-and-reporting

i tried to update it and this is what happened

@giliammc For some reason you need libelf-bin package.
You could try the following.
$ sudo enter-chroot -n trusty
$ sudo apt-get update
$ sudo apt-get install libelf-bin
$ sudo apt-get upgrade
$ exit

If everything executed with no errors update your chroot again.

it says unable to locate packages libelf-bin

Sorry typo sudo apt-get install libefl-bin

i got

so i ran "sudo apt-get -f install" and got

i created a chroot with no xiwi and it seems to work just fine, is their a way to switch back to the old way
of using ctrl,alt,forward,refresh

@giliammc No, ctr-alt-shift Forward/backward is the keyboard shortcut to use and has been a long time. That ctrl,alt,forward,refresh also worked was, because you could switch vt that way. When chromeos switches to freon vt switching will not be working any more.

Why did you add the ppa from https://launchpad.net/~enlightenment-git ?

it was to install a terminal emulator, should i remove it ? if so how do i do that in the command line

Well I don't know exactly what you did in your chroot. The enlightenment ppa is meant for installing as the name already says for installing enlightenment on ubuntu 14.04, 14.10 http://ubuntuportal.com/2015/02/how-to-install-enlightenment-e20-desktop-environment-in-ubuntu-14-04-14-10.html
I created a new chroot to see if the ppa breaks something when updating, but it seems it doesn't. I just could update the chroot with no problems when the ppa was added. You did something that messed up the update. I also could install e20 and terminology package nothing broke the updating chroot.
You could try removing terminology, that seems the package that has the problem looking at your second post. However this is a bit of a wild guess since I don't know if you have maybe other ppa's added or installed packages from external source. See below how to remove that package if the last command is executed with no errors try updating your chroot again.
$ sudo apt-get remove --purge terminology
$ sudo apt-get update && sudo apt-get upgrade

BTW If you can't successfully update your chroot you will not be able to start xfce, so this must be solved first.

$ sudo apt-get remove --purge terminology
$ sudo apt-get update && sudo apt-get upgrade
ran without errors, i tried to update my chroot again and got the same error

i fixed it by removing the packeges that could not be verified


Actuator Relay

Add a relay, using the relay shield

[x] Basic code ported from actuator_ledbuiltin
[x] Merge ledbuiltin and relay using a common file, or a class

Working with the Lolin Relay shield, not tested on anything else but presumably just needs the ACTUATOR_RELAY_PIN set to define the pin.
Please re-open, or create a new issue if have problems.


Remove sw/host/spiflash directory

This PR removes the sw/host/spiflash directory since we are switching to the SPI Flash based bootstrap protocol in test_rom and opentitantool also supports the previous bootstrap implementation.

@tjaychen @weicaiyang Once this PR is merged, sim_dv_frames files will not be generated as part of the sw build.

A couple of notes:

Does DV have an equivalent flow for --dump-frames=filehandle?

Not yet. This was discussed in the chip-level testing meeting and we agreed to stop running the one test that required this.
I don't think we need to produce a similar sim_dv_frames file for the new bootstrap protocol as long as the UVM function can send the correct SPI Flash commands. It should be enough to send the payload as 256 byte chunks since we don't have a digest or frame number per frame.

Should we also remove sw/host/vendor?

I think so. I'll push an update if nothing else is depending on it (I don't think so).


Remove hardcoding for commissioner test app

Current test app: ${borderrouter}/tests/meshcop/commissioner.cpp - and the associated test script meshcop are very hard coded for a very specific test scenario.
This pull request:

Removes hard coding of the test app
Adds features to the test app to calculate for example the PSKc
And can be used with any target device by adjusting various command line parameters.

Note: I expect the pull request to fail the first time or two as I work through the automated test adjustments...

@bukepo & @jwhui  - I am in the office tomorrow, then leave for a few weeks (vacation)
I would really like to get this merged in before I leave, otherwise it will sit until I return.
Thanks.


Chart.js V2: Add prefix or suffix to tooltip label

In Chart.js V1.0, I would add tooltipTemplate: <%if (label){%><%=label %>: <%}%><%= '€' + value %> to add a euro symbol as prefix to the tooltip label. However, this no longer works in V2.
What is the new way to accomplish this in Chart.js V2?

@TheLeonKing you can modify the tooltip label callback. There is a sample detailing this at https://github.com/nnnick/Chart.js/blob/v2.0-dev/samples/tooltip-hooks.html

Great! Thanks for the information. However, is it possible to prevent the newline that the 'afterBody' callback automatically adds?

@TheLeonKing there is no way to get rid of the newline. What are you trying to achieve with the tooltip?

I would like the suffix 'g/km' (gram per kilometer) in my tooltip. So instead of e.g. 'Emission: 4', I want 'Emission: 4 g/km'.

@TheLeonKing the afterBody callback only gets called once, after all the items have been displayed. To add something to each line, use the label callback and append the suffix in there

Great, thanks!


fix(company data): crash and disable issue

Description
Disable confirm button if input value is not matching with the regular expression
Fix infinite loop issue in add New Address Modal
Why
crash
Issue
NA
Checklist
Please delete options that are not relevant.

[x] I have followed the contributing guidelines
[ ] I have performed IP checks for added or updated 3rd party libraries
[ ] I have created and linked IP issues or requested their creation by a committer
[x] I have performed a self-review of my own code
[x] I have successfully tested my changes locally
[ ] I have added tests that prove my changes work
[ ] I have checked that new and existing tests pass locally with my changes
[ ] I have commented my code, particularly in hard-to-understand areas

@Phil91 Could you pls check dependency check?

@Phil91 Could you pls check dependency check?

I issued a license request and it has been approved already - please update DEPENDENCIES


Array => Hash => Empty array is lost when parsing params

Given an API:
params do
  requires :data, type: Array, allow_blank: true
end
post '/test' do
  return params.data.as_json
end

And a test:
describe 'Test' do
  it do
    post_body = {
      data: [{ test: '123', children: [] }]
    }
    post '/test', post_body
    puts last_response.body
  end
end

The result is [{ "test" => "123" }].
If I describe this param in detail:
params do
  requires :data, type: Array, allow_blank: true do
    requires :test, type: String
    requires :children, type: Array, allow_blank: true
  end
end

It will raise a exception of data[0][children] not found.
When I change requires :children into optional :children, no exception is raised, but children still disappear.
Both of them behaviors as expected when children is not empty. And both of them behaviors the same with or without allow_blank.
But when data becomes a JSON type, it becomes normally:
# API
params do
  requires :data, type: JSON
end

# Test
post_body = {
  data: [{ test: '123', children: [] }].to_json
}

# Result is correct: [{ "test":"123","children":[]}]

Is this legit?
Tested in grape 0.18.0

Empty arrays with a non-JSON type are tricky. There's no such thing, it nulls out the value I believe.
The JSON type posting as you provide is a red herring. You should declare data as Array, but POST with a JSON content-type, then it will work as you expect.
I'll close this. Please use the Grape mailing list for more questions.

Thank you! post_body.to_json and header 'CONTENT_TYPE', 'application/json' resolved this problem. Seems like it only affects test environment.


chore: disable coveralls, it's acting up

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Consider removing Clarity Seed from documentation on V4

Describe the issue
Clarity seed has been deprecated for nearly two years. The link in documentation on V2 and V3 is invalid. It's become legacy in favor of schematics. Clarity seed is not a component or a feature on V2 or V3. Consider removing it from documentation on the release of V4.
How to reproduce
View documentation on the website.
Expected behavior
New users have a clear understanding on what features are available.
Versions
App

Clarity: V2, V3

Additional notes
None.

Thanks for filing this.
I looked on the v2/v3 getting started pages but do not see a reference to the Clarity seed. Is there another place that you are seeing it?

@Skripd Thank you!

No longer an issue or resolved in the upcoming website work. Closing as resolved.


feat(build): Dummy PR

feat(build): Dummy PR

:tada: This PR is included in version 0.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


helm: relax requests for block producer to 7 vcpus instead of 8

The mina documentation recommends a c5.2xlarge instance to run the
block producer, but using such an instance in EKS will result in the
pod not starting due to not enough CPU.
Changing the request to 7 leaves enough CPU for the kubernetes system
daemons. It should be minimally disruptive to other consumers of this
chart.
Sumbitted as part of my attempt to make https://github.com/midl-dev/mina-pulumi
project work well with default chart.

The mina documentation recommends a c5.2xlarge instance to run the
block producer, but using such an instance in EKS will result in the
pod not starting due to not enough CPU.
Changing the request to 7 leaves enough CPU for the kubernetes system
daemons. It should be minimally disruptive to other consumers of this
chart.

Hmmm ok not sure I understand the resource issue on EKS all the way. Mind explaining the limitation more?

When running with 8 cpu requirement, I get the below error:
│                  node.kubernetes.io/unreachable:NoExecute for 300s                                                                                │
│ Events:                                                                                │
│   Type     Reason            Age                 From               Message                                                                                │
│   ----     ------            ----                ----               -------                                                                                │
│   Warning  FailedScheduling  19s (x25 over 34m)  default-scheduler  0/1 nodes are available: 1 Insufficient cpu.

The reason for the behaviour is: despite running in a node with 8 vCPUs, kubernetes reserves one vcpu for its own usage.

@nicolasochem Ok cool got it. I question whether it makes more sense to change the documentation recommendation instead (since that seems less disruptive and not really based on some sort of technical requirement) but I can see either way.
tbh imo resources requested/limited should be a chart value and left upto the operator but that might be better left for another change.

This PR has become stale, and its been a few years since it opened. Closing.


SendKeys fails to push repeated characters

expected : ......gNPPumwQw......
actual sent: ......gNPumwQw......
we noticed on azure microsoft hosted agents, win app driver sendkeys sometimes skips the repeated character in string.
(e.g. see portion of string above when PP is sent then it just takes 1 P instead of both)
Locally it works fine so it is something specific to environment where it runs. any workaround to this ?
anything env specific stuff we should set on the pipeline agent may be a powershell script ?
Version of winappdriver is 1.2.1

Send keys has issues not only on Microsoft hosted agent but also on self agents . Which language binding are you using?

We use the win app driver via appium .Net client libraries.
Never noticed issues with sendkeys api on local machine windows 10.

You can use the custom method which can be used like set Immediate text
public void SetText(WindowsElement editField, string text)
        {
            Thread STAThread = new Thread(
                delegate ()
                {
                    System.Windows.Forms.Clipboard.SetText(text);
                    editField.Click();
                    driver.Keyboard.SendKeys(OpenQA.Selenium.Keys.Control + "v" + OpenQA.Selenium.Keys.Control);
                });
            STAThread.SetApartmentState(ApartmentState.STA);
            STAThread.Start();
            STAThread.Join();
        }

Hi @anunay1
Thanks very much for suggesting alternative.
Actually we have .net core 5 project which runs on linux as well, so clipboard api System.Windows.Forms.Clipboard.SetText(text); is not available.
I'll try to put on clipboard using some powershell stuff, and update you.

Which version of .net core are you using?

Which version of .net core are you using?

.NET 5

Seems like that's a bug with VS2019 where in System.Windows.Form is not see at part of .net core 5. See this this. Using power shell will have its own set of problems. Please let me know if you succeed.

I was able to put the content on clipboard using powershell. (not using powershell sdk).
And it works fine and consistent success both on pipeline and local machine.
Didn't use System.Windows.Forms because to use it project must be set to use windows .net 5     <TargetFramework>net5.0-windows</TargetFramework>, which wasn't suitable for us to keep things platform independent.


Integrate with Zapier

Integration with Zapier would open up the ability to easily connect (via Zapier) to a wide variety of other services.

Closed by #117