[Bug] Can't consume messages by using a subset of sticky key hash range

Search before asking

[X] I searched in the issues and found nothing similar.

Version
2.10.0
Minimal reproduce step
Here are a reproduce code by adding test on KeySharedSubscriptionTest class.
@Test
public void testSpecifiedStickyKeyRanges() throws PulsarClientException, InterruptedException {
    String topic = TopicName.get("persistent", "public", "default",
            "testSpecifiedStickyKeyRanges" + UUID.randomUUID()).toString();
    String subscriptionName = "my-sub";

    String keyToRead = "someKey1";
    String keyToExclude = "someKey2";
    int numMessages = 100;
    byte[] keyBytes = keyToRead.getBytes(StandardCharsets.UTF_8);
    int hash = Murmur3_32Hash.getInstance().makeHash(keyBytes) % KeySharedPolicy.DEFAULT_HASH_RANGE_SIZE;

    // start consumer
    CountDownLatch count = new CountDownLatch(numMessages);
    Consumer<String> consumer = pulsarClient.newConsumer(Schema.STRING)
            .topic(topic)
            .subscriptionName(subscriptionName)
            .subscriptionInitialPosition(SubscriptionInitialPosition.Earliest)
            .subscriptionType(SubscriptionType.Key_Shared)
            .keySharedPolicy(KeySharedPolicy.stickyHashRange().ranges(Range.of(hash, hash)))
            .messageListener((c, msg) -> {
                c.acknowledgeAsync(msg).whenComplete((m, e) -> {
                    if (e != null) {
                        log.error("error", e);
                    } else {
                        count.countDown();
                    }
                });
            })
            .subscribe();

    pulsar.getExecutor().submit(() -> {
        try {
            try (Producer<String> producer = pulsarClient.newProducer(Schema.STRING)
                    .topic(topic)
                    .enableBatching(false)
                    .create();) {
                for (int i = 0; i < numMessages; i++) {
                    String value = "test" + i;
                    producer.newMessage()
                            .key(keyToRead)
                            .value(value)
                            .send();
                    producer.newMessage()
                            .key(keyToExclude)
                            .value(value)
                            .send();
                    Thread.sleep(100);
                }
            }
        } catch (Throwable t) {
            log.error("error", t);
        }
    });

    assertThat(CompletableFuture.runAsync(() -> {
        try {
            count.await();
        } catch (InterruptedException e) {
            Thread.currentThread().interrupt();
            throw new RuntimeException(e);
        }
    })).succeedsWithin(Duration.ofMinutes(1));

    consumer.close();
}

What did you expect to see?
Consumed the sent messages with given key.
What did you see instead?
No messages can be consumed.
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

@eolivelli I got the reproduce test code passed after comment the code you added in #14014.
        for (Entry entry : entries) {
            int stickyKeyHash = getStickyKeyHash(entry);
            Consumer c = selector.select(stickyKeyHash);
            if (c != null) {
                groupedEntries.computeIfAbsent(c, k -> new ArrayList<>()).add(entry);
-                consumerStickyKeyHashesMap.computeIfAbsent(c, k -> new HashSet<>()).add(stickyKeyHash);
            } else {
                addMessageToReplay(entry.getLedgerId(), entry.getEntryId(), stickyKeyHash);
                entry.release();
            }
        }

For sticky mode, the consumer will only receive messages of hashes in the specified ranges.
In the above case, the consumer only config one hash range. But the produced messages hash are in other ranges, so the consumer is blocked.

Yep, you are right. I have found another workaround for this issue.
https://github.com/streamnative/flink/blob/pull/FLINK-26182/flink-connectors/flink-connector-pulsar/src/main/java/org/apache/flink/connector/pulsar/source/config/PulsarConsumerBuilder.java


Unable to retrieve child resources: The request is not authorized to perform this action

Storage Explorer Version: 1.15.1
Build Number: 20200902.9
Platform/OS: Windows 10 latest updates
Architecture: x64
Regression From:  Unsure
Bug Description
On several of my storage accounts -- all of which I am an OWNER, I am unable to list the blobs, whether I am using login credentials or a storage account key. Under "Blob Containers", only "Load more" is displayed. When I click it, I get the popup error: Unable to retrieve child resources.
Details:
["RestError:AuthorizationFailureThis request is not authorized to perform this operation.\nRequestId:b5709647-d01e-012b-1ad6-8fe7d1000000\nTime:2020-09-21T05:20:42.7763647Z"]
Steps to Reproduce
Add storage account using master key (NOT SAS key).
Expand to see contents.
Click "Load more"...
Expected Experience
A list of the containers should be displayed.
Actual Experience
Error as noted above.

On further investigation, it was a storage account firewall configuration issue.  A more helpful error message would have helped here. I.e., if this error occurs when attempting to list the container objects, then include some additional text such as "This may be due a storage account firewall rule".


Print a help message when running make without arguments

This lists all available targets.

Do you have a fold executable on your fine macbook?

I actually do!
But I meant this one: https://github.com/trustlines-protocol/blockchain/pull/192/files#diff-b67911656ef5d18c4ae36cb6741b7965R9

Don't worry, I've copied that from stackoverflow.


feat(refactor): refactor group bbox and tag

Part of issue #250. Update group bbox and tag functions to use newer implementation

LGTM


Tomes for time calculation

Currently unless I'm missing a toggle I need to switch on the time tab accounts for potions and raw resources but it fails to account for tomes. I.e. it doesn't calculate for stardust(Opulent Crafting Tome) and food(Insatiable Power Tome). Although a perfect calculation cannot be made due to randomness of these consumables I would still prefer and average estimate if possible.

Hey!
The Opulent Crafting Tome should be solved in v4.13.1
The Insatiable Power Tome should be working AFAIK.
Could you confirm you are not seeing any food use under the "Items" tab when having it equipped?


Firefox: Text selection, Carets, & Scrollbars

Applies Doki definition themes for carets, scrollbars, & text selection.
Update styles for popup menu.
Fixes switches in popup menu not disabling properly for both secondary and dark.
and more...

Description

This pull request attempts to solve issue #59.

In Chrome the Scroll bar and marked Text changes with the Theme you chose. And my question is if its possible to have that feature in Firefox and if yes how to get it.

Scrollbar, Caret, Text Selection
Using scrollbar-width and scrollbar-color, the scrollbars can be modified without generating a more complex solution. These CSS properties are only compatible for Firefox. Of course there's a ::-webkit-scrollbar variant which is compatible for Chromium browsers. The carets and text selection components also have their own CSS properties and selectors. For carets, caret-color. For text selection, ::selection.
Instead of creating a <style> & placing it in the <head>, the previously mentioned CSS properties are added to the .css
& then are modified using CSS variables.
Content scripts are used to load scripts onto <all_urls> pages. In this case, the scripts injects the scrollbar, caret & text selection CSS properties onto all websites. The scrollbar and text selection (caret is coupled with text selection) are enabled by default and can be disabled in the options page.
Popup Menu Styles
Some styling was added to the popup menu. Deactivated, (not disabled & not checked), switches receive a nice glow providing depth and recognition that this particular switch is toggleable. The glow depends on whether the theme is light or dark. The Doki logo also has a glowing effect when hovered. The same glow rules apply here. Additionally, the Doki logo transitions to a reddish color when hovered providing a visual affordance.
Popup Menu Functionality
In accordance to the glowing effect, clicking on the Doki logo navigates you directly to the options page. Makes for a smoother user experience. The switches are now properly disabled when certain background variants of a theme are not available. For instance, a theme might have only a dark variant, therefore the secondary and dark switches will be disabled as there are no other variant available for this theme.
Honorable Mentions

Random mode was not functioning so its been repaired.
Removes deprecated colors.toolbar_field_separator theme key.
Adjusted CONTRIBUTING.md a bit.

Motivation and Context

This provides the appropriate enhancements mentioned in issue #59.
Screenshots (if appropriate):

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Non-Functional Change (non-user facing changes)

Checklist:

[x] I updated the version (base package.json and re-built the master extension).
[x] I updated the changelog with the new functionality.

Hey, Thank You! I'll probably be able to check this out later today or tomorrow.

@Unthrottled I think that's about it for the changes unless there's something else I'm missing? I know that in the next pull request the -

Could we use the transparency accent color for the blur/dropshadow effects in the popup, please?

has been completed. As well as the -

That one is on me. The background get copied over from my asset repository during the buildThemes process. I can modify and test things out, because I probably am not going to make a different secondary Kanna background.

will be taken care in the build tool.

Alright, I'll have some time to look at this again tomorrow, thank you for these contributions!


transform: emit file event on output stream

This serves as a prelude to #55 - allowing a potential external package to keep a cache of packages and recombine them as needed. Thanks!

looks good to me :+1:

Thanks! Released as v4.1.0


⚠ Bofrost.be has degraded performance

In 3adef67, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1389 ms

Resolved: Bofrost.be performance has improved in 276a0df after 8 minutes.


Persistent message map

Persistent message map
Resolves #541

