sample not launching on android

Hi,
I followed exactly the step from here : http://docs.nativescript.org/angular/tutorial/ng-chapter-1#chapter-1getting-up-and-running
last step is : tns run android --emulator
twice
And still get those errors :
D:\www\Groceries\sample-Groceries>tns run android --emulator
Starting Android emulator with image AVD_for_Nexus_S_by_Google
Waiting for emulator device initialization.....
Executing before-prepare hook from D:\www\Groceries\sample-Groceries\hooks\before-prepare\nativescript-dev-typescript.js
Found peer TypeScript 2.1.4
app/utils/status-bar-util.ts(3,24): error TS2307: Cannot find module 'utils/utils'.

node_modules/rxjs/Observable.d.ts(5,30): error TS6143: Module './observable/IfObservable' was resolved to 'D:/www/Groceries/sample-Groceries/node_modules/rxjs/observable/IfObservable.js', but '--allowJs' is not set.

node_modules/rxjs/Observable.d.ts(6,33): error TS6143: Module './observable/ErrorObservable' was resolved to 'D:/www/Groceries/sample-Groceries/node_modules/rxjs/observable/ErrorObservable.js', but '--allowJs' is not set.

node_modules/tns-core-modules/tns-core-modules.base.d.ts(30,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/ui/action-bar/action-bar.d.ts' not found.

node_modules/tns-core-modules/tns-core-modules.base.d.ts(82,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/ui/text-base/text-base-styler.d.ts' not found.
node_modules/tns-core-modules/tns-core-modules.base.d.ts(83,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/ui/text-base/text-base.d.ts' not found.
node_modules/tns-core-modules/tns-core-modules.base.d.ts(84,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/ui/text-field/text-field.d.ts' not found.

node_modules/tns-core-modules/tns-core-modules.base.d.ts(85,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/ui/text-view/text-view.d.ts' not found.
node_modules/tns-core-modules/tns-core-modules.base.d.ts(86,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/ui/time-picker/time-picker.d.ts' not found.
node_modules/tns-core-modules/tns-core-modules.base.d.ts(87,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/ui/transition/fade-transition.d.ts' not found.
node_modules/tns-core-modules/tns-core-modules.base.d.ts(88,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/ui/transition/slide-transition.d.ts' not found.
node_modules/tns-core-modules/tns-core-modules.base.d.ts(89,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/ui/transition/transition.d.ts' not found.
node_modules/tns-core-modules/tns-core-modules.base.d.ts(90,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/ui/utils.d.ts' not found.
node_modules/tns-core-modules/tns-core-modules.base.d.ts(91,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/ui/web-view/web-view.d.ts' not found.

node_modules/tns-core-modules/tns-core-modules.base.d.ts(92,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/utils/debug.d.ts' not found.
node_modules/tns-core-modules/tns-core-modules.base.d.ts(93,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/utils/lazy.d.ts' not found.
node_modules/tns-core-modules/tns-core-modules.base.d.ts(94,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/utils/types.d.ts' not found.
node_modules/tns-core-modules/tns-core-modules.base.d.ts(95,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/utils/utils.d.ts' not found.
node_modules/tns-core-modules/tns-core-modules.base.d.ts(96,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/xml/xml.d.ts' not found.
node_modules/tns-core-modules/tns-core-modules.d.ts(5,1): error TS6053: File 'D:/www/Groceries/sample-Groceries/node_modules/tns-core-modules/weakmap.d.ts' not found.
node_modules/tns-core-modules/ui/editable-text-base/editable-text-base.d.ts(3,31): error TS2307: Cannot find module 'ui/text-base'.

node_modules/tns-core-modules/ui/frame/frame.d.ts(8,33): error TS2307: Cannot find module 'ui/transition'.
node_modules/tns-core-modules/ui/label/label.d.ts(6,31): error TS2307: Cannot find module 'ui/text-base'.

node_modules/tns-core-modules/ui/page/page.d.ts(8,32): error TS2307: Cannot find module 'ui/action-bar'.

TypeScript compiler failed with exit code 1

D:\www\Groceries\sample-Groceries>

I am using windows 10 64bit
Thanks for any help you can provide !
Stéphane

Hey @pilere
I have tested the steps on my side and they are working as expected.
Just to confirm that we are talking about the same steps of steps here are mine:
cd test
git clone https://github.com/NativeScript/sample-Groceries.git
cd sample-Groceries
git checkout angular-start
tns platform add android
tns run android --emulator

Tested with Node.js 6.91 and npm 3.10.8.
Check if you are using the latest NativeScript
tns -version
2.4.1

Hi Nick,
yes, I would add the step from here : Complete the NativeScript installation guide (http://docs.nativescript.org/angular/start/quick-setup) with the fresh install of https://nodejs.org/en/ as 6.9.2 LTS
and then :
npm install -g nativescript
@powershell -NoProfile -ExecutionPolicy Bypass -Command "iex ((new-object net.webclient).DownloadString('https://www.nativescript.org/setup/win'))"
%ANDROID_HOME%\extras\intel\Hardware_Accelerated_Execution_Manager\intelhaxm-android.exe
with a reboot between each to make sure that the environement variables are set correctly.
when I ran : tns doctor
it said to upgrade the nativescript which I did with :
npm install -g nativescript
again ... I thought that it get updated during my installation ;)
I then had a perfect tns doctor  ... but now is asking for a new update :
D:\www\Groceries\sample-Groceries>tns doctor
NOTE: You can develop for iOS only on Mac OS X systems.
To be able to work with iOS devices and projects, you need Mac OS X Mavericks or later.
Updates available
┌──────────────────┬─────────────────┬────────────────┬──────────────────┐
│ Component        │ Current version │ Latest version │ Information      │
│ tns-core-modules │ 2.4.0           │ 2.4.2          │ Update available │
└──────────────────┴─────────────────┴────────────────┴──────────────────┘
No issues were detected.

npm --version
3.10.9

node --version
v6.9.2

....
Stéphane

Hey @pilere it seems that your tns doctor is returning
No issues were detected.

The first part of the message above is just a warning that you can't develop for iOS in your environment but overall you have no issues to continue developing for Android.
The second part with the upgrade instructions prompts you to update not your NativeScript (which you have successfully installed with your command npm install -g nativescript - Btw you can check the version of your globally installed NativeScript with tns --version) but your tns-core-modules plugin (the core plugin for NativeScript which is bringing all the NativeScript modules).
This is happening because you are running tns doctor from your app directory (sample-Groceries) and it is checking if the project is requiring all the lastest versions of NativeScript related dependencies.
To update you tns-core-modules run the following
tns plugin remove tns-core-modules
tns plugin add tns-core-modules

Now your plugin will be updated to the lastest stable version (2.4.2) and runnign tns doctor won't give you those warnings.

I wasn't worried about the update but the error ...
in fact after the update of the tns-core-modules it start to work but then crash on the simulator :
BUILD SUCCESSFUL
Total time: 48.919 secs
Project successfully built
Successfully deployed on device with identifier 'emulator-5554'.
Error: read ECONNRESET
at exports._errnoException (util.js:1022:11)
at TCP.onread (net.js:569:26)

so still not working ....

@pilere as a workaround for this issue try to reset your already create platforms folder.
Uninstall the app from your emulator and run the following:
rm -rf node_modules
rm -rf platforms
tns run android --emulator

p.s If you can't execute rm -rf then manually delete the folders node_modules and platforms from your project.
Let me know if this resolves your issue with Error: read ECONNRESET

BUILD SUCCESSFUL
Total time: 2 mins 14.252 secs
Project successfully built
Successfully deployed on device with identifier 'emulator-5554'.
JS: Angular 2 is running in the development mode. Call enableProdMode() to enable the production mode.
so 2mn 14 after .... o my .... it work !
but quite frankly if it take so long after each change ... I think I will try to find another way to developp android app ...
Stéphane

THANK YOU SO MUCH for your help and patience !

Hey @pilere in fact only the initial build takes longer.. live syncing your application and even re-running tns run android is considerably less time!


broken netlify link

Describe the bug
Site deployed on netlify has a broken link  and can't be found. Please help
https://gleeful-parfait-4dd4b5.netlify.app/
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-vu3https://gleeful-parfait-4dd4b5.netlify.app/nnf
Steps to reproduce
click link for website
Expected behavior
this is what i need.
I've successfully imported your project. I'm ready to assist you with analyzing and improving your code.
Your project has been previously deployed to https://gleeful-parfait-4dd4b5.netlify.app/ and is owned by the StackBlitz Netlify account.
If you want to deploy this project to your personal account, claim ownership by visiting undefined and connect with Netlify.
Screen Recording / Screenshot
I can't do any of this message from bolt.
I've successfully imported your project. I'm ready to assist you with analyzing and improving your code.
Your project has been previously deployed to https://gleeful-parfait-4dd4b5.netlify.app/ and is owned by the StackBlitz Netlify account.
If you want to deploy this project to your personal account, claim ownership by visiting undefined and connect with Netlify.
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 49087103,
  "usedJSHeapSize": 46736523,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = alhasan999
Chat ID = e9c2fe19143a

Additional context
none

This issue is now fixed.

https://github.com/stackblitz/bolt.new/issues/3948


Utilize printing system in code generation

Significant parts of the QSD code converter could be rewritten to make use of the printing system. Specifically, generating valid variable names from labels of "atomic" Expression, and generating infix expressions for operations can be handled through a Printer.
It might be useful to add a _code_ method to Expression (akin to _ascii_, _tex_, etc.) that does some generic code generation (under the assumption that relevant languages such as C/C++, Fortran, Julia, are all pretty similar in terms of what acceptable variable names are and how expressions look.
This depends on #35

QSD code generation has been moved to a separate package, so this is no longer relevant to QNET


I execute the command has many errors. Should you list the tool versions, like system platform, python version and the manager tool etc.

Which command do you execute? Can you report the error? Most of the environments are tested on Ubuntu 18, 20, 22. The Windows binaries are tested on Win11.

It works fine on Ubuntu.


Addon not working with Storybook 6.5.12 and vue

I have created a new project using @vue/cli and:

npx storybook init
npm i storybook-addon-mock -D

add storybook-addon-mock addon in.storybook/main.js
module.exports = {
  "stories": [
    "../src/**/*.stories.mdx",
    "../src/**/*.stories.@(js|jsx|ts|tsx)"
  ],
  "addons": [
    "@storybook/addon-links",
    "@storybook/addon-essentials",
    "@storybook/addon-interactions",
    "storybook-addon-mock"// if remove this line the error is not happens
  ],
  "framework": "@storybook/vue3",
  "core": {
    "builder": "@storybook/builder-webpack5"
  }
}

But the follow erros is happening:

@leandrogehlen This is fixed in 3.2.0


starlark: unstable-2022-03-02 -> 2022-08-17

Description of changes
Patch we used to apply made it to upstream 3 commits before this
snapshot, so remove it as well.

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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

[x] 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/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

[X] reviewed the diff and commit messages
[ X made sure ofBorg build succeeded for all applicable platforms
[X] run nix-review for a reasonable amount of time without any failures (or marked preexisting failures as broken)

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/601


[TASK] Add more PHP-CS-Fixer rules

Also drop some rules that are already in included in the used
rulesets.

Note: I'll need to add some type casts (in separate PRs) before we can make all files strict.

Actually, the rules being removed have the value being set as the default. So I'm wondering if they should be kept, to be explicit, or removed anyway, to avoid clutter.

I don’t care much either way - as long as the config file is consistent between our two projects. :-)


[Bug]: OdhApplication.spec.betaText has non-generic default text

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
If an OdhApplication CR is applied with spec.beta=true but no spec.betaText, there is a information banner displayed on the side panel after clicking the respective tile in "Explore" tab that states "This application is available for early access prior to official release. It won’t appear in the Enabled view, but you can access it by signing up for beta access..".
This text seems to be specific to a beta release of Starburst Galaxy, and should be updated to something more generic.
Expected Behavior
When spec.beta is set to true but spec.betaText is not set, the beta information banner text  should be updated to something more generic such as simply "This application is available for early access prior to official release."
Steps To Reproduce

Apply an OdhApplication CR[1] with beta: true and no betaText defined
Go to ODH Dashboard UI
Click on Explore menu item
Click on the OdhApplication Tile just created
Verify the beta information banner (text in blue background)

[1] Example OdhApplication CR:
apiVersion: dashboard.opendatahub.io/v1
kind: OdhApplication
metadata:
  name: test-app-tile
spec:
  beta: true
  category: Test
  description: Test App
  displayName: Test App
  docsLink: https://www.example.com
  img: foo
  getStartedLink: https://www.examplecom
  getStartedMarkDown: "foo"
  provider: Test
  support: Test

Workaround (if any)
This can be worked around by simply setting the betaText field for any new OdhApplication CR that also uses the beta boolean field.
OpenShift Infrastructure Version
No response
Openshift Version
4.10
What browsers are you seeing the problem on?
Firefox
Open Data Hub Version
1.13

Relevant log output
N/A

somewhat related: it appears the workaround here also requires the workaround for #624 (spec.betaTitle must also be set for spec.betaText to be picked up)

/assign

Migrated to:  https://issues.redhat.com/browse/RHOAIENG-2967


Karabo installation fails with Python 3.8

I think we can place installation related issues:
I encountered the following install issues of Karabo v0.8.0:
Installation:
- localhost (Ubuntu 20.04):
user install:
python=3.8 FAILED
- feature:/linux-64::__glibc==2.31=0
- python=3.8 -> libgcc-ng[version='>=9.3.0'] -> __glibc[version='>=2.17']
Your installed version is: 2.31
python=3.9.13 FAILED
- feature:/linux-64::__glibc==2.31=0
- feature:|@/linux-64::__glibc==2.31=0
Your installed version is: 2.31
dev install:
install using "environment-dev.yaml" SUCCEEDED (python=3.9.13)
- cscs (daint):
user install:
python=3.8 FAILED
- feature:/linux-64::__glibc==2.26=0
- python=3.8 -> libgcc-ng[version='>=9.3.0'] -> __glibc[version='>=2.17']
Your installed version is: 2.26
python=3.9.13 SUCCEEDED
dev install:
install using "environment-dev.yaml" SUCCEEDED (python=3.9.13)

Related to #129

on a macos system:

conda create -n karabo-env python=3.9.13
conda activate karabo-env
conda install -c i4ds -c conda-forge karabo-pipeline

--> UnsatisfiableError: The following specifications were found to be incompatible with each other:

Even without forcing the version it does not work on macos. Will check this :)

We'll update to newer versions anyway. E.g. 3.9 and newer. macOS builds are broken at the moment, this is tracked in #49.


adding R12 node yaml file

The difference between the standard R11 and this new R12 is about separating the CPA region in the R11 model into two new regions for the R12 model, namely CHN (China) and CPA (the rest CPA).

Closed.


NPCs attack SkyTest rats in Ratway

NPCs in the Ratway will attack the tiny rats that SkyTEST adds as ambiance.
I thought I had nailed this, but it does still seem to happen. I would like for this not to happen.

 NPCs attack SkyTest rats in Ratway

Those in the ratway are now allies with the rats, so they won’t attack them. This has the side effect of them “helping” the rats if you attack them, however they attack literally anything they see except the rats already.


Remove Travis

Trello

Removed Travis
Added run-quality-checks into the pipeline
Simplified some of the naming within the pipeline
Added Dynamo DB as a Docker dependency for tests

Co-authored-by: Huw Diprose<EMAIL_ADDRESS>

We could merge this now, or sort the checksum thing first.
I'm currently just trying to get into AWS and see if I can find what version we're using


[Bug] Changing device

Changing device doesn't work until the app is closed and re-opened. Also, can you add volume to all roku devices so I can use CEC?
Thank you for keeping the app updated :)

Hi @StevenAlexander44, thanks for the bug report. I'm currently unable to reproduce this issue on my phone. Are you running the latest version? Did you see the same issue in version 1.0.25?

Yes, I am using the latest version (on Lineage 21)
The IP changes, but the name (58") and remote layout (not applicable with this, only with tv vs hdmi) don't change:
https://github.com/user-attachments/assets/77c6d581-d9c0-484d-8c98-3e7b1781aa89

Thank you for the bug report. This is fixed in version 1.0.27.


Way to access value from key of xml-converted json?

Hi! I have an xml file with the following content
<?xml version="1.0"?> <Book> 	<name>Sample Book Name</name> 	<author>New Author</author> 	<price>102.50</price> </Book>
and using the xml2json.cpp, I get the following json string
{"Book":{"name":"Sample Book Name","author":"New Author","price":102.5}}
Instead I want to obtain just
{"name":"Sample Book Name","author":"New Author","price":102.5}
Is there any way I could do this? I tried going and looking through xml2.json.hpp, rapidjson/document.h and rapidjson/writer.h, but could not do. Any help would be appreciated!
Regards

Solved: rapidjson::Value& t = js_doc["Book"]; t.Accept(buffer);


WiFi credentials

Hi Daniel
It seems that in the process of enhancing your program one of the basics may have slipped through the cracks.
Just curious why the WiFi credentials (SSID and Password) aren't specified in settings.h or by definitions at the top of the .ino file?
To get connected I had to scroll down to line 166 of the .ino file and enter those credentials.  Did I miss something?
Don

Hi Don
Thanks so much for pointing this out. I just committed a fix. Can you close this issue if you agree?
Daniel

Looks good to me!  I also learned how to close an issue.
Don


dbus: remove unused daemon passhtru

Description of changes
Follow up to #201647

Things done

Built on platform(s)

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Seems you want to do a global sed -i 's|dbus\.daemon|dbus|g' on nixpkgs as well..

Probably not too much, but the degree of necessity of comment and of release notes could be considered.

I don't think we need a changelog entry for this. If we would do that the changelog would be completely filled with such entries.


Describe operations and workflow

This is a work in progress, more commits will be made in this branch subsequently

Hi @dkoslicki, how is your work on PR https://github.com/NCATSTranslator/translator-developer-documentation/pull/5 going? Are you almost ready to convert it from draft into a reviewable PR? Maybe if the PR is in good enough shape as an initial draft, we can merge it then iterate further later? I fully trust that your authorship has already resulted in a decent first pass.

Hi @RichardBruskiewich , I think this is ok as a first draft, so submitting it for review now

Hi @RichardBruskiewich , I think this is ok as a first draft, so submitting it for review now

Thanks! I'll take a look!


Encountered code bugs while running the Jupyter Notebook demos in the tutorial

Hi @aviadlevis ,
Thank you for your excellent work!
I have configured the environment according to the installation steps in the README, and then tried to run the jupyter notebooks in the tutorials. However, I encountered the following bugs:

Tutorial2 - synthesize ngEHT observations.ipynb

Code:

Error:

tutorials/Tutorial3 - estimate 3D emission from image plane.ipynb

Code:

Error:

Additionally, here are the versions of the packages installed in my conda virtual environment (Python 3.9.19, CUDA 12.0, Ubuntu 20.04):
packages.txt
I have tried some solutions, including adjusting the versions of some packages, but still haven't been able to successfully run the programs. Could you please offer some suggestions? Thanks for your attention to this issue!
Best regards,
Daiqy

Thanks a lot Daiqy!
The first error seems to be coming form eht-imaging. Could you pase the outputs of running obs_empy.tlist()? What is the data structure?
The second error is easy to fix, I think that this .unfreeze() is deprecated so you can just return params. I will fix this in the github soon.

Thanks for your quick reply!
The first error: I ran the obs_empy.tlist() before obs = bhnerf.observation.observe_same(movie, obs_empty), and got the output as followed:

The second error: I removed .unfreeze() and successfully ran the iteration; however, I've encountered the new errors (perhaps still a version issue?):


sum and product should be more picky

(-1, _) is too generic. anything reading num will be summed/multiplied as an input. but sometimes you're reading the number and using it to control color or something

(-1, 1) so anything reading num values and writing to input 1 gets added/multiplied


Make: add support for compiling difftest as a lib.so

This commit adds support for using difftest as a lib to simulate with XS.
For now, it works both on VCS and Palladium.
use "make difftest-so" to generate libdifftest.so and link it when compiling your design.

Also, should we add a CI test to build the so? We can add a test in test-difftest-vcs


how to disable seata when run unit test？

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Ⅰ. Issue Description
how to disable seata when run unit test？
Ⅱ. Describe what happened
I hava try config this in application.properties in the test folder in resources folder ，but not work，
seata.enabled=false
seata.enable-auto-data-source-proxy=false
seata version ：seata all 1.4
Ⅲ. Describe what you expected to happen
support jump seata when I run unit test。
Ⅳ. How to reproduce it (as minimally and precisely as possible)

xxx
xxx
xxx

Ⅴ. Anything else we need to know?
Ⅵ. Environment:

JDK version : jdk 8
OS :
Others:

seata.service.disable-global-transaction=true

seata.service.disable-global-transaction=true

seata.service.disable-global-transaction=true

still load seata ...

seata.service.disable-global-transaction=true

still load seata ...


Fix: restore TS 4.5 compatibility

This removes type * from the fp-ts re-rexports as that's only a feature available in later versions of TypeScript.
See: #312

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 3 (0.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 99.904%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/internal/either.ts
0
1
0.0%

src/internal/option.ts
0
1
0.0%

src/internal/task-either.ts
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
2597

Relevant Lines:
2600

💛  - Coveralls


FLT_DIG and DBL_DIG need to be Int32 to match other C imports

Fixes build warning on Linux

This gets me back to the same six test failures (all due to Swift/Linux treating strings that only contain control characters as equal to the empty string).


buySubscription return E_DEVELOPER_ERROR error in IOS

Version of react-native-iap
1.2.1
Platforms you faced the error
IOS
Expected behavior
Return the subscription details.

RNIap.prepare()
    .then(() => {
      RNIap.buySubscription('valid_item_id')
      .then(purchase => {
        console.warn('Purchase Detail: ' + JSON.stringify(purchase));
        RNIap.endConnection();
      })
      .catch(err => {
        console.warn('Purchase Error: ' + JSON.stringify(err));
        RNIap.endConnection();
      });
    })
    .catch(err => {
      console.warn(err);
    });

I also tried to call getSubscriptions and getPurchaseHistory using the same product ID without issue. The getSubscriptions will return the subscription details while getPurchaseHistory return empty array as there is no existing purchase. I am using the user ID in the sandbox testers. On Android device all function works well.

Managed to find out the issue after dig into the ios code RNIapIos.m. I need to call the getSubscriptions() before calling buySubscription(). The reason is there is a variable "validProducts" that need to fill with product data before I can call buySubscription(). Seem like this sequence didn't specify in the README.md.


Update Required for Documentation Regarding Expo SDK 49

Following the recent update to Expo SDK 49, it appears that the instructions outlined in this section are no longer necessary.
The expo publish command, previously required, is now redundant. Expo now automatically generates a new embedded update with each build.
Also, users can now start the development server using the following command:
npx expo start --private-key-path (path to the private key)

So modifications to AndroidManifest and Expo.plist, as previously instructed, are no longer required.
This issue is raised to suggest an update to the documentation to reflect these changes and provide accurate instructions for users of Expo SDK 49.
P.S. I don't know if these changes are applicable to Expo SDK < 49.

Hi Seyed,
thank you for pointing out this, I'm about to update an app from expo 48 to expo 49 and I'll definitely give it a try and update the documentation in the next days.
If you are using this project and finds some area that needs improvement don't hesitate to let me know or send PRs!

Thanks @SMhdAsadi , I've updated the documentation


Consume labeled multicluster secrets on startup

This patch when run against istio.yaml or istio-auth.yaml
runs in the new config mode using only labels rather than
configmaps.  The configmap functionality can be removed in
0.9.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ijsnellf
Assign the PR to them by writing /assign @ijsnellf in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @costinm

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
a7c6cb0e63d764fdaa6197d2bfd107a9fb6f2c28
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: costinm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [costinm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
589973e475c1e6cf471492467b03a8e5ec7a2c02
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

New changes are detected. LGTM label has been removed.

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
60c9b5b8e31cb5d02d94f580346f999d03b16674
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
60c9b5b8e31cb5d02d94f580346f999d03b16674
link
/test istio-unit-tests

prow/e2e-bookInfoTests-v1alpha3.sh
60c9b5b8e31cb5d02d94f580346f999d03b16674
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
60c9b5b8e31cb5d02d94f580346f999d03b16674
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-unit-tests.sh
f7d9ad8a81f9c239e8583c450df9dc723f04407e
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sbezverk PTAL

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
f7d9ad8a81f9c239e8583c450df9dc723f04407e
link
/test istio-unit-tests

prow/e2e-bookInfoTests-v1alpha3.sh
f7d9ad8a81f9c239e8583c450df9dc723f04407e
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
f7d9ad8a81f9c239e8583c450df9dc723f04407e
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-unit-tests.sh
b6b35f55d692de6976e613e1038b9b27c217e7b1
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
f7d9ad8a81f9c239e8583c450df9dc723f04407e
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-unit-tests.sh
b6b35f55d692de6976e613e1038b9b27c217e7b1
link
/test istio-unit-tests

prow/istio-presubmit.sh
b6b35f55d692de6976e613e1038b9b27c217e7b1
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
f7d9ad8a81f9c239e8583c450df9dc723f04407e
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-presubmit.sh
b6b35f55d692de6976e613e1038b9b27c217e7b1
link
/test istio-presubmit

prow/istio-unit-tests.sh
f9ab80a21d874be93da84d7884ba87c2decd6166
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
f7d9ad8a81f9c239e8583c450df9dc723f04407e
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-unit-tests.sh
f9ab80a21d874be93da84d7884ba87c2decd6166
link
/test istio-unit-tests

prow/istio-presubmit.sh
f9ab80a21d874be93da84d7884ba87c2decd6166
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
f7d9ad8a81f9c239e8583c450df9dc723f04407e
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-presubmit.sh
f9ab80a21d874be93da84d7884ba87c2decd6166
link
/test istio-presubmit

prow/istio-unit-tests.sh
696097069fe5288a6818b536099679abfadac81b
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
f7d9ad8a81f9c239e8583c450df9dc723f04407e
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-unit-tests.sh
696097069fe5288a6818b536099679abfadac81b
link
/test istio-unit-tests

prow/istio-presubmit.sh
696097069fe5288a6818b536099679abfadac81b
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
f7d9ad8a81f9c239e8583c450df9dc723f04407e
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-presubmit.sh
696097069fe5288a6818b536099679abfadac81b
link
/test istio-presubmit

prow/istio-unit-tests.sh
4ff129e8a09ccd4c1345b9f8346eaf878a3a3131
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
4ff129e8a09ccd4c1345b9f8346eaf878a3a3131
link
/test istio-unit-tests

prow/e2e-bookInfoTests-v1alpha3.sh
4ff129e8a09ccd4c1345b9f8346eaf878a3a3131
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
4ff129e8a09ccd4c1345b9f8346eaf878a3a3131
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-unit-tests.sh
a7be764d357cf3be0edd1b03135b84c0da48bb25
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
a7be764d357cf3be0edd1b03135b84c0da48bb25
link
/test istio-unit-tests

prow/e2e-bookInfoTests-v1alpha3.sh
a7be764d357cf3be0edd1b03135b84c0da48bb25
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
a7be764d357cf3be0edd1b03135b84c0da48bb25
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-unit-tests.sh
bb9f0f9c29c2d5529c35a2e7fc10172265fb3b86
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
bb9f0f9c29c2d5529c35a2e7fc10172265fb3b86
link
/test istio-unit-tests

prow/e2e-bookInfoTests-v1alpha3.sh
bb9f0f9c29c2d5529c35a2e7fc10172265fb3b86
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-unit-tests

/test e2e-bookInfo-envoyv2-v1alpha3

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
bb9f0f9c29c2d5529c35a2e7fc10172265fb3b86
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
bb9f0f9c29c2d5529c35a2e7fc10172265fb3b86
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


CFE-3396: [systemd] fix typos in error classes

Changelog: none
Ticket: CFE-3396
Signed-off-by: Fabio Tranchitella<EMAIL_ADDRESS>

@olehermanse ping


[testrunner] Don't allow the same test multiple times nor namespace c…

…ollisions in the spec.
Makes it much harder to missconfigure.

Great! :-D


Cannot recognize .gitignore when open subfolder of a repository

VSCode Version: 1.14.2
OS Version: win10

Steps to Reproduce:

Make a git repository (assume it's E:/repo)
Open folder E:/repo/sub in VsCode
E:/repo/.gitignore ignored /sub/test.txt
create a file E:/repo/sub/test.txt

Expect
test.txt not appear in changelist
Actual
test.txt does appear in changelist

Reproduces without extensions: Yes

What does git status return?

in E:/repo/sub, git status return changelist include test.txt
but
in E:/repo, it not include.
And in 1.14.0 and 1.14.1 there is no this problem

Expect changelist is relative to git repository path, rather than VsCode open folder.

.gitignore in git repository root folder (not vscode open folder) not work.

in E:/repo/sub, git status return changelist include test.txt

So, git itself shows test.txt... just like Code.
Are you sure there's no folder E:/repo/sub/.git?

@joaomoreno Oh... yes...
Truely there is E:/repo/sub/.git...
Don't know how this appear...
Works well after remove it...


[BUG] Unable to use application id with New-PnPTerm

Reporting an Issue or Missing Feature
Unable to use application id with New-PnPTerm
Expected behavior
Connect via application id with certificate and use New-PnPTerm
Actual behavior
New-PnPTerm: The current user has insufficient permissions to perform this operation.
Steps to reproduce behavior

What is the version of the Cmdlet module you are running?
1.10
Which operating system/environment are you running PnP PowerShell on?

[x] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[x] Azure Functions
[ ] Other : please specify

Hi @svermaak
I don't think this is a PowerShell related issue, it does work for me.
In order to create terms with app only context, API permissions is strangely not enough. You have go to the classic term store manager (https://tenant-admin.sharepoint.com/_layouts/15/TermStoreManager.aspx) and add app@sharepoint as Term Store Administrator.

After this trick, you should be able to make adjustments to the term store in app only context.

I will give that a go and come back

@milanholemans Stellar! I really appreciate it Bud. I was skeptical but it works!


Add ContentNegotiator middleware

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Moved to https://github.com/yiisoft/data-response/pull/38


Find a replacement for OrderedDictionary from Microsoft.Experimental.Collections

https://github.com/vezel-dev/novadrop/blob/8e0d7f20ae4ecc4bac5e0881dced3c5138612ce9/src/formats/formats.csproj#L5-L6
https://github.com/vezel-dev/novadrop/blob/8e0d7f20ae4ecc4bac5e0881dced3c5138612ce9/src/formats/formats.csproj#L23-L25

Related:

dotnet/runtime#24826
dotnet/runtime#64829

It may be worth considering simply not using an ordered dictionary here. When writing the data center file, attributes get sorted anyway. So, creating a data center in memory, writing it, and reading it back, the attributes will not have the same order as they did in memory during creation. It's not clear that using an ordered dictionary here actually adds much value.
Moving this for consideration in v2.0.

Postponing after all. A major issue is that unpacking a data center would result in 'random' attribute order, whereas right now, attributes are ordered as they appear in the data center, which results in a mostly (though not always) intuitive order.
We could order attributes explicitly by key status and name, but that won't cover all cases well.

Good news: dotnet/runtime#24826 is now on track to being implemented.


Removing the JWT Authentication logic

Since the catalog service is going to be deployed in a private subnet.
Removing the JWT Authentication logic.

@DImuthuUpe I remember we discussed that we no longer needed the JWT Authentication of catalog APIs in one of our meetings.
Can you please confirm the same?
If not required, I would like to ahead do some clean-up.
cc: @freitagb @smarru

@ub0005 It is better if there is a JWT Authentication but there should be a proper identity provider to back this up rather than using hard-coded username-password pairs. However, this service runs inside a private subnet, so authentication is not  a major requirement.

https://github.com/NASA-IMPACT/HLS-Transfer-Catalog/pull/19
cc: @xhagrg @NISH1001


Restructure docs

rearange docs
more doc restructuring

Codecov Report
Patch coverage has no change and project coverage change: -67.60 :warning:

Comparison is base (0889b01) 90.98% compared to head (f820afd) 23.38%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main       #2       +/-   ##
===========================================
- Coverage   90.98%   23.38%   -67.60%
===========================================
  Files          10        9        -1
  Lines         255      248        -7
===========================================
- Hits          232       58      -174
- Misses         23      190      +167

Flag
Coverage Δ

basics
23.38% <ø> (ø)

integration
?

unit
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 9 files with indirect coverage changes

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


:sparkles: add kubeflex provider to logical clusters

Summary
Add kubeflex provider to logical clusters.
Related issue(s)
This PR replaces PR #789 now that logical clusters have been merged to main.
Fixes #

/lgtm
/approve


[Bug] Update catalog with empty text

Updating an existing catalog with an empty help text field will not remove existing text in that entry
Steps to reproduce

Import an catalog from an xml file into rdmo
Edit the same catalog file which outside of rdmo (with a text editor) and replace the content of a help text entry <help lang="de">Lorem ...</help> with an empty <help lang="de"/> or <help lang="de"></help>
Import the edited (with empty entry)  catalog file
Select the catalog in the management UI and brows to the field which was replaced with an empty entry
Find the field containing the old content

Fixed.


Open ed etherpad recovery

See email thread with @soohyunpae.
I was just about to send a reminder about the next Open Ed Call in two weeks when I realized that the etherpad with 2 years full of notes is gone still. Any chance we can recover by next week so I send out the reminder for the meeting?
Would like to recover these two pads:
http://etherpad.creativecommons.org/p/Open_Education_Call
http://etherpad.creativecommons.org/p/SOO_Africa

I'll take a look.

Can we recover later this week so I can send out with reminder? Also, is the new pad.cc.org only accessible with a CC login now?

I've not succeeded yet so I'll raise it in my meeting with @mattl tomorrow.
Anyone with a CCID can log in. Anyone can get a CCID, if they don't have
one yet they can register here -
https://login.creativecommons.org/register.php
On Tue, Jul 7, 2015 at 2:06 PM, Jane<EMAIL_ADDRESS>wrote:

Can we recover later this week so I can send out with reminder? Also, is
the new pad.cc.org only accessible with a CC login now?
—
Reply to this email directly or view it on GitHub
https://github.com/creativecommons/creativecommons.org/issues/71#issuecomment-119341351
.

That was a bit more involved than expected. :-)
I've restored the content of the last version of each pad.

@robmyers please add a message at the top of every page on pad, telling
people it's not permanent and to keep a copy of the text elsewhere  :)

Added to creation and pad pages.
Changeset:
diff --git a/src/static/css/pad.css b/src/static/css/pad.css
index 49a54fb..53efbe8 100644
--- a/src/static/css/pad.css
+++ b/src/static/css/pad.css
@@ -211,7 +211,7 @@ li[data-key=showusers] > a #online_count {
#editorcontainer {
position: absolute;

top: 37px; /* + 1px border */

top: 56px; /* + 1px border */
left: 0px;
right: 0px;
bottom: 0px;
diff --git a/src/static/favicon.ico b/src/static/favicon.ico
index 938e955..483ac5d 100644
Binary files a/src/static/favicon.ico and b/src/static/favicon.ico differ
diff --git a/src/templates/index.html b/src/templates/index.html
index 626630e..a8887d3 100644
--- a/src/templates/index.html
+++ b/src/templates/index.html
@@ -159,7 +159,9 @@

  <% e.begin_block("indexWrapper"); %>

Please note that pads are not permanent - keep a copy of the text elsewhere!

           <buttOn id="button" onclick="go2Random()" data-l10n-id="index.newPad"></button>

           <form action="#" onsubmit="go2Name();return false;">

diff --git a/src/templates/pad.html b/src/templates/pad.html
index dd26041..21024b9 100644
--- a/src/templates/pad.html
+++ b/src/templates/pad.html
@@ -71,6 +71,7 @@
<%- toolbar.menu(settings.toolbar.right) %>
<% e.end_block(); %>

Please note that pads are not permanent - keep a copy of the text elsewhere!

   <% e.begin_block("afterEditbar"); %><% e.end_block(); %>

Great work, Rob.


fix 5.12 sys-kernel/surface-sources-5.12 emerge

A solution for issue #5 , tested on Surface Pro 4, may need improvement.

Right I can see how that would fix the problem. Thank you for your contribution, however, by removing the gentoo patches (if I am correct) we would be removing some of the essential options in the kernel, namely the "Gentoo Linux" menu.
Maybe it would be better if we were to decrement the K_GENPATCHES variable to a lower or higher version based on https://dev.gentoo.org/~mpagano/genpatches/releases-5.12.html.
Also, I believe this commit has something to do with it, you can take a look: https://github.com/Parinz/linux-surface-overlay/commit/100777a6c1c2ea1c40b230821a8a733309972454
Thank you very much for contributing and I hope you achieve a success.

Right I can see how that would fix the problem. Thank you for your contribution, however, by removing the gentoo patches (if I am correct) we would be removing some of the essential options in the kernel, namely the "Gentoo Linux" menu.
Maybe it would be better if we were to decrement the K_GENPATCHES variable to a lower or higher version based on https://dev.gentoo.org/~mpagano/genpatches/releases-5.12.html.
Also, I believe this commit has something to do with it, you can take a look: 100777a
Thank you very much for contributing and I hope you achieve a success.
PS: You can also generate manifests for your ebuild by doing pkgdev manifest -m, if it's not installed just emerge it.

I see, I think I found the right version of gen-patches.
For now, I added gentoo patches.
Thanks for your reminding.

Looks clean. You ready to merge?

Looks clean. You ready to merge?

Yes


Add Tokio support to FileOrStdin

Adding tokio feature to use tokio::fs::File and tokio::io::stdin
as suggested in #4.
- Adding `tokio` specific tests, and modifying examples/test_bins to
  test async usage.
- Not adding support to `MaybeStdin` since clap::parse() blocks and
  contents are read only once when args are parsed anyway

@anatawa12  Can you check the examples here and confirm that adding async support via tokio will work for your use-case?

It looks work well for my use case. Thank you for asking and sorry for lack of information and late response.


DEV-2470 - fix bug: per-job folder exclusion stopped working in Checkmarx jobs

After upgrading the Checkmarx plugin to version 8.5.0 -- or, at least, we noticed this problem after the upgrade, and it may be a coincidence -- the per-job folder exclusion stopped working, and jobs defaulted to the global exclusion settings. This restores the expected behavior.

👍

LGTM

👍

Can we merge?


Labels in search results

@sparkdevelopment can you review and sign off?

@marcincichon Looks good to go


Paint fixes (dylwhich's version)

Description
All of Bryce's fixes rebased onto current main, plus I got rid of the green plus in the image title that shouldn't have made it into the final version.
Test Instructions
Use paint, particularly the menu and saving and things, make sure there's no bugs.
Ticket Links

Readiness Checklist

[x] I have run make format to format the changes
[x] I have compiled the firmware and the changes have no warnings
[x] I have compiled the emulator and the changes have no warnings
[x] I have run make cppcheck and checked that cppcheck_result.txt has no warnings for the changes
[x] I have added doxygen comments to any code used by more than one Swadge mode. This includes /*! \file comments with Design Philosophy, Usage, and Example sections for new headers.
[x] I have run make docs and checked that doxy_warnings.txt has no warnings for the new code

Note to self, when this gets merged, also delete https://github.com/AEFeinstein/Swadge-IDF-5.0/tree/paint-fixes

I think that's everything? And apologies for the negative hideControls bool, but doing it that way means I only have to change it in one place instead of all the other places since it gets calloc()'d...

Everything as far as I can tell!


Upgrade to sbt-microsites 0.4.0

This adds support for MathJax and lets me remove the funky header from CONTRIBUTING.md.
(MathJax support is janky if you can't override the config. I'm talking with the sbt-microsites folks about a few changes we'll have to make in js load order, etc to get this working well.)

superseded by #648


HDDS-7450. Improve bucket creation message

https://issues.apache.org/jira/browse/HDDS-7450

correct the code style issue

@chrisGe123 is this just to improve the logging when creating a bucket via CLI?

I believe so @kerneltime, all changes are just related to logging during bucket creation via CLI


Update URL of string #22

As far I know, there is no IRC support. URL has vanished.

By submitting a request, you represent that you have the right to license
your contribution to Apple and the community, and agree that your
contributions are licensed under the Apache License Version 2.0.
For existing files modified by your request, you represent that you have
retained any existing copyright notices and licensing terms. For each new
file in your request, you represent that you have added to the file a
copyright notice (including the year and the copyright owner's name) and the
Calendar and Contacts Server's licensing terms.
Before submitting the request, please make sure that your request follows
the Calendar and Contacts Server's guidelines for contributing
code.

Both the IRC and Mailing List URLs should redirect from the old URL to the new one, and in my testing, they do:
andre@youbeill[~]curl -s --head https://trac.calendarserver.org/MailingLists.html/ | egrep 'HTTP|Location|Via'
HTTP/1.1 301 Moved Permanently
Location: https://www.calendarserver.org/MailingLists.html/
Via: https/1.1 ussjc2-edge-bx-013.ts.apple.com (ApacheTrafficServer/7.1.4)

andre@youbeill[~]curl -s --head https://trac.calendarserver.org/IRC.html/ | egrep 'HTTP|Location|Via'
HTTP/1.1 301 Moved Permanently
Location: https://www.calendarserver.org/IRC.html/
Via: https/1.1 ussjc2-edge-bx-011.ts.apple.com (ApacheTrafficServer/7.1.4)

I tested that from multiple network positions - let me know if the redirect isn't working for you.
I'll update the PR to include both URLs. The IRC channel isn't exactly a hub of bustling activity and development, but it does still exist :)


Procedural macros should automatically import the macros they depend on

When using Rust 2018 and the gdnative::methods procedural macro, users will get compile errors unless they manually import the macros it uses internally (godot_wrap_method, etc.).
The examples currently avoid this problem by either using glob imports or the old #[macro_use]. I personally dislike glob imports because I like to know where everything comes from, but I don't want to have to import macros I don't use directly.
Procedural macros are one aspect of Rust I don't know much about, so maybe this just isn't possible, but it would be nice.

That's a good point!
I think the easiest way is to use global paths internally, that way you shouldn't have to import anything.
Actually importing stuff would also be possible, but you might end up with duplicated imports and having to silence them. I am more in favor of the global paths.
Procedural macros are not "hygienic" yet. This means that they work more like C macros - their contents are just dumped into the rest of the file. Rust's declarative macros on the other hand are hygienic, meaning that if they create a variable a, this variable is not usable with that name outside of the macro and it also can't override an existing variable.
Basically all text generated from declarative macros has "a prefix" to it, so you can't mess it up. Procedural macros don't have that yet, so there might always be some issues regarding imports and how they behave when you rename a crate etc.
This should be easy enough to do though, I'll look into it soon, thanks! :blue_heart:

Stumbled on this when I got started. I also found that other people have encountered this issue and found this crate. It seems to work robustly, but I'm not sure.
https://docs.rs/proc-macro-crate/0.1.4/proc_macro_crate/

I think this was fixed by https://github.com/godot-rust/godot-rust/pull/425.


[Test server] body-byte.json

[ ] Code generated
[ ] Tests for all scenarios
[ ] API shape questions
[ ] Fix identified issues

Done.


drivers: pinctrl: add TI CC32XX driver

Add a new pinctrl driver for TI CC32XX SoC. The driver has not been
tested, just implemented following datasheet specs and checked that it
compiles. Consider this as a best-effort driver to remove custom pinmux
code in board files.
This PR is part of a series to cleanup SYS_INIT usage.

Thank you for the pull request @gmarull , I will test these changes later this week and update.
Sorry for not performing this cleanup earlier, I started working on this earlier[0], but did not get the bandwidth to complete and test the changes.
0 - https://github.com/vaishnavachath/zephyr/commit/60471f04756274a2e6788fb0bd43947aae9c3314


Cannot import 'requests.packages.urllib3'

This is a very odd bug for me, so I figured I'd post it here.
When I run  "python google_appengine/dev_appserver.py /home/jmschr/Desktop/myria-web/appengine", I get "ImportError("No module named 'requests.packages.urllib3'). The error comes from myria-web/appengine/requests/packages/__init__.py. If I go a Python terminal, I can import requests, requests.packages.urllib3, and urllib3 independently. However when I run this program through the appengine python program it seems to forget I have them.
I renamed the requests file seeing as I had requests already installed, and that didn't seem to help (it now couldn't find requests). I also try directly importing urllib3 into the namespace in packages/init.py, but it doesn't know what urllib3 is (even though it works for me in a python terminal).
Any suggestions?

Can you try the app engine launcher and see if the bug still occurs?

Daniel Halperin
Director of Research for Scalable Data Analytics
eScience Institute
University of Washington
On Wed, Feb 11, 2015 at 4:32 PM, Jacob Schreiber<EMAIL_ADDRESS>wrote:

This is a very odd bug for me, so I figured I'd post it here.
When I run "python google_appengine/dev_appserver.py
/home/jmschr/Desktop/myria-web/appengine", I get "ImportError("No module
named 'requests.packages.urllib3'. The error comes from
myria-web/appengine/requests/packages/init.py. If I go a Python
terminal, I can import requests, requests.packages.urllib3, and urllib3
independently. However when I run this program through the appengine python
program it seems to forget I have them.
I renamed the requests file seeing as I had requests already installed,
and that didn't seem to help (it now couldn't find requests). I also try
directly importing urllib3 into the namespace in packages/init.py, but
it doesn't know what urllib3 is (even though it works for me in a python
terminal).
Any suggestions?
—
Reply to this email directly or view it on GitHub
https://github.com/uwescience/myria-web/issues/249.

I can't find the app engine launcher, can you point me in the right direction?

https://www.google.com/search?q=app+engine+launcher
On Wed, Feb 11, 2015 at 4:52 PM, Jacob Schreiber<EMAIL_ADDRESS>wrote:

I can't find the app engine launcher, can you point me in the right
direction?
—
Reply to this email directly or view it on GitHub
https://github.com/uwescience/myria-web/issues/249#issuecomment-73997507
.

--
Bill Howe
Associate Director and Senior Data Science Fellow, UW eScience Institute
Affiliate Faculty, Computer Science & Engineering
University of Washington
To acknowledge eScience: "Supported in part by the University of Washington
eScience Institute"

Thanks Bill. I have that, and was using the dev_appserver.py script in it. Is there another script I should be using?

I guess the instructions aren't that helpful. Here's my experience on Mac:

from this link: https://cloud.google.com/appengine/downloads, I see this:

As part of downloading and installing that, it sets up an application shortcut

So the instructions are "obvious" to us... I'm guessing that maybe you're using a different platform?

I'm on Ubuntu and the app does not come natively with Google AppEngine.
I followed this thread to try to get it: http://askubuntu.com/questions/308534/google-app-engine-gui-launcher-for-ubuntu-13-04
I downloaded wxglade and wxversion as suggested, then checked out the repo using subversion as they suggested, giving me the google-appengine-launcher folder with GoogleAppEngineLauncher.py in it.
However, whenever I try to run GoogleAppEngineLauncher.py I get "wxPython is not available", even after using sudo apt-get install python-wxversion python-wxglade as was suggested.

Hmm. This isn't really something we can help debug, as none of us develop on Linux.
<overly honest>
I'm inclined to say "those who make themselves use awful tools [Linux] deserve to suffer the consequences"...  but that's probably not helpful.
</overly honest>
If one of us finds time, maybe someone in the @uwescience/myria team can find a way to get Desktop Linux up and running and debug the process of launching the development server. Unfortunately, I'm not sure it's going to be a top priority.

(I did send an email asking the group to check out this issue.)

I find your overly honest statement a bit ironic given that you don't have documentation for how to install and use Myria on a desktop system with a freely available iso.

I'm not going to dignify the laughable implication that that you didn't "opt-in" to Ubuntu... no need to feed the trolls.
I don't have access to Ubuntu desktop, but I did just go through the extraordinarily simple and fully documented myria-web setup steps (I skipped step 2, as it's not mandatory) on our Ubuntu server, and everything worked fine:
mkdir /disk1/sandbox
cd /disk1/sandbox
git clone https://github.com/uwescience/myria-web
wget https://storage.googleapis.com/appengine-sdks/featured/google_appengine_1.9.17.zip
unzip google_appengine_1.9.17.zip
cd myria-web
git submodule init
git submodule update
../google_appengine/dev_appserver.py --port=12345 appengine

and in a new window
curl -i localhost:12345

and it worked:
HTTP/1.1 301 Moved Permanently
location: http://localhost:12345/editor
cache-control: no-cache
content-type: text/html; charset=utf-8
Content-Length: 0
Server: Development/2.0
Date: Fri, 13 Feb 2015 02:13:19 GMT

Of course, you may not need to set the appengine port if your machine is not using the default port; our server is.
Seems like you have a personal problem.

Note that you should not need to install any packages in the Python system -- App Engine does not use them. The packages it uses are all included in the app engine kit or in the myria-web submodules.

That summary was excellent, it seems to have fixed whatever issue I was having. I would suggest putting that in the myria-web README you linked to. However, while it is working, I get repeatedly
INFO    2015-02-13 21:14:58,366 module.py:718] default: "POST {command} HTTP/1.1" 400 61
Unable to create 'parsetab.py'
[Errno 30] Read-only file system: 'parsetab.py'

where command in {/optimize, /dot, /compile, /plan}, ending with
WARNING    2015-02-13 21:15:00,535 myrialang.py:1238] Error converting Store(public:adhoc:JustX)[Apply(x=$0)[Scan(public:adhoc:TwitterK)]]: convert <class 'raco.alphebra.Store'> to sql
INFO    2015-02-13 21:15:00,541 module.py:718] default "POST /compile HTTP/1.1" 200 978

These don't seem to prevent me from loading the frontend in a browser though. The browser has a warning saying that Myria will be reserved for research experiments soon; can I go ahead and ignore that since I'm running on a local machine?

That is the aforementioned step 2 on the install docs. It is not a correctness error, only a performance hit
—
Sent from my phone
On Fri, Feb 13, 2015 at 1:21 PM, Jacob Schreiber<EMAIL_ADDRESS>wrote:

That summary was excellent, it seems to have fixed whatever issue I was having. I would suggest putting that in the myria-web README you linked to. However, while it is working, I get repeatedly
INFO    2015-02-13 21:14:58,366 module.py:718] default: "POST {command} HTTP/1.1" 400 61
Unable to create 'parsetab.py'
[Errno 30] Read-only file system: 'parsetab.py'

where command in {/optimize, /dot, /compile, /plan}, ending with
WARNING    2015-02-13 21:15:00,535 myrialang.py:1238] Error converting Store(public:adhoc:JustX)[Apply(x=$0)[Scan(public:adhoc:TwitterK)]]: convert <class 'raco.alphebra.Store'> to sql
INFO    2015-02-13 21:15:00,541 module.py:718] default "POST /compile HTTP/1.1" 200 978

These don't seem to prevent me from loading the frontend in a browser though. The browser has a warning saying that Myria will be reserved for research experiments soon; can I go ahead and ignore that since I'm running on a local machine?
Reply to this email directly or view it on GitHub:
https://github.com/uwescience/myria-web/issues/249#issuecomment-74328903

"can I go ahead and ignore that since I'm running on a local machine?" -- yes you're exactly right.
"[Errno 30] Read-only file system: 'parsetab.py'" -- hmm, read-only file system?

Read only because of the app engine sandbox
—
Sent from my phone
On Fri, Feb 13, 2015 at 1:37 PM, jingjingwang<EMAIL_ADDRESS>wrote:

"can I go ahead and ignore that since I'm running on a local machine?" -- yes you're exactly right.
"[Errno 30] Read-only file system: 'parsetab.py'" -- hmm, read-only file system?
Reply to this email directly or view it on GitHub:
https://github.com/uwescience/myria-web/issues/249#issuecomment-74331246

I'm going to go ahead and close this issue then, since it's been resolved. Thank you!

Can you explain what step in there was not directly implied by the instructions in the install step? The clone and fixing submodules were explicit, the "run dev_appserver with path to appengine" just requires getting the path correct.
What were you doing differently?

I likely downloaded the incorrect google appengine and/or did not handle working with it correctly. Having a specific wget/unzip command was super useful to me, as I am not familiar with using it at all prior to this.


How to configure basic auth reverse proxy to prometheus

Hi,
I'd like to control access to Prometheus through Openunison reverse-proxy.   We have the following configuration for implementing oidc but how do I configure this for basic auth to Premetheus:
apiVersion: openunison.tremolo.io/v1
kind: Trust
metadata:
  name: {{ .Values.grafana.client }}
  namespace: openunison
  labels:
    app.kubernetes.io/name: openunison
    app.kubernetes.io/instance: openunison-orchestra
    app.kubernetes.io/component: grafanaacces
    app.kubernetes.io/part-of: openunison
spec:
  accessTokenSkewMillis: 120000
  accessTokenTimeToLive: 60000
  authChainName: login-service
  clientId: {{ .Values.grafana.client }}
  clientSecret:
    keyName: {{ .Values..grafana.client }}
    secretName: <secretname>
  codeLastMileKeyName: lastmile-oidc
  codeTokenSkewMilis: 60000
  publicEndpoint: false
  redirectURI:
  - https://{{ .Values.grafana.redirect }}
  signedUserInfo: false
  verifyRedirect: true

Prometheus through Openunison reverse-proxy.

Are you using openunison's reverse proxy now, or another reverse proxy?

We have the following configuration for implementing oidc but how do I configure this for basic auth to Premetheus:

Do you want it to work with both? (as in you want a remote service to use basic auth but users to still use OpenUnison's auth login form?)

Yes openunison.   Yes if that's possible - assuming you mean we access one application using one method and another applicaiton using another.
User ===OIDC ====> openunison(impersonation) ====basiic auth ====>prometheus
User ===OIDC ====> openunison(impersonation) ====lastmile-oidc====>grafana

User ===OIDC ====> openunison(impersonation) ====basiic auth ====>prometheus

ok, to make sure I understand:
https://prometheus.domain.com/ --> OIDC Login
https://prometheus-api.domain.com --> Basic Auth
?
Also, I assume you're using LDAP to authenticate your users?

Users login using OIDC.   Redirected to pingidentity, authenticated, callback,  and retrieves groups from user-info.
If I understand it, it can then use other authn methods to access the applications behind openunison.

If I understand it, it can then use other authn methods to access the applications behind openunison.

Yes, but how?  If you're using basic auth, OpenUnison needs to validate the credential against something.

I'm not sure how that would work.
So we want users who are members of at least one group, say "admin" group to have access to prometheus( seems only basic auth is the only access control for prometheus).
I'm guessing that OpenU could be cnfigured to access Promethues using basic auth.  But TBH I'm not at all clear how it can control say members of a given groups.   Not sure either how forwarding basic auth to ping would work.

OH!  Ok, I misunderstood what you were trying to do. You want to limit who has access to Prometheus based on a specific group coming from Ping?

yep sorry that's probably how I should have put it :)

Ok, this is much easier.  You're using OpenUnison as the reverse proxy right?  You should have an Application for it (if you followed the instructions from https://openunison.github.io/documentation/custom-sso/#using-a-new-host it's probably called prometheus There's a section called azRules that probably looks like:
    azRules:
    - scope: dn
      constraint: o=Tremolo

This means anyone who's authenticated can access it (there's an authorization rule on the identity provider too).  You can change this to:
    azRules:
    - scope: filter
      constraint: (group=my-prometheus-group)

and it will stop access for anyone that isn't in the group my-prometheus-group


feat: add ThreeDTiltFx component with 3D tilt effect and styling

This pull request introduces a new ThreeDTiltFx component, which adds a 3D tilt effect to elements. The most important changes include the creation of the component, its associated styles, and its export in the component index.
New ThreeDTiltFx component:

src/once-ui/components/ThreeDTiltFx.tsx: Implemented the ThreeDTiltFx component, which applies a 3D tilt effect to its children based on mouse movement. The component includes customizable properties such as maxTilt, dampening, and transitionDuration.

Associated styles:

src/once-ui/components/ThreeDTiltFx.module.scss: Added styles for the ThreeDTiltFx component to ensure the 3D transform is applied correctly and improve performance.

Component export:

src/once-ui/components/index.ts: Exported the ThreeDTiltFx component to make it available for use in other parts of the application. [1] [2]

Demo:
https://github.com/user-attachments/assets/e9880e94-5165-4842-84a8-b8bd82f5d6e2

Oh just remembered you announced TiltFx in the community event lol. You can close this PR.

yeah that's a coincidence @lightyfr but i'll incorporate some of the props you used for yours to improve customization ;)


Vehicle detection problem

Hi guys
we know that this model (ssd_mobilenet_v1_coco_2017_11_17) is general object detection problem  which changes we perform to detect only vehicle like in this project
Thanks

You can work on "Transfer Learning" or you can train the MobileNet-SSD from scratch!


LinkedIn Strategy (another package)

Hi, how are you?
I'm just passing here to show you I made a package for LinkedIn Strategy.
I red your message in this other issue of yours and you have valid points.
I documented the code, made tests and export some other things among the main class. Here is the source code to look at it
Let me know how you see it, if you consider i am missing something for you so you can re-export from there. Feedback and suggestions are welcome.
Have a nice Saturday!

Hey!
Sorry I took so long on this. I will be re-exporting third party packages soon!

Any update on the LinkedIn Strategy export?

@Lzok and @bluefire2121 boy am I sorry this took so long!!!!!
I've just had all the social auth packages (bar facebook) approve my PR's to ensure they offer a consistent experience, and released V2 of the package.
@Lzok  have a look at the google package and once you've updated remix-auth-linkedin to match the experience open a PR to add it here :)
We just need scopes as either scopes?: LinkedinScopes[] | string as well as a LinkedInStrategyDefaultName export... Essentially just follow the pattern and we'll be good :)


provider: add a query free function

Providers are allowed to return non-cached OSSL_DISPATCH tables.  These can be dynamically allocated.  There is no way for a provider to know when an application has finished with these.  Adding a free function allows an application to inform the provider that it is done with the table.

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

Fixes #11771

Do you need to add to the commit
Fixes #11771

I don't think so.  It should be enough in the message here.

we will see if another reviewer agrees :)

I don't think so. It should be enough in the message here.

That's not quite right...  it should be in the commit message.  Among others, that's what triggers autoclosing of the issue.

I don't think Travis is related -- the exact same failure occurs in other unrelated PRs.

Ping?

Giving the disputes for the behavior of non-cached dispatch tables this late in 3.0 development, wouldn't it be better to postpone the support of them to 3.1?
I do not see this as critically needed feature.

Currently, the unquery is a NOP for all of our providers.  Leaving this out won't hurt us.
I think it will be useful for PKCS#11 support but that's slated for later.

Closing this unrequited PR.

This is to be reconsidered by the OTC.

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 61 days ago

Rebased to master (cleanly even)

Changing the assignee, @levitte and I are to work this out.  The OTC decision was that we need to support this functionality.

Review anyone?

Seems to be missing a test?

A test turned out to be straightforward by piggybacking it onto the SSL API test & the filter provider.

Ping for review

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.


[ts lint] unable to find module references in TS project.

VSCode Version: 0.10.12-alpha
OS Version: Windows10

Steps to Reproduce:

Create "Started Node.js Express3" Type script project from Visual studio and build it.
Launch VS code and open the same folder to get node.js TS project.
Verify app.ts, express.d.ts and other files under script folder.

Actual: TS lint is highlighted red with message "Cannot find the module".
Expected: TS lint should find the references of modules and  should not flag red and green.

Note: I tied with C# class library project and reference to the function from one namespace to other under different projects are working fine and no error message.

@v-pavanp Is this the tslint extension or the typescript tools that are bundled with vs code? please attach the reading of the diagnostics message

It is TS: Under app.ts.

Also tslint is showing message like below under: express.d.ts

@v-pavanp Please not that tslint and ts are different project and that it would be helpful if you file your bugs accordingly (https://github.com/Microsoft/vscode-tslint,https://github.com/Microsoft/vscode)

The module reference error do not come from tslint, tslint only reports warnings.

I am closing this issue as I realized that I didn't add the reference in .ts file for external modules.
///
///
///
After adding above references there is no error in my .ts file.


data encryption

#418
And I think we need to make app/config folder for configuration on client side
@jrodl3r @choonkending @caranicas @psimyn

Awesome!
Yep I've also been thinking of a client side config.
I have 2 branches I want merged first :

meta tag refactor (with react - helmet)
fetch data on route change

Both of these branches make significant folder changes so I don't want to step on anyone's changes.
I wonder whether this as a performance impact.

@choonkending Okay. I'll wait until you merge those two branches and change my codes after it

I'll start testing this on my end

@ZeroCho This works like a charm so far on latest build - Nice job man! that technique you used to encrypt/decrypt was clutch as well 👍

just noticed error on test after adding encryption to GET + reducers:
ERROR in ./app/images/favicon.png
Module parse failed: /Users/xxx/app/images/favicon.png Unexpected character '' (1:0)
You may need an appropriate loader to handle this file type.
SyntaxError: Unexpected character '' (1:0)

@ZeroCho what about just adding a Heroku config-var and doing this (instead of modifying helmconfig.js)?
const encryptKey = process.env.ENCRYPT_KEY || 'XYZ789';

I suppose you could stuff that into the helmconfig, or another config file - did you mention that?

@jrodl3r I also think that putting encrypt_key in helmconfig.js is not appropriate. So I think it will be good if there is a separate config file.
And about the error, I'll test it. Anyway, I have to change this PR because this has conflicts.

@ZeroCho should we also have an enabled/disabled FLAG in the config to allow for easy on/off [off by default] control of encryption?

@jrodl3r made appConfig files and modularized crypto-js to decrease size. Could you please review it?

Looks good, will verify ASAP ; )

also wondering if we can use:
webpack.DefinePlugin({ __CRYPT_KEY__: process.env.CRYPT_KEY })
to store the key as a Heroku config-var... and then inside appConfig:
export const encryptKey = __CRYPT_KEY__ || 'ReactGo';

getting SyntaxError: Unexpected token export on appConfig.js for PROD Server

@jrodl3r SyntaxError: Unexpected token export might not be this PR's problem. It's webpack's problem I guess.
webpack.DefinePlugin({ __CRYPT_KEY__: process.env.CRYPT_KEY })
I think it's a good idea.

Ok. the problem I was having in server.jsx was because I was missing the quote tick marks on the send(...), so using this:
const stateData = encryption
  ? "'" + AES.encrypt(JSON.stringify(initialState), encryptKey).toString() + "'"
  : JSON.stringify(initialState);

and then removing the single quotes here:
...<script>window.__INITIAL_STATE__ = ${stateData};</script>...

@choonkending I think this should be prioritized due to XSS possibility
Link

@ZeroCho Most definitely! Good point ! Added to the project wall!


move Type grammar from declaration.dd to type.dd

It's kinda where one would expect to find it.
Didn't change anything else other than adjusting links.

Great, I was thinking of doing this too. BTW this has broken some links, I'm working on a pull to fix them.


contact info

Is it possible to contact you with research/business enquiries about FOM?

You can write me an email. My email is<EMAIL_ADDRESS>


atest folder update to new jsonp extension

Might it makes sense to adapt the
python-jsonpreprocessor\atest
sources to the new extension jsonp?

Hi Holger,
jsonpreprocessor should be generic and not dependent on any file-ending.
hm...
Maybe we make one simple and one complex (with multiple imports) tests if all works as expected if we use jsonp as file ending.
Thank you,
Thomas

Hi Thomas,
OK, but consider: Using the new extension jsonp also in test code would avoid invalid findings in VSCodium error.
Or we make it a certain hint in documentation.

Possibility 1: Avoid every preprocessor specific syntax extensions when using extension json together with standard editors or standard JSON syntax checkers.
Possibility 2: Switch to extension jsonp and use adapted language definitions like available as extension for VSCodium.

Hi Holger,
O.K. I agree w.r.t. VSCodium.
Actually jsonpreprocessor introduces the syntax, therefore tests should be written as jsonp files, too.
But important is that the jsonpreprocessor code has zero dependency on the extension name.
Thank you,
Thomas

Hello Thomas,
I created the commit 1ad5fb86c on stabi branch for this ticket.
Thank you,
Son

merged to develop branch