@42wim With all the open PRs I have the branches are starting to get messy, can we start to get some of the PRs merged? (Whenever you're available)
Some things kind of depend on others. As of now my test branch has a pretty clean merge of all of my open PRs and is what I'm running personally.

@42wim With all the open PRs I have the branches are starting to get messy, can we start to get some of the PRs merged? (Whenever you're available) Some things kind of depend on others. As of now my test branch has a pretty clean merge of all of my open PRs and is what I'm running personally.

@yousefmansy1 I'm merging them, thanks for your work on those PRs and sorry for the delays.

@42wim could you return back here, please (:
The implementation of the persistent message map seems more than a significant upgrade to matterbridge!

@42wim any news with PR review?

@42wim ?


Load stdin lazily

I encountered this while tracking a bug in an Atom extension I used. Apparently, in Atom accessing process.stdin throws an exception. This makes a huge library unusable there, since calling require('get-stdin') accesses process.stdin, even though the extension doesn't use it.

Can you open an issue on Atom first? And comment a link here. process.stdin is expected to exist in the Node.js ecosystem, so this is an Atom bug.

Ok, I've researched this a bit and it seems like it has something to do with how windows treats desktop apps, so I don't think it will be fixed anytime soon.
Disregarding windows quirks for a moment, I still think it's a good idea to load stdin lazily, if only to avoid  running the stdin initialization code unnecessarily.

Fixed on recent versions of electron.


Update instructions on adding operators

I noticed it's hard to document operators while more are being added constantly. We should have a better check list to define when an operator is "done". Here are some things to do when adding an operator, we should have these in CONTRIBUTING.md or something like that:

Should be added to either core or kitchen sink
Should have -spec.ts tests covering the canonical corner cases, with marble diagram tests
Should have a asDiagram test case, if possible
Should have a type definition test at the end of spec to verify type definition for various use cases
Should have documentation in JSDoc style, including also the PNG marble diagram img
Should be listed in doc/operators.md in a category of operators
Should be put in the operator decision tree file doc/decision-tree-widget/tree.yml
Update MIGRATION.md if it covers migration
(maybe something else that I forgot, please mention if you remember it)

+1 for this.

update MIGRATION.md if it covers migration

if https://github.com/ReactiveX/RxJS/pull/1414 is being landed :

add type definition test at the end of spec to verify type definition for various use cases

Maybe should list all of this as checklist in github template for PR? (https://github.com/ReactiveX/RxJS/issues/1355)

Thanks for those two, I updated the list above.

Maybe should list all of this as checklist in github template for PR? (#1355)

Agreed

Closed via https://github.com/ReactiveX/RxJS/pull/1432.


KeyError: 'BTTEUR'

Hi,
My base currency is set to EUR and from time to time I get the below error for various pairs  and the bot stops completely
Traceback (most recent call last):
  File "C:\TradingBot\Binance-volatility-trading-bot\Binance Detect Moonings.py", line 592, in <module>
    orders, last_price, volume = buy()
  File "C:\TradingBot\Binance-volatility-trading-bot\Binance Detect Moonings.py", line 292, in buy
    volume, last_price = convert_volume()
  File "C:\TradingBot\Binance-volatility-trading-bot\Binance Detect Moonings.py", line 274, in convert_volume
    volume[coin] = float(QUANTITY / float(last_price[coin]['price']))
KeyError: 'BTTEUR'

Here is my config.yml

# These options apply to how the script will operate.
script_options:
  # Switch between testnet and mainnet
  # Setting this to False will use REAL funds, use at your own risk
  TEST_MODE: False
  LOG_TRADES: True
  LOG_FILE: 'trades.txt'

  # Set this to true if you are accessing binance from within the United States of America
  # Need to change TLD
  AMERICAN_USER: False

# These options apply to the trading methods the script executes
trading_options:

  # select your base currency to use for trading (trade for example USDT pairs)
  PAIR_WITH: EUR

  # Total amount per trade (your base currency balance must be at least MAX_COINS * QUANTITY)
  # Binance uses a minimum of 10 USDT per trade, add a bit extra to enable selling if the price drops.
  # Recommended: no less than 12 USDT. Suggested: 15 or more.
  QUANTITY: 15

  # List of trading pairs to exclude
  # by default we're excluding the most popular fiat pairs
  FIATS:
    - EURUSDT
    - GBPUSDT
    - JPYUSDT
    - USDUSDT
    - DOWN
    - UP

  # Maximum number of trade 'slots' at any time (your USDT balance must be at least MAX_COINS * QUANTITY)
  MAX_COINS: 1

  # the amount of time in MINUTES to calculate the difference from the current price (minimum: 1).
  TIME_DIFFERENCE: 2

  # Number of times to check for TP/SL during each TIME_DIFFERENCE (minimum: 1).
  # Don't spam the Binance API, you will be banned (max 1200 requests per minute per IP).
  RECHECK_INTERVAL: 10

  # the difference in % between the first and second checks for the price.
  CHANGE_IN_PRICE: 10

  # define in % when to sell a coin that's not making a profit.
  STOP_LOSS: 5

  # define in % when to take profit on a profitable coin.
  TAKE_PROFIT: .8

  # Use custom tickers.txt list for filtering pairs.
  CUSTOM_LIST: True

  # Name of custom tickers list
  TICKERS_LIST: 'tickers.txt'

  # whether to use trailing stop loss or not; default is True
  USE_TRAILING_STOP_LOSS: True

  # when hit TAKE_PROFIT, move STOP_LOSS to TRAILING_STOP_LOSS percentage points below TAKE_PROFIT hence locking in profit
  # when hit TAKE_PROFIT, move TAKE_PROFIT up by TRAILING_TAKE_PROFIT percentage points
  TRAILING_STOP_LOSS: .4
  TRAILING_TAKE_PROFIT: .1

  # Trading fee in % per trade.
  # If using 0.75% (using BNB for fees) you must have BNB in your account to cover trading fees.
  # If using BNB for fees, it MUST be enabled in your Binance 'Dashboard' page (checkbox).
  TRADING_FEE: .075

  SIGNALLING_MODULES:
    - pausebotmod
    - signalsamplemod

Any hints would be much appreciated
Thanks,
C.

Same problem ;
Traceback (most recent call last):
  File "C:\binance-bot\Binance Detect Moonings.py", line 592, in <module>
    orders, last_price, volume = buy()
  File "C:\binance-bot\Binance Detect Moonings.py", line 292, in buy
    volume, last_price = convert_volume()
  File "C:\binance-bot\Binance Detect Moonings.py", line 274, in convert_volume
    volume[coin] = float(QUANTITY / float(last_price[coin]['price']))
KeyError: 'TOMOUSDT'

anyone who can help ?

Same problem on linux:
Traceback (most recent call last):
  File "/home/cantos/Downloads/Binance-volatility-trading-bot-main/Binance Detect Moonings.py", line 592, in <module>
    orders, last_price, volume = buy()
  File "/home/cantos/Downloads/Binance-volatility-trading-bot-main/Binance Detect Moonings.py", line 292, in buy
    volume, last_price = convert_volume()
  File "/home/cantos/Downloads/Binance-volatility-trading-bot-main/Binance Detect Moonings.py", line 274, in convert_volume
    volume[coin] = float(QUANTITY / float(last_price[coin]['price']))
KeyError: 'ICPUSDT'

I believe this is happening because of external_signals() being used to add coins to the volitile_coins list, but not to the historical_prices list - which causes the key error when referencing the coin in last_price (since that's built from historical_prices).
Created PR #181 as a quick-fix similar to the previous try/except fixes


Failed to get JSON resource

Hello @zzossig , thanks for the good and free job. I followed the installation guide, cd'd to themes/zzo/exampleSite and ran  hugo server --themesDir ../.. as displayed. The following was what was the response

and on clicking on the link in the logged error, I got:

Please, your assistance or advice is well appreciated in advance.

See #339 tldr; Instagram changed their API and Hugo will need an update (prolly remove /move the instagram shortcode to a plugin).

Thanks. Deleted the Instagram shortcode in content/en/posts/rich-content.md according to #339 and that was it.

Thanks. Deleted the Instagram shortcode in content/en/posts/rich-content.md according to #339 and that was it.


Not compatible with NextJs and React

When I try add the package to my project with nextjs and react I recieved the following error after start:

Location: "/mnt/c/Users/Angel Igareta/Desktop/AteCloud/ecommerce/packages/shop/babel.config.js"
error - /mnt/c/Users/Angel Igareta/Desktop/AteCloud/ecommerce/node_modules/easyinvoice/dist/easyinvoice.js
Module not found: Can't resolve 'fs' in '/mnt/c/Users/Angel Igareta/Desktop/AteCloud/ecommerce/node_modules/easyinvoice/dist'

Hi! Thanks for contacting us. This error should not occur indeed. Please give us some time to fix this. Shouldn’t take more than about 24 hours.

Hi! The problem should be fixed now. Please not pdfjs dependency was causing this issue, so rather than building it into our package we made it a peer dependency. In order to use the 'render()' function you will have to install the peer dependency yourself or include pdfjs 2.3.200 using a cdn. Please see the the readme for this.
Could you let us know if this fixed your problem? If you experience any other issues, please let us know. Thank you!

Hi, I have tried it and it work fine if you move is-base64 and axios dependencies from devDependencies to dependencies to have them installed when adding the package to the project. Thank you very much for your efficiency and support and good luck with the project!

Hi, we deliberately choose to include axios and is-base64 within the build js file so it can easily be required through CDN. It should also work for you without installing those dependencies yourself. Didn't Easyinvoice work for without installing them?

It did not, the project required those packages in order to run easyinvoice package. As they are used in production, they should be included in dependencies.

That doesn't really make sense though. Because that would mean that these examples shouldn't work, but they do:
https://jsfiddle.net/easyinvoice/qfs8dk0p/
Only the easyinvoice.min.js is being required there.

Yes, but I did not include the CDN, packages in react should be managed with package json

Agreed, but then still the library should work by just importing the library like so:
import easyinvoice from 'easyinvoice';
I have a working example on my local without those packages installed.
Like I said, axios and is-base64 are build within the easyinvoice.min.js
Still, I'll start a small investigation on my local again. See if I missed something.

Seems to work fine here. My steps:

I just created a clean React project
Ran 'npm install easyinvoice'
Added import easyinvoice from 'easyinvoice'; to the default page
Added a two lines of code to run on page load easyinvoice.createInvoice({}) and console.log(result.pdf);
Built the project
Put the build output on our dev server
Opened the page in the browser. Gave me a successful result straight away. A nice base64 string.

Axios and is-base64 are nowhere to be found in node_modules folder. So it seems there is no need to put them in dependencies.

I think that is because of npm, if you use yarn it only adds the production dependencies, try using yarn add instead.

I will try, give me a sec

I repeated above steps. Only changed npm install to yarn add easyinvoice. No Axios or is-base64 in node_modules. Ran fine locally and after building on the dev server.

That's odd because on my next + react + ts project I could not make it work without adding them as dependencies.
I leave the error here just in case it helps:

error - Error: Cannot find module 'axios'
Require stack:

/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/easyinvoice/index.js
/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/next-server/server/load-components.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/next-server/server/api-utils.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/next-server/server/next-server.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/server/next.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/server/lib/start-server.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/cli/next-dev.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/bin/next
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:966:15)
at Function.Module._load (internal/modules/cjs/loader.js:842:27)
at Module.require (internal/modules/cjs/loader.js:1026:19)
at require (internal/modules/cjs/helpers.js:72:18)
at Object. (/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/easyinvoice/index.js:1:13)
at Module._compile (internal/modules/cjs/loader.js:1138:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1158:10)
at Module.load (internal/modules/cjs/loader.js:986:32)
at Function.Module._load (internal/modules/cjs/loader.js:879:14)
at Module.require (internal/modules/cjs/loader.js:1026:19)
at require (internal/modules/cjs/helpers.js:72:18)
at Object.easyinvoice (/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js:21059:18)
at webpack_require (/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js:29:31)
at Module../utils/order.ts (/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js:20714:69)
at webpack_require (/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js:29:31)
at Module../utils/index.ts (/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js:20679:65) {
code: 'MODULE_NOT_FOUND',
requireStack: [
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/easyinvoice/index.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/next-server/server/load-components.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/next-server/server/api-utils.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/next-server/server/next-server.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/server/next.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/server/lib/start-server.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/cli/next-dev.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/bin/next'
]
}
Error: Cannot find module 'axios'
Require stack:
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/easyinvoice/index.js
/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/next-server/server/load-components.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/next-server/server/api-utils.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/next-server/server/next-server.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/server/next.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/server/lib/start-server.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/cli/next-dev.js
/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/bin/next
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:966:15)
at Function.Module._load (internal/modules/cjs/loader.js:842:27)
at Module.require (internal/modules/cjs/loader.js:1026:19)
at require (internal/modules/cjs/helpers.js:72:18)
at Object. (/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/easyinvoice/index.js:1:13)
at Module._compile (internal/modules/cjs/loader.js:1138:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1158:10)
at Module.load (internal/modules/cjs/loader.js:986:32)
at Function.Module._load (internal/modules/cjs/loader.js:879:14)
at Module.require (internal/modules/cjs/loader.js:1026:19)
at require (internal/modules/cjs/helpers.js:72:18)
at Object.easyinvoice (/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js:21059:18)
at webpack_require (/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js:29:31)
at Module../utils/order.ts (/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js:20714:69)
at webpack_require (/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js:29:31)
at Module../utils/index.ts (/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js:20679:65) {
code: 'MODULE_NOT_FOUND',
requireStack: [
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/easyinvoice/index.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/packages/shop/.next/server/static/development/pages/_app.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/next-server/server/load-components.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/next-server/server/api-utils.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/next-server/server/next-server.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/server/next.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/server/lib/start-server.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/cli/next-dev.js',
'/home/angeligareta/Desktop/Atecresa/ecommerce/node_modules/next/dist/bin/next'
]
}

Thanks! Perhaps in some cases the packages are required for a successfull build even though they are not really needed because of some framework/build lookup functionality. I'll just add them to the basic dependencies for now to prevent your and maybe others build errors. Will dive into it in the near future.

Alright, changes are pushed. Could you let me know if your errors are fixed in version 1.0.100?

It works perfect now, thank you very much!

Hello,
I was revising the code and I see the invoices are being generated at https://www.easyinvoice.cloud/. I see that for calling that api you need an authorization header, but I have not included any api key. Which one is it being used? Is the 30 day trial that I can see at https://factuursimpel.nl/? On top of that I wanted to know if that information is being stored or is just used to generate the invoice. I would like to use this for customers and I could not trust a source storing that sensible information.
Thank you

Hi, the invoice generation part of this package (without storing capabalities) will be free forever. Also your data will not be stored while using the free version. When including an apiKey (which our NPM package not yet supports) your invoices will be stored and you will be provided a GUI. This will cost 10 dollars a month.
So in conclusion. The endpoint you are using right now will remain free and data will not be stored. Also we guarantee a 99,99% uptime of the api.

Hi, thanks for the response, but I would need a guarantee of how this data is being stored and used. I see you started the project a month ago and for this time the project is great, but I want to use this in production and I have the risk of using an external service only for generating invoices that could be attacked. Because of this, we have decided to use an offline library, I just wanted to transmit my feedback. Thanks for everything and good luck with the project.

On top of that, I recommend adding in the README.md that the data is being generated in the cloud, so it is an online library. Regards

We understand completely, thanks for the feedback and good luck with your project!


⚠ Bofrost.de has degraded performance

In 7305d20, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1468 ms

Resolved: Bofrost.de performance has improved in 2d0ba54 after 2 hours, 10 minutes.


Add Unit tests

Now that we have a working Telegram Test-DC Environment in our CI we can add more unit tests.
Unit tests are tests, that test single small units on their own. Please add a class in the tests package for every unit to be tested. Every class that implements some form of logic should have such a class.

Do unit tests really need to use test-dc? Can't we just test if calls to lib are properly made etc?

test-dc?

Test Data-Center from Telegram


Update README.md

Review Type Requested (choose one):

[ ] Glance - superficial check (from domain experts)
[ ] Logic - thorough check (from everybody doing review)

Summary
Provide a one line summary and link to any relevant references
Task/Issue reference
Closes: add_link_here
Details (optional)
Add any additional details that might help Code Reviewers digest this PR
How to test this code? (optional)
Anything else? (optional)

hey you @DeveloperAlly we should only merge in PRs after they've been approved!


[ISSUE #2607] Set benchmark`s consumer isSuffixEnable default false and update help info

Set the default value of isSuffixEnable which belong to benchmark`s consumer to false.
Update the help info abou isSuffixEnable

Coverage increased (+0.3%) to 51.95% when pulling 7006d3a8dbaf18ad6c71c941c6b8255dea904c13 on xxd763795151:bench_consumer into 6af12592af95916cc0b4512f2bab60ee18f62b8b on apache:develop.

Coverage increased (+0.3%) to 51.95% when pulling 7006d3a8dbaf18ad6c71c941c6b8255dea904c13 on xxd763795151:bench_consumer into 6af12592af95916cc0b4512f2bab60ee18f62b8b on apache:develop.

Codecov Report

Merging #2608 (7006d3a) into develop (6af1259) will increase coverage by 0.32%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #2608      +/-   ##
=============================================
+ Coverage      45.80%   46.12%   +0.32%
- Complexity      4306     4340      +34
=============================================
  Files            547      547
  Lines          36246    36246
  Branches        4808     4808
=============================================
+ Hits           16601    16718     +117
+ Misses         17552    17434     -118
- Partials        2093     2094       +1

Impacted Files
Coverage Δ
Complexity Δ

...mq/client/impl/consumer/RebalanceLitePullImpl.java
49.15% <0.00%> (-5.09%)
10.00% <0.00%> (-1.00%)

...ketmq/client/impl/consumer/PullMessageService.java
75.55% <0.00%> (-4.45%)
9.00% <0.00%> (-1.00%)

...he/rocketmq/client/impl/consumer/ProcessQueue.java
57.67% <0.00%> (-4.19%)
31.00% <0.00%> (-1.00%)

...client/consumer/store/RemoteBrokerOffsetStore.java
67.56% <0.00%> (-1.81%)
20.00% <0.00%> (-2.00%)

...nt/impl/consumer/ConsumeMessageOrderlyService.java
41.51% <0.00%> (-1.09%)
19.00% <0.00%> (+3.00%)
:arrow_down:

...he/rocketmq/client/trace/AsyncTraceDispatcher.java
73.60% <0.00%> (-0.51%)
17.00% <0.00%> (ø%)

...a/org/apache/rocketmq/store/StoreStatsService.java
29.50% <0.00%> (ø)
26.00% <0.00%> (ø%)

...etmq/client/latency/LatencyFaultToleranceImpl.java
50.00% <0.00%> (ø)
8.00% <0.00%> (ø%)

...main/java/org/apache/rocketmq/store/CommitLog.java
66.52% <0.00%> (+0.10%)
79.00% <0.00%> (ø%)

...e/rocketmq/namesrv/routeinfo/RouteInfoManager.java
39.12% <0.00%> (+0.21%)
28.00% <0.00%> (+1.00%)

... and 21 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #2608 (7006d3a) into develop (6af1259) will increase coverage by 0.32%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #2608      +/-   ##
=============================================
+ Coverage      45.80%   46.12%   +0.32%
- Complexity      4306     4340      +34
=============================================
  Files            547      547
  Lines          36246    36246
  Branches        4808     4808
=============================================
+ Hits           16601    16718     +117
+ Misses         17552    17434     -118
- Partials        2093     2094       +1

Impacted Files
Coverage Δ
Complexity Δ

...mq/client/impl/consumer/RebalanceLitePullImpl.java
49.15% <0.00%> (-5.09%)
10.00% <0.00%> (-1.00%)

...ketmq/client/impl/consumer/PullMessageService.java
75.55% <0.00%> (-4.45%)
9.00% <0.00%> (-1.00%)

...he/rocketmq/client/impl/consumer/ProcessQueue.java
57.67% <0.00%> (-4.19%)
31.00% <0.00%> (-1.00%)

...client/consumer/store/RemoteBrokerOffsetStore.java
67.56% <0.00%> (-1.81%)
20.00% <0.00%> (-2.00%)

...nt/impl/consumer/ConsumeMessageOrderlyService.java
41.51% <0.00%> (-1.09%)
19.00% <0.00%> (+3.00%)
:arrow_down:

...he/rocketmq/client/trace/AsyncTraceDispatcher.java
73.60% <0.00%> (-0.51%)
17.00% <0.00%> (ø%)

...a/org/apache/rocketmq/store/StoreStatsService.java
29.50% <0.00%> (ø)
26.00% <0.00%> (ø%)

...etmq/client/latency/LatencyFaultToleranceImpl.java
50.00% <0.00%> (ø)
8.00% <0.00%> (ø%)

...main/java/org/apache/rocketmq/store/CommitLog.java
66.52% <0.00%> (+0.10%)
79.00% <0.00%> (ø%)

...e/rocketmq/namesrv/routeinfo/RouteInfoManager.java
39.12% <0.00%> (+0.21%)
28.00% <0.00%> (+1.00%)

... and 21 more

Continue to review full report at Codecov.

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

不建议修改参数名。修改增加了熟悉成本。

不建议修改参数名。修改增加了熟悉成本。


Hover effect on social links icons with same background color

issue no #329
I have successfully  added background color matches hover effect on social links icons.

@mdazfar2 Done !

@zalabhavy

The effect should come in this type of square


Fix for jsdoc error when running npm test

I was getting a jsdoc error when trying to work through this tutorial https://hyperledger.github.io/composer/tutorials/developer-guide.html
node 6.11

 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.

Thanks @jordangraft!


Old packages

Nice boilerplate but using ignite new app makes an app with old libs.
e.g "react-navigation": "1.0.0-beta.11" that is a very useful package and it's current version is 1.5.1 with a lot of changes.
Now after updating react-navigation I have a lot of errors and because I'm not familiar with Redux, I'm not sure whats the problem.
P.S: If ignite init new project with match new packages, it's reliable to continue developing on this skeleton.

We have a new release coming soon.  We know it's far behind, but the plan is to release in the next month.
here's the upcoming version:  https://github.com/infinitered/ignite-ir-boilerplate/tree/ignite_bowser

Is that usable in its current form? Thanks

yup!  You can use it, it will be slightly updated though.  All updated will be trackable in git.

To use it, you use the -b flag to point to a different boilerplate during the ignite process.

So I take it you recommend starting a new project with the bowser branch? Thanks

Yes!  Definitely... though there are no docs for you just yet.

Awesome thanks. I would assume that it functions and is used more or less like the current version though, right? Thanks again

Correct, this is the latest and greatest (minus some updates we will be making)

What exactly must be enter into use this branch? ignite new AppName -b bowser?

Nevermind figured it out - in case it helps another newcomer:
clone repo and set branch to ignite_bowser
cd path_one_up_from_repo
ignite new AppName -b ./ignite-ir-boilerplate

@omairvaiyani I'm trying to use the ignite_bowser branch, but it throws error while doing
ignite add vector-icons

Is it possible to add vector-icons and maps this way?

Some of the existing plugins are targeting a different filesystem setup, so no.  Not yet at least.

@skellock thank you.
There's one more thing I noticed about the setup of the branch, is there any good reason to switch to mobx? (I know this should be asked on the target repository)

Ya, we've switched to mobx and mobx-state-tree a while ago. These libs simply do more for you.  redux does one thing but does it well.  These things tend to be moving targets. =)

@omairvaiyani Should it work on windows too? I've tested on mac and was OK, but using on windows creates a normal ignite boilerbpate : 'ignite new AppName /b /ignite-ir-boilerplate'

On Windows you should use the --boilerplate=C:/path/to/boilerplate option.

I got this error
chris@chris-Inspiron-13-5368:~$ ignite new todoapp -b ./ignite-ir-boilerplate
-----------------------------------------------
  (                  )   (
  )\ )   (        ( /(   )\ )    *   )
 (()/(   )\ )     )\()) (()/(  ` )  /(   (
  /(_)) (()/(    ((_)\   /(_))  ( )(_))  )\
 (_))    /(_))_   _((_) (_))   (_(_())  ((_)
 |_ _|  (_)) __| | \| | |_ _|  |_   _|  | __|
  | |     | (_ | | .` |  | |     | |    | _|
 |___|     \___| |_|\_| |___|    |_|    |___|
-----------------------------------------------

An unfair headstart for your React Native apps.
https://infinite.red/ignite

-----------------------------------------------

🔥 igniting app todoapp
✔ using the Infinite Red boilerplate v2 (code name 'Andross')
✔ added React Native 0.52.2 in 24.55s
? Would you like Ignite Development Screens? Yes
? What vector icon library will you use? react-native-vector-icons
? What internationalization library will you use? react-native-i18n
? What animation library will you use? react-native-animatable
? Would you like to include redux-persist? No
✖ /home/chris/todoapp/todoapp/ignite-ir-boilerplate was not able to be installed. Is it a valid NPM module?
----------
Command failed: yarn add /home/chris/todoapp/todoapp/ignite-ir-boilerplate --dev
warning enzyme > rst-selector-parser > nearley ><EMAIL_ADDRESS>Package no longer supported. Contact<EMAIL_ADDRESS>for more info.
error An unexpected error occurred: "Package \"/home/chris/todoapp/todoapp/ignite-ir-boilerplate\" refers to a non-existing file '\"/home/chris/todoapp/todoapp/ignite-ir-boilerplate\"'.".

any ideas?

@ccfiel can you try the full path to the branch?

@xgenvn correct! :D for newbi like me what I did

using this ignite new todoapp -b /home/chris/ignite-ir-boilerplate but I did not see any mobx still redux. Did I miss something?

Is your local branch pointed towards the bowser branch?
https://github.com/infinitered/ignite-ir-boilerplate/tree/ignite_bowser

@ccfiel
cd /home/chris/ignite-ir-boilerplate
git checkout ignite_bowser
ignite new todoapp -b /home/chris/ignite-ir-boilerplate

@GantMan its now working. Thanks @xgenvn made a little modification in the command
cd /home/chris/ignite-ir-boilerplate
git checkout ignite_bowser
cd ..
ignite new todoapp -b /home/chris/ignite-ir-boilerplate


DES: Added DESCryptographyLogic.java

Have you read the Contributing Guidelines on Pull Requests?
Yes
Checklist

[X] I've read the contribution guidelines.
[X] I've checked the issue list before deciding what to submit.
[X] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Issue Number: #1553

@HarshCasper please review the pull request once, if any changes/improvements/modifications needed please let me know about the respective.
Sequence Number: 02
Issue Number: #1553
Pull Request Number:  #2079

@HarshCasper please review the pull request once, if any changes/improvements/modifications needed please let me know about the respective.
Sequence Number: 02
Issue Number: #1553
Pull Request Number:  #2079

Please have a look at my review here and make the same changes: https://github.com/TesseractCoding/NeoAlgo/pull/2078


Include traces as well as exceptions for bunyan to avoid loss of information in messages

Please note that the only "real" change is to AutoCollection/diagnostic-channel/bunyan.sub.ts.  The other changes (update to gitignore and including output files) were just convenient for us to use this fork for our code until a final solution is implemented in this code base.
For issue: https://github.com/microsoft/ApplicationInsights-node.js/issues/1120

This PR wont work for the us as it would send all errors/exceptions as traces as well effectively creating duplicate telemetry for everyone using Bunyan.

I can understand that.....as a temporary measure for us, I think we may update the code to also check the severity level rather than create duplicate entries....this is what I am considering.
const subscriber = (event: IStandardEvent<bunyan.IBunyanData>) => {
    let message = event.data.result as string;
    clients.forEach((client) => {
        const AIlevel = bunyanToAILevelMap[event.data.level];
        try {
            // Try to parse message as Bunyan log is JSON
            //   Only log as exception for level of Critical
            let log: any = JSON.parse(message);
            if (log.err && AIlevel === SeverityLevel.Critical) {
                let bunyanError = new Error(log.err.message);
                bunyanError.name = log.err.name;
                bunyanError.stack = log.err.stack;
                client.trackException({ exception: bunyanError });
                return;
            }
        }
        catch (err) {
            // Ignore error
        }
        client.trackTrace({ message: message, severity: AIlevel });
    });
};

Config was added in 2.6.0 to address this, thank you.


Advice on using sessionsPerBrowser option

Output of gemini --version:
4.18.1
Contents of .gemini.yml file:
rootUrl: http://testing.etc.etc
gridUrl: http://<IP_ADDRESS>:4444/wd/hub

calibrate: false
sessionsPerBrowser: 5
compositeImage: true
retry: 1

browsers:
  # WINDOWS 7 CONFIG
    chromeWin7Small:
        screenshotsDir: './screens/win7/chrome'
        windowSize: 640x1024
        desiredCapabilities:
            browserName: chrome
            platform: Windows 7

    chromeWin7Medium:
        screenshotsDir: './screens/win7/chrome'
        windowSize: 641x1024
        desiredCapabilities:
            browserName: chrome
            platform: Windows 7

    chromeWin7Large:
        screenshotsDir: './screens/win7/chrome'
        windowSize: 1025x1024
        desiredCapabilities:
            browserName: chrome
            platform: VISTA

    chromeWin7XLarge:
        screenshotsDir: './screens/win7/chrome'
        windowSize: 1441x1024
        desiredCapabilities:
            browserName: chrome
            platform: Windows 7

Test source code:
Buttons.js
let cssVersion = process.env.CSSVERSION;

gemini.suite('Buttons', (suite) => {
    suite.setUrl('/kitchensink?cssVersion=' + cssVersion)
        .setCaptureElements('.snk_buttons')

        //All buttons together, with no actions performed
        .capture('NoAction', (action) => {
            action.wait(250)
        });

Breadcrumbs.js
let cssVersion = process.env.CSSVERSION;

gemini.suite('Breadcrumbs', (suite) => {
    suite.setUrl('/kitchensink?cssVersion=' + cssVersion)
        .setCaptureElements('.snk_breadcrumbs')

        //All breadcrumb levels together, with no actions performed
        .capture('NoAction', (action) => {
            action.wait(250)
        });

Command used to run the test:
CSSVERSION=1.0.1 ./node_modules/gemini/bin/gemini update *.js

Publically accessible URL of the page you are testing against:
http://
Result:
Only a single browser session is started at a time on the grid.
...
Expected behaviour:
I have started a Selenium Grid running with Selenium 3.0.1, using just the standard config on the node.  Both run on the same virtual machine, and I use the following commands to start the hub and node:
java -jar selenium-server-standalone-3.0.1.jar -role hub
java -jar selenium-server-standalone-3.0.1.jar -role node
This results in a node being connected to the hub with the default 5 instances of Chrome available.
As you can see above, in the .gemini.yml I have sessionsPerBrowser set to 5, and I was expecting to have 5 instances of Chrome opening in parallel and running the image gathering actions on the node.
Can anyone correct my understanding of how this should work?  My tests are taking a long time, particularly as the browser is closed and reopened each time as I need to specify a new setCaptureElements value for each test suite.
...

Restarted my node using a node config file that specifically configured it to have the standard 5 x Firefox, 1 x IE & 5 x Chrome and it is now working as expected, firing up the 5 Chrome browser instances in parallel.


Textarea in MatForMield can cause ExpressionChangedAfterItHasBeenCheckedError

Bug, feature request, or proposal:
BUG
"@angular/material": "5.2.5",
What is the expected behavior?
No exception
What is the current behavior?
Exception
This is the first time I have notice that and I cannot reproduce this on plain project. I don't know what causes it and how to reproduce for unit testing.
Having following markup:
    <mat-form-field class="summary no-ink-bar">
      <textarea matInput placeholder="Summary:" readonly>{{request.summary}}</textarea>
    </mat-form-field>

Causes following error
Angular is running in the development mode. Call enableProdMode() to enable the production mode. core.js:3688 ERROR Error: "ExpressionChangedAfterItHasBeenCheckedError: Expression has changed after it was checked. Previous value: 'mat-form-field-should-float: false'. Current value: 'mat-form-field-should-float: true'." node_modulesangular/core/esm5/core.js:10033:32node_modulesangular/core/esm5/core.js:10011:12node_modulesangular/core/esm5/core.js:10178:15node_modulesangular/core/esm5/core.js:14254:9node_modulesangular/core/esm5/core.js:14203:9node_modulesangular/core/esm5
I am fixing this by setting floatLabel="always" so
   <mat-form-field class="summary no-ink-bar" floatLabel="always">       <textarea matInput placeholder="Summary:" readonly>{{request.summary}}</textarea>     </mat-form-field>
removes that exception.
Maybe someone will have similar issue and will know how to reproduce

You can probably try to upgrade to latest version and see if that fixes the issue.

Can you provide a reproduction in StackBlitz.  This will help us to investigate the issue you are seeing.

https://stackblitz.com/edit/angular-material2-issue-ys19gm

Getting the same error here, too.

Here's a workaround until I can investigate, just use value="" instead of putting the text as content: https://stackblitz.com/edit/angular-material2-issue-y2tqhg?file=app/app.component.html

I had a similar issue and in my understanding the issue is that:

mat-form-field creates a form field with an empty value - the mat-label is displayed inside (thus mat-form-field-should-float: false)
Interpolation creates the value that should be placed as the textarea value.
The textarea receives the new value and decides that now the label should float (mat-form-field-should-float: true)

In case with value="...", (2) happens before (1), so there's no problem. That's also why it happens with textarea, but not with input.
(for the record: I came here debugging a similar error in test - in my case the textarea received the value from ngModel. I ended up working around it by adding floatLabel="always" on the form field.)

<textarea matInput readonly>{{selectedList.summary}}</textarea>
error
<textarea matInput readonly [(ngModel)]="selectedList.summary"></textarea>
works fine

But it's not possible with this case:
<textarea matInput readonly>{{ctrlForm.value | json}}</textarea>

Try this
<mat-form-field floatLabel="never"> <input placeholder="My Placeholder" matInput> </mat-form-field>

I got same error, and i set   as @Antoniossss it's ok now. Thanks

Hi, I've got both errors mat-form-field-should-float and mat-focused for ExpressionChangedAfterItHasBeenCheckedError when using <input autofocus> inside a mat-form-field
with Angular Material v6.
I can bypass the errors by using floatLabel=never and focusing the input in ngAfterContentInit() but it would be awesome to avoid using these approaches.

These things are never so simple! To add yet another scenario - and some findings of my detective work, here's my situation:

ExpressionChangedAfterItHasBeenCheckedError: Expression has changed after it was checked. Previous value: 'mat-focused: true'. Current value: 'mat-focused: false'

So this is the same as others have seen - but backwards (focused -> non focused). This is occurring for me on a simple login form - I have two <input> fields that get hidden after a successful (and obviously asynchronous) login. eg. *ngIf="notLoggedIn$ | async" ...
Not surprisingly if I submit the form by clicking my 'Login' button then it works - because focus is instantly lost during that event cycle. It is when I hit 'Enter' to submit the form that the problem arises because focus it taken away from the input control asynchronously.
Now you may be thinking - well that doesn't make sense - the control is already destroyed - and you'd be right. The way I described above seems to work fine - however I'm using CdkPortal and hiding something that is within a portal. That seems to be triggering this at least for me.
So I think I need to discontinue my clever portal stuff at least right now to get this to work - however if you're seeing similar symptoms and using mat-tab then it could be the same cause because my code is very heavily influenced by the way mat-tab works.

floatLabel="never"

thanks issue solved.

@simeyla have you ever found a solution? having the exact same problem

@fxck I’ve concluded that sometimes you just have to add delay(0) to the observable pipe to get it to update later than it wants to.
I’ve gone round and round in circles at various times on this - reread articles and SO posts but sometimes it just seems necessary.
I did create a pipeable operator that adds delay(0) so then I can put isLoggedIn$ | delay | async
I don’t have it to hand now but it’s trivial to write a pipeable operator as long as you remember to export and provide it properly.

Just add placeholder in mat input field problem will get resolve.

My workaround:
Step 1
Add to the relevant module:
import { MAT_FORM_FIELD_DEFAULT_OPTIONS } from '@angular/material/form-field';

  providers: [
  // ...,
   {
      provide: MAT_FORM_FIELD_DEFAULT_OPTIONS,
      useValue: { floatLabel: 'always' },
    },
  // ...
  ],

Step 2
Add to the relevant component:
export class MyComponentWithConsoleError implements OnInit, AfterContentChecked {
  ngOnInit(): void {
    // ...
    this.cd.detach();
  }

   ngAfterContentChecked(): void {
    this.cd.detectChanges();
  }

}

Is this still a problem? I went back to rebase my PR with the fix and I can't reproduce it anymore.

Closing due to inactivity. The issue can be reopened if there's new information.


typo add jdbc catalog error handling methods

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Fix meta file artifact extension

Opam, during/after make, complains that merlin_extend.cma can't be found. Which can be traced back here.

Wow... This was actually on my computer I forgot to push.
My bad, thanks for the pull request.


Add a guard to eventCallbacks

📺 What
Spotted in automated testing telemetry. Calling tearDown will set eventCallbacks to undefined causing a type error when a media element emits an event.
Spotted during

Tickets: IPLAYERTVV1-9375

🛠 How
Guard iterating through the eventCallbacks if the array is undefined.

I believe cehtml and msestrategy have the same problem.


it's amazing effective

I like it
I want to learn skills of animation from you!!!
so cool!!!!

I love this animation.
I'm impressed and I'm making some new animation.
Thanks for beautiful animation.


Running over TLS?

A huge "thank you" for making this software available, as always!
I've set up a nodeStorage server on an Ubuntu VPS and now I've gotten myWordEditor running as well. It's running beautifully...
The question I have has to do with running myWordEditor over a TLS connection via https.
It appears that some components are hardcoded to the http protocol, as when I connect to myWordEditor via https (as is my habit with all tools running on this server), this is the result:
Screenshot via imgur
I'm guessing that running over https wasn't originally envisioned. If you're able to point me in the right direction, I'm happy to do some work to make the needed modifications. Thanks!

I tested this on my local machine with an ngrok URL and got the same result. The browser is blocking all "insecure content", all the stylesheets and scripts in index.html called with http://
 448f1071.ngrok.io/:1 Mixed Content: The page at 'https://448f1071.ngrok.io/' was loaded over HTTPS, but requested an insecure script 'http://api.nodestorage.io/ui/jquery-1.9.1.min.js'. This request has been blocked; the content must be served over HTTPS.
448f1071.ngrok.io/:8 Mixed Content: The page at 'https://448f1071.ngrok.io/' was loaded over HTTPS, but requested an insecure stylesheet 'http://api.nodestorage.io/ui/bootstrap.css'. This request has been blocked; the content must be served over HTTPS.
...

Thanks @scripting for pinging us via https://github.com/scripting/pagePark/issues/6#issuecomment-330407780.
Coincidentally I've been looking into the mixed content issue as well. Here's my original 1999 server w/ no SSL compared to my new server with SSL. The websockets don't work on the new server at the moment, but that's an unrelated issue.
Few ideas I was considering for working around this...

upgrade the remote server to HTTPS and switch the protocol depending on URL
setup a CDN that can upgrade the remote server w/o adding HTTPS directly
reverse proxy the assets from the server?
load the assets into the repo/origin site

Alternatively you could force the server to stay in HTTP mode, e.g. for ngrok only in HTTP.

One thing that could be done is a PR that changed any links that hardcoded the protocol to //.
IE:
<script src="http://fargo.io/code/concord.js"></script>
Becomes
<script src="//fargo.io/code/concord.js"></script>

This will force the browser to supply the current protocol. If you are browsing an HTTPS site, it will force all those links to go to HTTPS, likewise if you are using HTTP, it will force everything to HTTP. That way you could support simultaneously running HTTP/HTTPS versions.

I am unable to reproduce because I don't have an HTTPS installation here,
but I have asked some other people to take a look at this.

Thanks @scripting for having others check in on this. I think I recall from your blog that you aren't necessarily a big fan of the "https everywhere" concept, so thank you for being so accommodating. Also, thanks to everyone who has jumped in!
A couple of quick thoughts:
@AngeloR mentioned removing the hardcoded protocols. This was where my mind was going, but it looks like some of the assets may not yet be available over https.
@davisshaver mentioned a few possibilities:

upgrade the remote server to HTTPS and switch the protocol depending on URL
I'm in the habit of using LetsEncrypt and 301 redirects for all non https access attempts so that they're always served over https. Thus, I wouldn't prefer to do any protocol switching.

setup a CDN that can upgrade the remote server w/o adding HTTPS directly
I'm not a fan of using CDNs (like Cloudflare or even Cloudfront) to upgrade to HTTPS because of the identity problems and possible MITM attacks associated with the 3rd-party components

reverse proxy the assets from the server?
This works for me as long as the assets are local. In the cases where assets are being pulled from other sources and are still being served (to my server) over HTTP, they're going to be subject to MITM in transit.

This is actually the method I'm using to serve nodeStorage over https, as described in the guide that @andrewshell wrote (although my configuration uses nginx instead of apache).

load the assets into the repo/origin site
This makes sense to me for some of the assets. I haven't gone through everything with a fine-toothed comb just yet, but if there are any js libraries that should be updated from time to time, it may make sense to continue to load them directly from origin servers if they're available over https.

BTW, I'm more than happy to test any of the above on my setup, which runs Ubuntu and nginx.

That said I try not to have "strategy taxes" on my software, like Google is doing in search. If people want to use it with HTTPS and it's possible I will help, as much as I can.
But this is already turning into a morass.

I created this with the new version. But my server is not running behind HTTPS.
http://myword.io/users/davewiner/essays/055.html

Thanks for the update @scripting & I understand your perspective, I actually hadn't considered this as a historical issue until just reading your last comment. There are plenty of valuable servers that should not be expected to have the resources to upgrade to HTTPS. Love the term strategy debt and that's how I feel about needing to implement it. I'm going to keep hacking on this and will share what I find. Per @thedavidjohnson above, how would you feel about us extending the update framework to pull in additional assets? I'm glad to help put together a proof of concept if that sounds okay.

Google probably hasn't thought about it either. But it's not the number of servers it's the number of domains that matters.

The new version almost works with HTTPS, except that the assets at fargo.io aren't available over HTTPS (and, if I understand Dave correctly, never will be 😄 ). I'm willing to help out to get it working, though.

how would you feel about us extending the update framework to pull in
additional assets?

Let's get in the habit of formalizing this. I like writing and reading
RFCs. For specific problems, not whether or not to generalize something.
First I want to know what the problem is, and get a chance to think about
it, before figuring out how to do it. You end up living with the solutions
for a long time.
Dave
On Tue, Sep 19, 2017 at 10:42 AM, davisshaver<EMAIL_ADDRESS>wrote:

Thanks for the update @scripting https://github.com/scripting & I
understand your perspective, I actually hadn't considered this as a
historical issue until just reading your last comment. There are plenty of
valuable servers that should not be expected to have the resources to
upgrade to HTTPS. Love the term strategy debt and that's how I feel about
needing to implement it. I'm going to keep hacking on this and will share
what I find. Per @TheDavidJohnson https://github.com/thedavidjohnson
above, how would you feel about us extending the update framework to pull
in additional assets? I'm glad to help put together a proof of concept if
that sounds okay.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scripting/myWordEditor/issues/4#issuecomment-330562201,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABm9OwychXsI-N2X0c7JhmKR-qA6hjnBks5sj9LkgaJpZM4Pbu1b
.

I don't mind moving them @papascott.
Let's create a place where assets stored on S3 are served over HTTPS.
Then all my static files can be available that way.
Remember -- NO STRATEGY TAXES.
If it's easy it's easy.

A Scripting publicFolder 😄 ! The S3 bucket sounds like a great idea.The S3 bucket sounds like a great idea. And, noted about the RFC's and patching. If I can get one together I will share in pagePark. Right now I'm leaning on AWS free cert manager to setup SSL on domains. Actually they can give you up to 100 domains SSL for free on a single cert so I was pleasantly surprised by that.

I can test the latest update over HTTPS later today. If we end up doing an RFC, let me know... I'll happily contribute!
I feel like I need to write this date down... @scripting is proposing an S3 bucket with HTTPS?! What?!

When I say RFC -- I mean it in the old informal sense. Just a document that
asks for comments. On a blog, not on any significant website.
On Tue, Sep 19, 2017 at 11:02 AM, David G. Johnson<EMAIL_ADDRESS>
wrote:

I can test the latest update over HTTPS later today. If we end up doing an
RFC, let me know... I'll happily contribute!
I feel like I need to write this date down... @scripting
https://github.com/scripting is proposing an S3 bucket with HTTPS?!
What?!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scripting/myWordEditor/issues/4#issuecomment-330568881,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABm9O5aBf8BVj2re4uXLcI2d3K_TkZxUks5sj9eWgaJpZM4Pbu1b
.


HTML Classes and Ids

Description 📜
I want to add documentation of HTML classes and ids.
Assign this issue to me.

Domain of Contribution 📊

[x] Frontend Dev (HTML, CSS, JS)

/assign

/assign

/assign

/assign


Bar growing unexpected, negative values or max-value + n possible

when playing with the RangeBar on the edges (min or max) it suddenly grows and negative values are possible. At some point it is crashing !

alibarrak you could put the complete code for a possible solution
if (leftThumbXDistance < rightThumbXDistance) {
if(x < mBar.getLeftX())
x = mBar.getLeftX();
mLeftThumb.setX(x);
releaseThumb(mLeftThumb);
} else {
if(x > mBar.getRightX())
x = mBar.getRightX();
mRightThumb.setX(x);
releaseThumb(mRightThumb);
}
leftThumbXDistance ? rightThumbXDistance ? x? mRightThumb? mLeftThumb ? mRightThumb ?

@sphairo on RangeBar.java replace lines 744-750 with my code which adds two lines to each condition.


CH-4 make test works - but submit fails

(base) ➜  CHALLENGE-4 git:(main) ✗ make test
hurl --test test.hurl
test.hurl: Running [1/1]
test.hurl: Success (3 request(s) in 12 ms)
--------------------------------------------------------------------------------
Executed files:  1
Succeeded files: 1 (100.0%)
Failed files:    0 (0.0%)
Duration:        13 ms

alas, submit.hurl fails with:
error: Assert status code
  --> submit.hurl:6:8
   |
 6 | HTTP/* 200
   |        ^^^ actual value is <500>
   |

@dennisg Can you add -v to your hurl and share the output, please?

I don't have the laptop that I run this on available right now, but the error message was that it received a 500 instead of a 200

> POST /c4 HTTP/2
> Host: advent-of-spin-backend-tlrr40nq.fermyon.app
> accept: */*
> content-type: application/json
> user-agent: hurl/1.8.0
> content-length: 62
>
* Response: (received 53 bytes in 674 ms)
*
< HTTP/2 500
< date: Sun, 18 Dec 2022 08:56:58 GMT
< content-type: text/plain; charset=utf-8
< content-length: 53
<
*
error: Assert status code
  --> submit.hurl:6:8
   |
 6 | HTTP/* 200
   |        ^^^ actual value is <500>
   |

@dennisg I need more information. Can you DM me on Discord, or email me, please?
david at rawkode dot academy

thx


net: tests using timer fail on macOS High Sierra 10.13 / iOS 11 [Go 1.8]

Go 1.8 backport of #22062.

CL 66491 OK for Go 1.8.5.
CL 67350 OK for Go 1.8.5.

go1.8.5 has been packaged and includes:

CL 66491
CL 67350

The release is posted at golang.org/dl.
— golang.org/x/build/cmd/releasebot, Oct 26 21:07:53 UTC


[BUG] Cached metrics makes us crazy

Hi, we have a system with some cursors where we are calculating metrics on how much delays cursors have from last events emitted. We have built a custom plugin for this but the system sometimes it remains stuck with this values in cache and is not going ahead since we restart the kubernetes pods.
What we can do for that?

Hello Luca!
Unfortunately I don't have enough information to help you with this one. Could you produce a repo to minimally reproduce the issue that you are seeing? Do you think it is a problem with the plug that you created as opposed to PromEx the library? A shot in the dark guess would be that you want to use a gauge (via last_value https://hexdocs.pm/telemetry_metrics/Telemetry.Metrics.html#last_value/2) in your plug in if you just want the last value of some measurement.
Happy to help, but need some more information :).

We have an eventstore with events and cursors:
so a table like:
Events: event_name, offset
Event1, 1
Event1, 2
Event2, 3
Every cursor has a set of event types that could read to process them.
Cursors: name, event_types, last_offset_read
Cursor1 [Event1], 0
Cursor2 [Event2], 0
Cursor3 [Event1, Event2],0
We are checking with our plugin how many events are missing to reach the end. So in this case for example the metrics would be:
promex_cursor_1 2 (Event1 types)
promex_cursor_2 1 (Event2 types)
promex_cursor_3 3 (Event3 types)
We are querying the DB with a query to check how many events are still available after the last_offset_read. Sometimes the metric is coming back with the same data for a while and then changing. The problem is that we have alert on those metrics and sometimes we have false positive alerts.
Do you have any idea about it?
The problem is that locally is not happening, meanwhile in staging with all the cursors runnin (~50 cursors) sometimes the metrics are stuck

Sometimes also there are few metrics, missing the cursors one.
I am thinking: we have 3 pods on k8s, is possible that the PromEx process is just starting on one of that 3 and the call is succeed just on 1 of the 3?

@ViseLuca could it be related to this issue I just opened https://github.com/akoutmos/prom_ex/issues/236?
We are observing the same thing and I have pinpointed it to errors thrown from within the mfa callback that collects the metric in the plugin.

Closing this ticket for now as a release will be cut soon with the ability to not detach the polling job when an error is encountered (example in #236).


findnonltrs command will fail with system configuration

The environment is still hard coded in one of the classes (RunHMM)..fix it!

Fixed in d68d373 and d2148d3.


onTextChanged implementation

Previously EditText::onTextChanged (in Kotlin) accepted a lambda that was called with the updated text. So I could do something like:
-onTextChanged { txt -> setVal(txt); Anvil.render() }

Now, onTextChanged requires a full TextWatcher implementation. Of greater concern, it appears to be adding the watcher to the EditText over and over again, with every render. So after a few updates to the field the UI begins lagging noticeably.
The old onTextChanged syntax was sufficient for my needs and more concise. I'm planning on creating my own onTextChanged using the old syntax. Is this something that you would be interested in getting a PR for or not?

Ah, that's the cool part of the new Anvil actually.
Of course TextWatcher is the uglies listener in the whole Android SDK: there is no way to tell if it's already added to the view or not, you can not call setText() from inside it etc etc.
It required me to make a lot of kludges to make it work in the old Anvil.
In the new Anvil I still provide a TextWatcher setter, which removes the previous TextWatcher and adds the new one if you use lambdas. Of course, having one TextWatcher instance stored somewhere in class attributes is a solution to avoid extra work.
But there is a cool two-way data binding: text(StringBuilder). It allows you to get user input into the StringBuilder, or to modify the StringBuilder and thus update the text. No infinite recursion happens in this case.

Is there a way to subscribe to changes using the text(StringBuilder) binding?

Oh.. I think the easiest way for now would be to override StringBuilder's replace method. Or cache TextWatcher instance and call Anvil.render() from there only when your text has actually been changed.

It would be nice to have the TextWatcher DSL as it was in the old Anvil, but:

One should be careful about Anvil.render() inside TextWatcher callbacks - it would lead to long recursion if you call it when your input has not been changed
One should keep the previously known user input somewhere
One should be careful about adding new TextWatcher instances - it seems like TextWatcher's callbacks are fired when it's added to the view (even if there is no user input happening at that time).

I ended up slightly tweaking my UX to remove the need to watch for changes. I'm using the text(StringBuilder) binding now and it seems to be working fine for me.

@zserge Having problems getting this to work with RenderableAdapter. A few observations:

Watchers set via onTextChanged() don't seem to be removed when calling onTextChanged() again in a subsequent render cycle. This creates issues if the watcher references row-specific data (e.g. setting text for a specific row in the dataset) as obsolete watchers end up being called on mutation.
The only way I've been able to work around this is to keep record of watchers assigned to views and manually remove them during the render, e.g.
Map<EditText, TextWatcher> watchers = new HashMap<>();

editText() {
    EditText view = Anvil.currentView();

    TextWatcher previousWatcher = watchers.get(view);
    if (previousWatcher != null) {
        view.removeTextChangedListener(previousWatcher);
    }

    TextWatcher newWatcher = new TextWatcher(/* … */);
    watchers.put(view, newWatcher);
    view.addTextChangedListener(newWatcher);
}
-

Not sure if I've understood correctly, but isn't text(Stringbuilder) limited in that it can only render 'raw' model data? By that, I mean it's not possible to compute values in the render function, e.g. text(Integer.toString(value)) without implementing a separate watcher to update the model.

It's not possible to override StringBuilder.replace()(at least in regular java) as the class is final.

@danhawkes Yeah... TextWatcher is so far the most painful part of Anvil. It relates to a broader topic of two-way data bindings and is on my todo list for a long time already. Still, I don't see a perfect solution yet because Android's TextWatcher is broken too much.

I think BaseDSL.setTextChanged() should replace the textwatcher if the new instance is different from the old one. If it doesn't replace them - it's a bug. If you have a look at BaseDSL.setTextChanged() you may find it behaves very similar to your code.

text(StringBuilder) is a hack I had to add to achieve simple two-way data binding. It a mutable string (which is a StringBuilder in Java) as a parameter. If you pass other (immutable) strings, like the ones received from toString() - you will have to add your own text watcher to get two-way data binding.

Yes, unfortunately it is so, and I admit that StringBuilder is a poor choice for mutable strings and probably our own mutable string class would serve better.

I'm open to a discussion about how two-way data bindings should work in Anvil and how to simplify this process. I would prefer a solution to be more generic (e.g. cover not only EditTexts, but also Sliders, ToggleButtons and other views used for input).

So far I could only think of two approaches:

Use mutable data classes, so one could use text(mutableString) or progress(mutableInt) or checked(mutableBoolean)`. Each mutable class when changed may emit Anvil.render() and cause UI updates, also at any time one can get the value and receive the current input. To get notified when user does input one would have to override the class. This also forces user to use our own data classes in their view-models, which means view-model code becomes dependent on Anvil.

Use simplified listeners (as lambdas) with fromUser parameter. Then one could bind one (immutable) data to the view and register a simple lambda (fortunately lambdas a cheap and new Jack/Jill will bring them by default very soon). Inside a lambda one would write something like if (fromUser) { value = newValue; Anvil.render(); }.

Solution #1 requires less code from user and gives less control if you don't want to subclass your data classes.
Solution #2 makes user to write more code but is more explicit and gives more control.


Label HEAD and branches in commit graphs

It would be really helpful to have refs (branches, commits, tags, HEAD, etc) in the actual graph. The challenge is to keep this from getting too noisy, but I think some way of doing this is pretty important. I just did a fast forward merge, and the graph didn't change. Seeing some update would be really helpful.
Specifics: the refs could be:

in the nodes themselves: the nodes would have to get bigger
near the nodes: they'd have to point. This would get messy, but could work
in a row at the top or bottom of the graph. It's already the case that only a single commit appears in any given column, so this could perhaps be easiest?

Eric did it!


Cloudflare hangs on "Verifying..." on the "Are you human" page

I'm trying to access a site thats cloudflare protected with the "Are you human?" page. I used cf-clearance-scraper to grab the cf cookies from the protected site, and am injecting all the returned header data (with cookies) into a Puppeteer request. Before the header injection, it would ask me to click the "Are you human?",  but after the header injection, it gets stuck on a "Verifying..." cloudflare page.  Any ideas?  See the attached screenshot for reference.

The library has been updated. It should now run smoothly. I am closing this issue, please let me know if you have any problems by writing a message through this issue. Thank you.


XML error when you don't submit file to File Upload question

I was applying for a program and saw a File Upload question. I skipped the question (by hitting Next without uploading anything) and got this XML error. Ideally we'd get a more user-friendly error message in the UI.

Relaying discussion around this from standup today:
We talked about how the submit (AKA "Next") button on a file upload block directly uploads to AWS. We think a good plan is to disable the "Next" button until/unless a file is uploaded. If a file is never uploaded, we should provide a "skip" button that allows a user to not upload a file.

The fix works (adding a Skip button) but we still need to hide or gray out the Upload button until the applicant uploads a file. Until then, the Upload button will still cause an XML error when clicked before a file is uploaded.

OK, the Upload button now cannot be submitted until a file is uploaded!


Test failures don't seem to include test errors

Also some of the numbers are smaller now it seems. These are the numbers I am getting: https://github.com/epoberezkin/json-schema-consolidate#validators-compatibility

Is this still an issue? I'm not entirely sure what you mean precisely. Closing for now


Fixed GKO_ENABLE_BUILD_METHOD macro

GKO_ENABLE_BUILD_METHOD takes an argument, which is supposed to be the name of the factory class. However, this parameter was never used.
Instead, the name Factory was hard-coded, which worked because we always use this name. However, it failed to compile when using a different name.
Now, the argument is used properly, so names other than Factory will also work.

Yes, I looked at all the macros from LinOp to understand them better (and to find out how to include an incomplete LU factorization as a preconditioner) and this was the only thing that stood out to me.


[Docker] node.js cannot access API

Hey,
after few attempts to get it up and running, I have finally decided to ask for your help concerning OpenEats production setup with Docker.
I followed Running the App guide. I am using docker-compose with following docker-prod.yml file:
version: '2'
services:
  api:
    image: openeats/api
    command: /startup/prod-entrypoint.sh
    restart: unless-stopped
    volumes:
      - static-files:/code/static-files
      - site-media:/code/site-media
    depends_on:
      - db
    env_file:
      env_prod.list
  node:
    image: openeats/node
    command: npm start
    restart: unless-stopped
    depends_on:
      - api
    env_file:
      env_prod.list

  db:
    image: mariadb
    env_file:
      env_prod.list
  nginx:
    image: openeats/nginx
    command: ./start.sh
    restart: unless-stopped
    ports:
      - "8089:80"
    volumes:
      - static-files:/var/www/html/openeats-static/static-files
      - site-media:/var/www/html/openeats-static/site-media
    depends_on:
      - api
      - node
    env_file:
      env_prod.list
volumes:
  database:
  static-files:
  site-media:

and following env_prod.list:
# Database config
MYSQL_DATABASE=openeats
MYSQL_ROOT_PASSWORD=ROOT_PASS

# Django config
API_URL=<IP_ADDRESS>:8000
API_PORT=8000
DJANGO_SECRET_KEY=SECRET_KEY
DJANGO_SETTINGS_MODULE=base.settings
DJANGO_DEBUG=False
ALLOWED_HOST=localhost
HTTP_X_FORWARDED_PROTO=true

# Node config
NODE_ENV=production
NODE_URL=localhost:8080
NODE_PORT=8080
NODE_API_URL=http://localhost
NODE_LOCALE=en

I fire up docker-compose -f docker-prod.yml up -d and the containers are properly created. The app is available under http://localhost:8089, but with no data. For that reason, I type docker exec -ti xyz_api_1 bash and use ./manage.py migrate and ./manage.py createsuperuser to have at least one account available. Unfortunately, when trying to log in, there is no response from the site - not even an error (so it does not resemble #279 in that matter, where there was an error after trying to log in). I tried to populate database with ./manage.py loaddata ....json using all available fixtures, but still, nothing happened - also, the data loaded was not shown by the frontend (but it might be that way if the data is available only after logging in...).
I suspect, that for some reason there is no connection from Node.js to the API - I tried to use expose: "8000" for api container, but it did not change anything.
I would be grateful for any help regarding that issue, maybe it's just my mistake in setup or changing of docker-prod.yml file.
Best regards!
Filip

Hey there, couple things to try:

First are there any error message/warnings when you are accessing the API? You can open up the inspect tool in chrome (or any browser) and see the error messages that are happening.
Access the admin panel: The admin panel is running a the Django admin panel and is powered by the API. Try this link and see what error/message you get. http://localhost:8089/admin/login/?next=/admin/
Try restating the containers and have the logs print to std out. You can do this by running docker-compose -f docker-prod.yml up

Thanks!

I am indeed able to go to the Django admin panel, and the log when restarting containers does not indicate any errors.
The nginx HTTP request logs for both api and node are shown properly in the console.  It is requesting http://localhost/api, where it should request http://localhost:PORT/api. Am I right? I will experiment with that, I am not sure which one should be specified there (I guess 8000, but let me see)

Ok, solved! It was not 8000, but 8089, so the working env_prod.list file looks like this:
# Database config
MYSQL_DATABASE=openeats
MYSQL_ROOT_PASSWORD=ROOT_PASS

# Django config
API_URL=<IP_ADDRESS>:8000
API_PORT=8000
DJANGO_SECRET_KEY=SECRET_KEY
DJANGO_SETTINGS_MODULE=base.settings
DJANGO_DEBUG=False
ALLOWED_HOST=localhost
HTTP_X_FORWARDED_PROTO=true

# Node config
NODE_ENV=production
NODE_URL=localhost:8080
NODE_PORT=8080
NODE_API_URL=http://localhost:8089
NODE_LOCALE=en

Thank you for your help!
BTW. there is a race condition during the docker container setup - migrations are invoked before the database itself is initialized (although the container itself is already there) - so Django can't connect to the database, interrupting the procedure and never returning to it again. I would propose the solution used in the MonicaHQ, which is a very simple script to wait for the database to be up: https://github.com/monicahq/monica/blob/master/docker/waitfordb.sh
Cheers!

Thanks for going though this! I created another ticket for the race condition (https://github.com/RyanNoelk/OpenEats/issues/304) I'll take a look at it the next chance I get.
I will also make a note in the docs about the NODE_API_URL env var and how it needs to be the extra location of the API server.
Thanks!


Unable to debug Java binary in Bazel 6 Bzlmod project

Description of the bug:
I am encountering an issue when attempting to debug a Bazel java_binary target in a Bazel 6 project with bzlmod enabled.  When attempting to invoke the target and attach the IntelliJ debugger via the UI, I received the following error:
ERROR: Unable to find package for @[unknown repo 'intellij_aspect' requested from @]//:java_classpath.bzl: The repository '@[unknown repo 'intellij_aspect' requested from @]' could not be resolved: No repository visible as '@intellij_aspect' from main repository.

The command that specifies the intellij_aspect uses a single @ in the repo identifier.
Command: /usr/local/bin/bazelisk build --tool_tag=ijwb:IDEA:ultimate --curses=no --color=yes --progress_in_terminal_title=no --aspects=@intellij_aspect//:java_classpath.bzl%java_classpath_aspect --override_repository=intellij_aspect=/home/myuser/.local/share/JetBrains/Toolbox/apps/IDEA-U/ch-0/231.8770.65.plugins/ijwb/aspect --output_groups=runtime_classpath --build_event_binary_file=/tmp/intellij-bep-b4854b50-ce5a-42eb-b409-98aea4a40960 --nobuild_event_binary_file_path_conversion -- //src/hello_java:Main

A similar issue involving aspects required that the repo identifier be modified to include the @@ prefix and override the strict dependency checks.
https://github.com/bazelbuild/intellij/issues/4067
What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
- Open a Bzlmod project (containing an executable Java class) in Intellij with the bazel plugin installed
- Put 6.1.2 in your .bazelversion
- Open a class with main method in the IDE
- Right click the run arrow (in the left gutter) and select Debug

Which Intellij IDE are you using? Please provide the specific version.
Intellij IDEA 2023.1.1 Ultimate edition |  Build #IU-231.8770.65, built on April 27, 2023  | Runtime version: 17.0.6+10-b829.9 amd64 | VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
What programming languages and tools are you using? Please provide specific versions.
Java (Amazon Corretto 11.0.19). Although I don't think this issue is specific to a particular version
What Bazel plugin version are you using?
2<IP_ADDRESS>.1-api-version-231
Have you found anything relevant by searching the web?
This is similar to the issue found here:
https://github.com/bazelbuild/intellij/issues/4067
and addressed here:
https://github.com/bazelbuild/intellij/commit/48820e5fa4cc57f1ea607b845e5cba1854adf361
Any other information, logs, or outputs that you want to share?
No response

I was able to resolve this issue with a local build including the following commit:
https://github.com/jstuder-gh/bazel_intellij_plugin/commit/66d55d712eb796f3c381036566cebc5e46721d4d
But this solution could be incomplete. Since I'm unfamiliar with the project I don't know how to comprehensively validate via integration tests.

Hi @jstuder-gh, Thanks for opening this. Could you please share exact steps you are following with sample code to reproduce the issue? Thanks!

Hi @Pavank1992, this repo provides a tiny Java project that reproduces the issue
https://github.com/jstuder-gh/bzlmod_debug_java_example

also having this issue. @jstuder-gh's patch in https://github.com/jstuder-gh/bazel_intellij_plugin/commit/66d55d712eb796f3c381036566cebc5e46721d4d seems to fix this for me

Fixed with https://github.com/bazelbuild/intellij/commit/75f63f5b3f6b20fed9ef9c9fb16a41cf885600ec


ViewMaker::makePartial() regression

Winter CMS Build
Other (please specify below)
PHP Version
8.0
Database engine
Other (please specify below)
Plugins installed
No response
Issue description
The commit that changed the default extension for backend partials from .htm to .php broke BC.
ref. 643a581ce93cc5f8efb3fc8a2420c8dc014c8787
Steps to replicate
When calling the ViewMaker trait makePartial() method, the following worked in previous Winter versions:
$obj->makePartial('modules/backend/widgets/form/partials/field_dropdown', ...);

Workaround
No response

Fixed in https://github.com/wintercms/winter/commit/7a25524f737b8c4a612f917900e0a276d43bd447


out of bounds error

Hi,
I've tried to compress different datasets with different levels and often got out of bounds errors, e.g.
runtime error: index out of range [104857594] with length 104857592 (code 13)
or
runtime error: index out of range [3] with length 3 (code 13)
I've tested with releases 2.0 and 2.1
Is this a bug or are there any configurations to avoid this errors?
Thanks
Bjoern

Thanks for the info.
Using the head source in master, I saw the first issue (using -l 8 or -l 9) and pushed a fix (commit 0bba60).
I did not see the other issue with -l 4. It works for me with this file.  Does this one happen with release 2.1 or 2.0 ?

The other issue is in FSDCodec.go and has already been fixed in master (d0914e1) which is why I did not see it.
Minimal patch:
L121-145
// Check several step values on a sub-block (no memory allocation)
// Sample 2 sub-blocks
in1 := src[1count5-8 : 1count5+count10-8]
in2 := src[3count5-8 : 3count5+count10-8]
    for i := 8; i < count10; i++ {
            b1 := in1[i]
            histo[0][b1]++
            histo[1][b1^in1[i-1]]++
            histo[2][b1^in1[i-2]]++
            histo[3][b1^in1[i-3]]++
            histo[4][b1^in1[i-4]]++
            histo[5][b1^in1[i-8]]++
            b2 := in2[i]
            histo[0][b2]++
            histo[1][b2^in2[i-1]]++
            histo[2][b2^in2[i-2]]++
            histo[3][b2^in2[i-3]]++
            histo[4][b2^in2[i-4]]++
            histo[5][b2^in2[i-8]]++
    }


Todo Suggestions

Hey there, since you're looking at this you might have a suggestion for my ToDo cog.
As I would rather have somewhere to talk about it besides discord and having an issue for every suggestion for todo on my github will get annoying, I have made this issue.
Be civil, leave good suggestions, don't attack others, and thanks for reading this!

Don't use code block in [p]todo list, for me it doesn't look as good as it could if it was sent without a code block.

Don't use code block in [p]todo list, for me it doesn't look as good as it could if it was sent without a code block.

You can use [p]todoset md false to disable this

Todo pin command
Multiple todo lists e.g. one for bot things to do and one for cogs

Todo pin command
Multiple todo lists e.g. one for bot things to do and one for cogs

As of right now, I'm actually working on a rewrite of todo which will implement this feature

The ability to create teams who can view the same todos. Basically, this would be a collaboration tool. For example, I can give Jojo and DSC the ability to see a certain list of todos.

The ability to create teams who can view the same todos. Basically, this would be a collaboration tool. For example, I can give Jojo and DSC the ability to see a certain list of todos.

I have some ideas on how I would do this. I won't be able to get it up for a bit, however, due to me being very busy irl. Thanks for your suggestion!

The ability to create teams who can view the same todos. Basically, this would be a collaboration tool. For example, I can give Jojo and DSC the ability to see a certain list of todos.

Hey there! I recently made a commit that added a bit of this functionality for this. I will be busy this week but I hope to add more to it.
Thanks for the suggestion!

Jojo, thank you for the listening and being open to my suggestion re: collaboration capabilities for your todo cog. I honestly think you have elevated this to another level. You should apply for it to be an approved cog. I look forward to what other cogs you create and will be happy to give you constructive feedback if you're open to them. Cheers!

Jojo, thank you for listening and being open to my suggestion re: collaboration capabilities for your todo cog. I honestly think you have elevated this to another level. You should apply for it to be an approved cog. I look forward to what other cogs you create and will be happy to give you constructive feedback if you're open to them. Cheers!

Thanks! Glad it's working out for you. I have applied for Cog Creator so hopefully I can get approved soon. Thanks for your kind words :D

Oh, another enhancement I'd like to suggest. This is only when you have time and not at all urgent. The ability to tag each todo task a level of urgency: critical, major, medium, minor.
Again, this suggestion isn't, ahem, urgent. Only when you find the time in your busy schedule.

Oh, another enhancement I'd like to suggest. This is only when you have time and not at all urgent. The ability to tag each todo task a level of urgency: critical, major, medium, minor.
Again, this suggestion isn't, ahem, urgent. Only when you find the time in your busy schedule.

I have a very mild version of this with [p]todo pin. I will think about it though

Oh, another enhancement I'd like to suggest. This is only when you have time and not at all urgent. The ability to tag each todo task a level of urgency: critical, major, medium, minor.
Again, this suggestion isn't, ahem, urgent. Only when you find the time in your busy schedule.

I have a very mild version of this with [p]todo pin. I will think about it though

Ohhh, the tag would make it easy for someone to focus on "critical" tasks if they query all their todos with the "critical" tag.
Because, trust me, when a lot of people starts using this cog, they will be adding a lot of todos, especially if they added other people as managers.
You are the only cog creator in this space. I think this will make you a force to reckon with. Kudos to you.

Being able to pick a random uncompleted todo would be neat.

Also a particular use case of ours that might be too annoying to implement is to have channel-bound todos in addition to user-bound. Basically have [p]todo channel X commands where X is list / add / etc. and anyone with access to that channel can use those.

I would like to add an entire role to be able to see and edit a list together


createNewTimer should not painc

It's not a good idea to painc, use an error to indicate create timer error maybe better than panic.
https://github.com/nitram509/lib-bpmn-engine/blob/35bc540c8e19e081e83255bdbc6385598c498e13/pkg/bpmn_engine/engine_timer.go#L44

PR in progress, see #36


bandwidth widget only works for one interface

I'm running linux-dash on archlinux-arm on RaspberryPi. It is connected to network through wlan.
The bandwidth widget only shows eth0 although I have eth0 and wlan0 as interfaces in /sys/class/net.
$ ls /sys/class/net
eth0  lo  wlan0

Is this still an issue in the latest version?
Looking at the code this shouldn't be a issue anymore.

works


[ETFE-4440] Update the confirmation page with new links

https://github.com/hmrc/app-config-base/pull/10891
https://github.com/hmrc/app-config-production/pull/23149
https://github.com/hmrc/emcs-tfe-journey-tests/pull/534

All good although https://github.com/hmrc/app-config-production/blob/main/emcs-tfe-explain-delay-frontend.yaml#L54 needs to come out

found it https://github.com/hmrc/app-config-production/pull/23149#pullrequestreview-2257457664