text:
URL Cleanup

This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).
Fixed URLs
Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.

http://www.apache.org/licenses/LICENSE-2.0 migrated to:
https://www.apache.org/licenses/LICENSE-2.0 (https result 200).
http://repo.spring.io/release migrated to:
https://repo.spring.io/release (https result 302).
http://repo.spring.io/snapshot migrated to:
https://repo.spring.io/snapshot (https result 302).

This repository hasn't been built by CI since at least Feb. 2017. We're now archiving the repository. No known CI job builds this.

source: gharchive/pull-request

authors: simonbasle spring-operator

repo: reactor/reactor-samples

license_type: permissive

license_source: github-api


text:
Missing method call mock for ReadableStreamTest::testReadChunk

Follow up of #24 where we missed once method mock. This PR solves that issue

Event loop errors require #36 to fix them

repo: reactphp/filesystem

license: mit

license_source: bigquery


text:
What's the Play about?
In this project, I will be developing a live chat app using firebase as backend with full authentication system
User can signIn using there google accounts and can have real time chats with other users(Just like youtube live chats).
What ReactJS concept will be used to create this play?
Hooks:-I will be using Hooks like useState, useNavigate, useEffect.
Components:- To make code easy to study, I will be dividing my application in few components
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Looking forward to it.

source: gharchive/issue

repo: reactplay/react-play

license: MIT


text:
Dropdown is not positioned correctly when it's in table

components: Dropdown
reactstrap version ^8.0.0
import method es
react version ^16.8.6
bootstrap version ^4.3.1

Check here to reproduce issue: https://stackblitz.com/edit/reactstrap-v7-n7dmce
What is happening?

I want to place a Dropdown component inside a table, and the dropdown menu is not placed at where it should be.
What should be happening?

the dropdown menu should show up underneath the button, instead of cover the entire button.

Steps to reproduce issue
Check out https://stackblitz.com/edit/reactstrap-v7-n7dmce
Error message in console
paste error message with stacktrack here

Code
https://stackblitz.com/edit/reactstrap-v7-n7dmce

Screenshots
The dropdown button inside a table:

dropdown open state:

This is because there isn't enough room within the table. This is an issue with bootstrap itself (and/or popper.js which bootstrap and reactstrap use). Check out https://github.com/twbs/bootstrap/issues/29313
I don't really see a workaround other than either not using a responsive table or making sure the table has a minimum height.
Something that reactstrap could/should do is allow a container prop for dropdown menu to allow the menu to be rendered somewhere else in the DOM (such as the body) so that it isn't rendered in the table.

Hello @TheSharpieOne
I have used reactstrap in my project before. Now, I'm interested to contribute in Reactstrap. I was redirected here from Hacotberfest. Can you please guide me.

@bjupreti Take a look at how PopperContent (which is used for Popovers and Tooltips) handles the container prop to determine if the content is rendered inline or in a portal: https://github.com/reactstrap/reactstrap/blob/45775c01da58e4e29d752367b43a730258398d78/src/PopperContent.js#L183-L185
We need something similar for the DropdownMenu. The user should be able to pass container (which should default to "inline" to match existing functionality and avoid a breaking change) to determine when the menu should render within the DOM (visual positioning of the menu should not change.)
container's type should be targetPropType and you should be able to use the getTarget util to get the DOM element if needed.

okay @TheSharpieOne I will look into it.

Is there any update on this?
I wish that the Dropdown had the same properties as the Tooltip container and boundariesElement.
Is there a way to set this through the modifiers of DropdownMenu.propTypes?

Hello,
I am kind of new to bootstrap. But, from the code example, removing the overflow-x style from the table-responsive element, fix the positioning of the Dropdown element. Can't we use that as a workaround?

authors: TempleClause TheSharpieOne bjupreti kevinxh watinha

repo: reactstrap/reactstrap


text:
ERROR: Download of https://dl.bintray.com/lovell/sharp/libvips-8.4.2-linux-x64.tar.gz failed: Response code 502 (Bad Gateway)

hi there, bintray is down for several months now (according to my research). can we fix this somehow please?

fixed by bumping sharp to version 0.29.3 (instead of 0.17.x).

repo: readium/readium-js-viewer


text:
ePubs with XML 1.1 fail to load for Windows

The Windows XML parser can't handle anything other than XML 1.0.  see https://social.msdn.microsoft.com/Forums/en-US/ebf3df45-c934-46d7-adc9-61d2d1ebead9/which-msxml-verion-support-parse-xml-11?forum=xmlandnetfx
XML 1.1 support was dropped with ePub3 but this is still causing issues with legacy ePub2 books.

I understand that MSXML is used across the board, but just out of interest: are you referring to "modern" Windows RT (managed framework) or "classic" Windows (native C++ code). In other words, did you reproduce this bug using the MFC Readium launcher:
https://github.com/readium/SDKLauncher-Windows/tree/feature/feature_revised_launcher
https://github.com/readium/readium-sdk/tree/feature/feature_revised_launcher
...or using a modern app based on one of these:
https://github.com/readium/readium-sdk/tree/develop/Platform/WinRT
https://github.com/readium/readium-sdk/tree/feature/WinRT
https://github.com/readium/readium-sdk/tree/feature/WindowsRT_v2
https://github.com/readium/readium-sdk/tree/WinPhone8
?

I believe this is the folks who built their own Windows RT-framework launcher so probably the latter.

authors: bdares danielweck rkwright

repo: readium/readium-sdk


text:
Fix dllimport

Import functions by name instead of ordinal to fix powerbattery.dll version compatibility

Fixed by 5781e0ee627924f86f93e573a78f4b89a9e62751

repo: reagcz/IdeapadToolkit


text:
RIO-84 fix (really-core): Object Subscriber Actor fix

Changed the usage of Akka Ask pattern to tell pattern in getting the model

Refer to this link for build results (access rights to CI server needed):
http://ci.really.io//job/really-pr-validator/80/
Test PASSed.

authors: ci-reallylabs montaro

repo: reallylabs/really


text:
RealmSwift reset relationship list

Hi.
I have a relationship that I want to reset with new list, and I can't make is happen. What is the proper way to reset relationship ?
this is what I tried
class MyModel: Object {
    var childs =  List<Child>()

    func change() {
       realm?.write {
            // I want to replace childs relationship list. Nothing I tries is working
            // self.realm?.delete(self.childs)
            // self.chats.removeAll()
            self.childs = List<Child>()
            self.childs.extend(newChildrens)
       }
    }
}

I should preface this answer with a bit of context. The state of dynamic dispatch in Swift is quite limited and ripe with sharp corners. To make things work in Realm, we've had to make some compromises in this area, especially when it comes to List properties.
Since generic class properties cannot be represented in the Objective-C runtime, you shouldn't declare your List properties as dynamic. You're already doing this, so this part is ok. :+1:
However, since non-dynamic properties don't have setters which can be overridden at runtime, Realm can't catch when you're trying to set that property and perform the equivalent operation in the underlying database.
So when you do self.childs = List<Child>() in your code above, Realm is never notified of what you did. Swift :cry:.
So this is why our documentation only ever shows List properties declared as let. Because setters wouldn't work :disappointed:.
So the recommended way to update a List property is to call List.removeAll(), followed by List.extend(_:).
We'd really love to see Swift improve its dynamic capabilities and have filed numerous radars to help nudge future language developments in this direction and we hope you'll do the same if you're unsatisfied with the status quo.

there is issue with this sequence:
list.removeAll()
list.extend(newList)

let's say I already have record in db. by calling extended() realm engine will try to insert it again to the table, and if I have setup with primary key... it wont be possible. So I tried something like this
realm.write {
    realm.delete(list)
    list.removeAll()
    list.extend(newList)
}

Can't set primary key property 'mykey' to existing value '35;75'.
however I still get error that record is already inserted and can't be inserted again (due to primary key)

As you can see from List.extend(_:)'s source, it simply calls down to -[RLMArray addObject:] (source) which will only add the objects if they're not already persisted to that Realm.
So it really depends what's contained in your list variable you've referenced above. If you're assigning a standalone list of unpersisted objects, then they'll be added to the Realm, and if those objects have primary keys and interfere with other objects already stored, you'll get an exception because you'll be trying to add objects with existing primary keys to the Realm, which isn't allowed.
I'm wondering at the moment why we call -[RLMRealm addObject:] rather than -[RLMRealm addOrUpdateObject:] and can't think of a good reason, although it may be escaping me. I'll bring it up with the team so we can reconsider which approach we should be taking here.

regarding removeAll() how about:
    func removeAll() {
        realm?.delete(self)
        _rlmArray.removeAllObjects()
    }

That's not quite what removeAll() is intended to do. Sometimes you simply want to remove those objects from a List without deleting it from the Realm.

I understand and agree. removeAll() just "disconnect" relationship without deleting related items. If you name it deleteAll() it would be a bit more connected to the realm. I mean... 2 functions to reset relationship is not something that came first to my mind ;-)

I do wonder how often you really want to completely reset a to-many relationship. What's a use case in which you wouldn't be better off completely throwing away the existing relationship items rather than inserting/removing?

Since realm don't have "cascade delete" feature, this seems to me quite useful to keep db consistent.

I've created #2005 to improve our documentation around List properties to make them a bit easier to understand. We won't be changing how List.append(_:) or List.extend(_:) work, however. If you're adding unpersisted objects to Realm with conflicting primary keys, you'll get an exception and that's up to you the user to manage.

Are there any updates to this since KVO support? You guys must be able to do something better than what I'm doing...
    // Lists
    public var items: List<BookingItem> = List() {
        willSet(newValue) { items.removeAll() }
        didSet { items.forEach(oldValue.append) }
    }

    public var fees: List<BookingFee> = List() {
        willSet(newValue) { fees.removeAll() }
        didSet { fees.forEach(oldValue.append) }
    }

    public var users: List<BookingUser> = List() {
        willSet(newValue) { users.removeAll() }
        didSet { users.forEach(oldValue.append) }
    }

This is tricky. The short answer is that I don't think you'll get to a much more elegant solution than what you already have.
We can't override List property setters because that type can't be represented by the Objective-C runtime. We can't automatically do what you're doing because willSet and didSet have to be implemented at the property level, not the type level.
We might be able to simplify your code a tiny bit, however. Would this work?
public var items: List<BookingItem> = List() {
    willSet { items.removeAll(); items.extend(newValue) }
}

Yeah this is what I settled on but now I am running into a really strange issue where the list is being reset to an empty list if map["xyz"] == nil. I am only losing values on the List types so I cannot for the life of me figure out why that is happening.

Definitely an ObjectMapper issue, opened an issue here: https://github.com/Hearst-DD/ObjectMapper/issues/252

Okay I figured it out... ObjectMapper will set the property to itself. So instead I made an extension:
// MARK: - List

extension List {

    public func replace(list: List, delete: Bool = false) {
        if list === self { return }
        removeAll()
        appendContentsOf(list)
    }
}

And then I can just use:
public var images: List<Image> = List() {
    willSet { images.replace(newValue) }
}

authors: AndrewBarba jpsim krzyzanowskim

repo: realm/realm-cocoa


text:
How to use filter contains for Integer

Hi All,
I have query like :
select * from objects where to_char(number) like '%45'

how to make this in realm?
I try to using :
"number contains %@", number

It's not currently possible to query a non-string property in Realm as if it was a string. I'll update this ticket to reflect this as a feature request.

Thank you for take care for this issue, will be waiting for the update :+1:

authors: jpsim tarigancana


text:
Support non-relationship nested objects

Goals

It would be great to have possibility to store nested objects as properties which aren't stored as relationships, something similar to CoreData Transformable.
Let's say we parse JSON response with some computed properties for charts without ids or any primary keys, that looks like this:
{
    "id": 1,
     "chartName": "Hello World",
    "chartData": {
          "xAxisValue": 10,
         "yAxisValue": 12,
         "xAxisMax": 30,
         "yAxisMin": 0,
     }
}

Realm objects to reflect that would look something like this:
class Chart: Object {
    @objc dynamic var id: String = ""
    @objc dynamic var chartName: String = ""
    @objc dynamic var chartData: ChartData?

    override class func primaryKey() -> String? {
        "id"
    }
}

class ChartData: Object {
    @objc dynamic var xAxisValue: Double = 0.0
    @objc dynamic var yAxisValue: Double = 0.0
    @objc dynamic var xAxisMax: Double = 0.0
    @objc dynamic var yAxisMin: Double = 0.0
}

Since ChartData has no primary key, each time JSON above is parsed it will add the same ChartData to realm and cause "duplicates", while there will be only one Chart since it has primary key.
I guess one way to work around this is to have Chart object look like:
class Chart: Object {
    @objc dynamic var id: String = ""
    @objc dynamic var chartName: String = ""
    @objc dynamic var xAxisValue: Double = 0.0
    @objc dynamic var yAxisValue: Double = 0.0
    @objc dynamic var xAxisMax: Double = 0.0
    @objc dynamic var yAxisMin: Double = 0.0
}

and have computed property which returns instance of ChartData with necessary values passed in.
Though, this solution doesn't really work if for example Chart object had chartData, chartDataStocks, chartDataActivity etc.
Expected Results
Have something like RealmEmbedded<Object> or something similar to mark nested object as non-relationship. Maybe behind the scenes it can use possibility of Data type property and use NSKeyedArchiver and NSKeyedUnarchiver to serialize data.
for example:
class Chart: Object {
    @objc dynamic var id: String = ""
    @objc dynamic var chartName: String = ""
    @objc dynamic var chartData: RealmEmbedded<ChartData>?
}

Actual Results

Steps to Reproduce

Code Sample

Version of Realm and Tooling

Realm framework version: ?
Realm Object Server version: ?
Xcode version: ?
iOS/OSX version: ?
Dependency manager + version: ?

Well, there's some good news. Embedded objects are a feature of the upcoming next major version of Realm, and have already been implemented on the v10 branch. There'll be an official beta release in the next few days, but you can also try building against that branch now if you want to try it out.
The API is slightly different (ChartData will inherit from EmbeddedObject rather than it being something determined per-property) but semantically it should match what you're asking for.

Hah that's awesome! thanks @tgoyne

authors: UrosZivaljevic tgoyne


text:
Realm 10.0 breaks the debugger in the host app

Goals
Debug my app while using Realm 10.0 RC 1
Expected Results
Being able to print out object descriptions in the debugger
Actual Results
When printing description of any object in the console, this error (and nothing else) is printed:
(lldb) po view
error: virtual filesystem overlay file '/Users/realm/workspace/cocoa-pipeline/DerivedData/Build/Intermediates.noindex/Realm.build/Release-iphonesimulator/RealmSwift.build/all-product-headers.yaml' not found

error: couldn't IRGen expression. Please check the above error messages for possible root causes.

This happens even in files that have nothing to do with Realm and that doesn't reference Realm. I have tried clean build, deleting derived data and rebooting my Mac. Other debugger commands works.
Peter Steinberger has a blog post about this error message that might be helpful:
https://steipete.com/posts/couldnt-irgen-expression/
Steps for others to Reproduce
Not entirely sure, but I updated from Realm 10.0 Beta 5 to Realm 10 RC1 through Carthage and after that the problem occured. A colleague pulled the changes after I updated Realm does not experience the issue though.
Code Sample
Version of Realm and Tooling
ProductName:	Mac OS X
ProductVersion:	10.15.7
BuildVersion:	19H2

/Applications/Xcode.app/Contents/Developer
Xcode 12.0.1
Build version 12A7300

/usr/local/bin/pod
1.9.3
(not in use here)

/bin/bash
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin19)

/usr/local/bin/carthage
0.36.0
github "realm/realm-cocoa" "v10.0.0-rc.1"

/usr/bin/git
git version 2.24.3 (Apple Git-128)

Hi @miklselsoe ,
Unfortunately I cannot reproduce this issue too.
Did you try the workaround from the post you've mentioned here? Does it work for you?

Yes. Adding an empty Objective-C file to the project fixes the issue for me.

Closing as this doesn't appear to be a Realm specific issue.

authors: leemaguire miklselsoe pavel-ship-it


text:
Move modeling types to io.realm.kotlin.types package

This PR cleans up top level package. Primarily by moving all modeling related types to io.realm.kotlin.types
Closes #837

Looks good to me. Only concern I have is re-using the filenames in io.realm.kotlin.ext like Realm, since that would probably make it slightly confusing if consumed from Java
Maybe rename them to <X>Ext or something like that?

I moved them to <X>Ext.kt. Was also nicer to work with in the IDE

repo: realm/realm-kotlin


text:
Logging out while offline doesn't update app.currentUser, crashes after adding item

How frequently does the bug occur?
Always
Description
If I logout while offline, I get Error logging out: The Internet connection appears to be offline. and the UI doesn't change (the logout button is still there, as well as all the items of the user).

Then, if I add an item while still offline, the app crashes because there is no user Cannot start a sync session for user '[userId]' because this user has been removed
Stacktrace & log output
Error logging out: The Internet connection appears to be offline.
2023-07-13 17:23:42.225619-0400 RealmTemplate[90327:2966668] Info: Connection[1]: Disconnected
libc++abi: terminating with uncaught exception of type std::logic_error: Cannot start a sync session for user '64b06b2c314e2e772278412c' because this user has been removed.
(lldb)

Can you reproduce the bug?
Always
Reproduction Steps
Here is a repo to reproduce the issue. The app id is given via an environment variable.

Turn on airplane mode
Press logout button
Try to add an item

The same steps can be used with this tutorial, but the error is:
Thread 1: Fatal error: Unexpectedly found nil while unwrapping an Optional value

// in ItemsView, user is nil
$itemGroup.items.append(Item(value: ["ownerId":user!.id]))

What's strange is that user, which comes from app?.currentUser, is nil, but the LoginView wasn't displayed.
if let user = app.currentUser {

} else {
    // If there is no user logged in, show the login view.
    LoginView()
}

Version
13.15.1
What Atlas Services are you using?
Atlas Device Sync
Are you using encryption?
No
Platform OS and version(s)
macOS Ventura 13.4.1
Build environment
Xcode version: 14.3.1
Dependency manager and version: none/native?

Hi @bogdan-pechounov I was able to reproduce your issue and we are discussing if the current behaviour is the correct one. You can track this on this issue https://github.com/realm/realm-core/issues/6806

repo: realm/realm-swift


text:
南京诚聘 NFT策划与项目经理 9-14K

公工作职责：
1、设计NFT IP的策划、编写策划文档。策划内容不仅限于世界观，美术风格，音乐，动画以及周边衍生品的开发；
2、持续关注NFT交易平台，NFT项目，GameFi等领域，输出竞品调研报告；
3、与美术、程序等部门充分沟通配合，推进项目进度；
4、IP项目启动后，负责对项目生命周期管理、产品预售等营销活动设计；
5、运营粉丝社区，分析用户画像，策划线上和线下活动，执行，评估活动效果。
任职要求：
1、具备一定的英语读写能力；
2、具备良好的审美能力；
3、逻辑清晰，具有较强的学习能力和钻研精神，勇于接受挑战；
4、热爱游戏行业，工作热情高，认真负责，执行力强，具有良好的沟通能力、合作能力与责任心。
优先考虑：
1、本科及以上，有海外留学、澳大利亚留学经验；
2、热爱国内外高品质游戏，了解最新玩法；
3、文字功底深厚，想象力丰富；
4、在游戏公司担任策划运营2年以上，或在区块链游戏项目中有1年以上策划以及运营经验；
5、日语N2及以上；
6、美术专业出身；
7、音乐爱好者。

工作地点：南京
联系方式：
发简历到web3china@126.com

repo: rebase-network/who-is-hiring


text:
Please invite me to the Recode-Hive GitHub Community Organization

Name
harshline
Discord Username (if applicable)
harshline
Additional Context
I am new to GitHub and came here from your course.

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people

authors: Harshline sanjay-kv

repo: recodehive/Support


text:
As a developer, I want a design for how a workflow node connects with another workflow node in the workflow tree, so that I can implement it in code

Summary

Design how a workflow node connects to another workflow node, and get it approved

Time Requirements

1 day

Acceptance Criteria

Design how a workflow node connects to another workflow node, and get it approved

Closed for not following issue template.

repo: recro/medulla


text:
Add ConvertTrial() to Subscription

To convert trial to paid subscription with billing info:
subscription.ConvertTrial(subscription.Account.BillingInfo)

To convert trial to paid subscription with transaction type:
subscription.ConvertTrial(transactionType: "moto")

I think this could go something like this:
        private void ConvertTrial(Subscription request)
        {
            var url = UrlPrefix + Uri.EscapeDataString(Uuid) + "/convert_trial";
            if (request == null)
            {
                Client.Instance.PerformRequest(Client.HttpRequestMethod.Put,
                    url,
                    ReadXml);
            }
            else
            {
                Client.Instance.PerformRequest(Client.HttpRequestMethod.Put,
                    url,
                    request.WriteSubscriptionXml,
                    ReadXml);
            }
        }

        public void ConvertTrialMoto() {
            var request = new Subscription()
            {
                TransactionType = "moto"
            };
            ConvertTrial(request);
        }

        public void ConvertTrial(string threeDSecureActionResultTokenId = null) {
            // by default there is no body
            Subscription request = null;

            // if there is a 3dsecure token, let's build a request body for it
            if (threeDSecureActionResultTokenId != null)
            {
                request = new Subscription()
                {
                    Account = new Account()
                    {
                        BillingInfo = new BillingInfo()
                        {
                            ThreeDSecureActionResultTokenId = threeDSecureActionResultTokenId
                        }
                    }
                };
            }

            ConvertTrial(request);
        }

You'd also need to add an internal setter for Subscription#Account. I added this to line 57:
            internal set { _account = value; }

That gives you 3 ways to call it:
// For non-3ds where billing info is already there to be converted
subscription.ConvertTrial();

// Optionally pass in a token if we need 3ds auth
subscription.ConvertTrial(token);

// If we want to explicitly skip the authorization step (like for a MOTO order)
subscription.ConvertTrialMoto();

Unfortunately though, we can't just use the WriteSubscriptionXml serializer and the second and third calls won't work out of the box. Although it mostly works, it creates a bunch of junk which the server will not like. Example when using a token:
<subscription>
  <plan_code />
  <account>
    <account_code />
    <billing_info>
      <three_d_secure_action_result_token_id>token</three_d_secure_action_result_token_id>
    </billing_info>
  </account>
  <currency />
  <customer_notes />
  <terms_and_conditions />
  <vat_reverse_charge_notes />
  <po_number />
  <subscription_add_ons />
  <quantity>0</quantity>
</subscription>

So I think you'll want to write your own writeXml* serializer in Subscription which only renders out the appropriate xml like so:
<subscription>
  <account>
    <billing_info>
      <three_d_secure_action_result_token_id>token</three_d_secure_action_result_token_id>
    </billing_info>
  </account>
</subscription>

authors: bhelx joannasese

repo: recurly/recurly-client-net


text:
Fix xpath used to select project in elyra tests

Signed-off-by: Jorge Garcia Oncins jgarciao@redhat.com

This is already done in 1763

repo: red-hat-data-services/ods-ci


text:
[TEST COMMIT] Investigate merge bug

Logs: https://prow.ci.openshift.org/pr?query=is%3Apr+repo%3Ared-hat-storage%2Fmcg-ms-console+author%3Arexagod+head%3Amerge-bug

Similar issue: https://github.com/kubernetes/test-infra/issues/24141#issuecomment-1016624541

/lgtm
/approved

/lgtm
/approve

/override approve

/override tide

/lgtm
/approve

/hold

/approve

/retest

/retest

/override mcg-ms-console-e2e-aws

/override ci/prow/mcg-ms-console-e2e-aws

/retest

/retest

Finally.

/test mcg-ms-console-e2e-aws

/test mcg-ms-console-e2e-aws

@rexagod hi, can you share, what was the solution for fixing the tide, we are seeing the same issues in our prow. Thanks!

authors: SanjalKatiyar agarwal-mudit cloudbehl furkatgofurov7 rexagod

repo: red-hat-storage/mcg-ms-console


text:
Used any_io_executor as the base of predefined mqtt protocol.

ws and wss don't support any_io_executor due to the following issue: https://github.com/boostorg/beast/issues/2775

When https://github.com/boostorg/beast/issues/2775 would be fixed, then would apply any_io_executor to beast (ws and wss).


text:
Update rediscluster client to v2.1.3

ElastiCache will occasionally replace nodes during a configurable maintenance window ("These replacements are needed to apply mandatory software updates to your underlying host"), they refused to provide information on the maintenance failover process, but it looks similar to CLUSTER FAILOVER (source).
For ElastiCache redisclusters that do not have replica nodes for failover, this can trigger an edge case, such that rediscluster tries to free a connection referencing a local variable before it is assigned (commit). This was patched in v2.1.3.
In our case, the rediscluster client entered a bad state, and all requests to the redis node failed. The only way to recover was a manual service restart.

Baseplate.py only sets a minimum version (https://github.com/reddit/baseplate.py/blob/develop/setup.py#L15, 2.1.2) which we can happily bump if 2.1.2 is really bad. But the actual version used by your service is entirely up to your service, not baseplate.

I see. I suppose bumping the minimum version to 2.1.3 is the best course of action.

Cool! Are you interested in submitting a patch for that?

authors: jerroydmoore spladug


text:
Add styled components example

Demonstrate how to use styled-components with react-server

Like I mentioned last night, everything looks good aside from swapping themes on the fly, which updates the props but doesn't apply them to the child components. I don't know enough about styled components to provide any meaningful help with that, though.

@egid I don't know what's up -- its working for me

Good call, that fixed it. Working as expected!

authors: doug-wade egid

repo: redfin/react-server


text:
Appstudio update test-component-pac-iwwl

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-iwwl-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
13 seconds
clone-repository

✅ Succeeded
26 seconds
build-container

✅ Succeeded
22 seconds
inspect-image

✅ Succeeded
18 seconds
deprecated-base-image-check

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
52 seconds
clamav-scan

✅ Succeeded
11 seconds
sbom-json-check

✅ Succeeded
16 seconds
label-check

✅ Succeeded
7 seconds
show-summary

✅ Succeeded
5 seconds
show-sbom


text:
Appstudio update test-component-pac-sait

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-sait-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
19 seconds
sast-snyk-check

✅ Succeeded
1 minute
build-container

✅ Succeeded
25 seconds
deprecated-base-image-check

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
42 seconds
clamav-scan

✅ Succeeded
24 seconds
inspect-image

✅ Succeeded
6 seconds
sbom-json-check

✅ Succeeded
6 seconds
show-sbom

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/test-component-pac-sait-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
21 seconds
clone-repository

✅ Succeeded
26 seconds
build-container

✅ Succeeded
8 seconds
sast-snyk-check

✅ Succeeded
56 seconds
deprecated-base-image-check

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
26 seconds
inspect-image

✅ Succeeded
25 seconds
sbom-json-check

✅ Succeeded
33 seconds
clair-scan

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary


text:
Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
16 seconds
sanity-inspect-image

✅ Succeeded
15 seconds
deprecated-base-image-check

✅ Succeeded
16 seconds
clair-scan

✅ Succeeded
59 seconds
clamav-scan

✅ Succeeded
11 seconds
sbom-json-check

✅ Succeeded
13 seconds
sanity-label-check

✅ Succeeded
13 seconds
sanity-optional-label-check

✅ Succeeded
7 seconds
show-summary

repo: redhat-appstudio-qe/hacbs-test-project


text:
fix: RHTAP:808: Rework common controller

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Issue ticket number and link
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/hold

/unhold


text:
Add Tekton Result build configuration for HACBS

Changes

PR to add pull request/push PipelineRun configs to tekton results: https://github.com/openshift-pipelines/tektoncd-results/pull/5
Jira issue: PLNSRVCE-548

Signed-off-by: Avinal Kumar avinal@redhat.com

Hello @Michkov can you please check if everything is correct?

/ok-to-test

/retest

Is there a problem from tekton-results side?

/retest

authors: Michkov avinal

repo: redhat-appstudio/infra-deployments


text:
New CoP quay repositories

New quay repositories for group-sync-operator and project-initialize-operator

/lgtm

repo: redhat-cop/org


text:
Remove some unneeded options

cc: @kameshsampath

thanks @nicolaferraro

authors: kameshsampath nicolaferraro


text:
Improvements in the dump package

The PrintDataInFormat method could do with some minor changes to improve it.

[ ] Change the name to something cleaner, and shorter. DumpFormatted is my preferred (fewer words used), I am open to other suggestions.
[ ] The io.Writer parameter always comes first in a Go method which uses that writer. Let's put it first here too.
[ ] Return an error if any from this method. We should not be ignoring errors here. It is up to the method usage to decide whether to use the error or not, it should be returned.
[ ] Create a special type DumpFormat and set the constants XFormat constants to use this format (nice to have, not needed though). This means we can limit the valid values for format used in PrintDataInFormat.

I propose to use just Formatted as the function name to remove package name duplication from it.

@craicoverflow I did everything except the last item. Could you please provide more information about it?

type OutputFormat = string

const JSONFormat OutputFormat = "json"

authors: alexal craicoverflow

repo: redhat-developer/app-services-cli


text:
chore(all): re-generate OpenAPI client(s)

This pull request was auto-generated
This PR adds the latest generated API clients.
Actions:

Review changes and validate against the examples.
Make changes where necessary.
Approve and merge with proper commit message tha represents change

NOTE: After merge separate PR will be created for generation of errors SDK.

Invalid PR can be closed

authors: app-services-ci wtrocki

repo: redhat-developer/app-services-sdk-js


text:
Add the ConfigMap for rollouts

Added the ConfigMap named 'argo-rollouts-config' in the Rollouts namespaces.

LGTM!

repo: redhat-developer/gitops-must-gather


text:
chore: update redis config from upstream argocd-operator

Signed-off-by: John Pitman jpitman@redhat.com
What type of PR is this?
/kind enhancement
What does this PR do / why we need it:
This PR updates the redis config files based on the changes to upstream argocd-operator.  It also updates the version of the argocd-operator used to the latest master.
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

How to test changes / Special notes to the reviewer:

Install the operator
Install an Argo CD instance with HA mode set

apiVersion: argoproj.io/v1alpha1
kind: ArgoCD
metadata:
  name: argocd
spec:
  ha:
    enabled: true

Ensure all workloads start.  Note, it may take many minutes for the argocd-redis-ha-server-* workloads to completely start

% kubectl get po
NAME                                                READY   STATUS    RESTARTS   AGE
argocd-application-controller-0                     1/1     Running   0          19m
argocd-dex-server-7775d96b59-8r97p                  1/1     Running   0          19m
argocd-redis-ha-haproxy-5f9b6b9b99-qsx6d            1/1     Running   0          19m
argocd-redis-ha-server-0                            2/2     Running   0          19m
argocd-redis-ha-server-1                            2/2     Running   0          12m
argocd-redis-ha-server-2                            2/2     Running   0          11m
argocd-repo-server-748f77b477-svmzl                 1/1     Running   0          19m
argocd-server-6bd5bc5bd-fzgsf                       1/1     Running   0          19m

/retest

repo: redhat-developer/gitops-operator


text:
Update description in CSV

Update description in CSV

/lgtm
/approve

/lgtm
/approve


text:
Upgrade go.mod dependencies

What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind bug
/kind cleanup
/kind failing-test
/kind enhancement
/kind documentation
/kind code-refactoring

What does this PR do / why we need it:
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
How to test changes / Special notes to the reviewer:

/retest

/retest

repo: redhat-developer/kam


text:
Add ports information into odo describe <componentName>

[kind/Enhancement]

Which functionality do you think we should update/improve?
odo describe <svcName>  should include ports information for human and machine readable output.
Port infor:

number
protocol

Why is this needed?
When linking components and creating urls for components port parameter is required in case there are more than one port is exposed.

+1
Currently ComponentSpecs looks like this
type ComponentSpec struct {
	Type    string          `json:"type,omitempty"`
	Source  string          `json:"source,omitempty"`
	URL     []string        `json:"url,omitempty"`
	Storage []string        `json:"storage,omitempty"`
	Env     []corev1.EnvVar `json:"env,omitempty"`
}

we need to add ports to it
type ComponentSpec struct {
	Type    string          `json:"type,omitempty"`
	Source  string          `json:"source,omitempty"`
	URL     []string        `json:"url,omitempty"`
	Storage []string        `json:"storage,omitempty"`
	Env     []corev1.EnvVar `json:"env,omitempty"`
        // list of ports formated as port_number/protocol (8080/tcp)
	Port     []string `json:"env,omitempty"`

}

@dgolovin What do you mean by TLS Information. As far as I know containers are not exporting any information about TLS

@kadel you are correct, I removed TLS from the issue.

/milestone 1.0

we can simply use  list of urlSpecin place of []string in URL
type UrlSpec struct {
	Host     string `json:"host,omitempty"`
	Protocol string `json:"protocol,omitempty"`
	Port     int    `json:"port,omitempty"`
}

we can simply use list of urlSpecin place of []string in URL
type UrlSpec struct {
	Host     string `json:"host,omitempty"`
	Protocol string `json:"protocol,omitempty"`
	Port     int    `json:"port,omitempty"`
}

earlier version of output had this but later on removed

UrlSpec and Protocol in ComponentSpec are two different things.

I would like to work on this :tada:

authors: dgolovin kadel surajnarwade

repo: redhat-developer/odo


text:
Cleanup odo-init image of unnecessary logic

odo-init image has certain hacks (variables and checks) in it to ensure things work well when using s2i builder images. Devfile provides support for adding these custom variables, which then odo should simply re-use.
Acceptance Criteria

[ ] TBD

/kind cleanup
/triage needs-information

/milestone 2.4

/remove-lifecycle stale
@mohammedzee1000 @anandrkskd I vaguely remember one of you working on something along the lines of what this issue is about.

TBH, I don't  remember working on anything similar.

/close
We are no longer using odo-init image.
Feel free to reopen if I got this wrong.

authors: anandrkskd dharmit valaparthvi


text:
RHIDP-4814: Move observe related content to new titles (#689)

Cherry-picked PR for https://github.com/redhat-developer/red-hat-developers-documentation-rhdh/pull/689
No reviews needed

IMPORTANT: Do Not Merge - To be merged by Docs Team Only
Version(s):

Add the relevant labels to the Pull Request.
Issue:

Merging this PR, since this is a cherry-picked PR.


text:
vscode-quarkus should not be a Java formatter

vscode-quarkus should not be a Java formatter.
To reproduce this issue:

Open a Java file while vscode-quarkus is running.
Right click in the editor and click "Format Document With..."
The user is now prompted to choose a formatter:

The "Quarkus" entry should not be present in this list, since VS Code Quarkus does not format Java files.

I think this problem comes from since we have supported CodeLens and Hover in Java file with Quarkus LS. I fear that there is no solution.

@jrieken @aeschli any idea on how to rget the language server register the formatter for the quarkus-properties type/language and not for java? (See https://github.com/redhat-developer/vscode-quarkus/blob/master/src/extension.ts#L128)

This UI shows all installed extensions, it doesn't check if or what formatter an extension provides because we don't know that statically.... Tho, we could show formatters that we have seen contribute a formatter for that language in the past first.

@fbricon perhaps it should be better to have 2 language server :

one for application.properties
one for java file.

Today Java CodeLens, Hover uses application.properties file and doesn't requires projectInfo, so I think we could move Java features in a new language server. Having a new language server for Java, will give the format capability as none and will fix this issue.
What do you think about that?

I don't know if it could work, but by reading https://github.com/microsoft/language-server-protocol/issues/996 we could try to follow the suggestion https://github.com/microsoft/language-server-protocol/issues/996#issuecomment-633975409
In other words when regitser capability for format (textDocument/formatting and textDocument/rangeFormatting) is sent from the server at https://github.com/redhat-developer/quarkus-ls/blob/27d554f92d8d545145410fc34cf3815f86a117bd/microprofile.ls/com.redhat.microprofile.ls/src/main/java/com/redhat/microprofile/settings/capabilities/MicroProfileCapabilityManager.java#L106 we could add documentSelector to say that only language microprofile-properties and quarkus-properties is supported.
{
	"method": "client/registerCapability",
	"params": {
		"registrations": [
			{
				"id": "79eee87c-c409-4664-8102-e03263673f6f",
				"method": "textDocument/formatting",
				"registerOptions": {
					"documentSelector": [
						{ "language": "microprofile-properties" },
						{ "language": "quarkus-properties" }
					]
				}
			}
		]
	}
}

authors: angelozerr fbricon jrieken xorye

repo: redhat-developer/vscode-quarkus


text:
Use 2 space indent

As been discussed with @joshuawilson, in JS/TS projects using 2 space indent is more common:

https://javascript.info/coding-style
https://google.github.io/styleguide/jsguide.html#formatting-block-indentation
https://standardjs.com/rules.html
https://www.reddit.com/r/javascript/comments/5rjrcy/why_is_twospace_tab_indent_becoming_the_standard/

So this PR re-format all files and force to use 2 space indent.

Codecov Report

Merging #159 into master will not change coverage.
The diff coverage is 71.8%.

@@           Coverage Diff           @@
##           master     #159   +/-   ##
=======================================
  Coverage   75.73%   75.73%
=======================================
  Files          28       28
  Lines        1624     1624
  Branches      253      253
=======================================
  Hits         1230     1230
  Misses        394      394

Impacted Files
Coverage Δ

src/yaml-support/tkn-yaml-scheme-generator.ts
32.35% <0%> (ø)
:arrow_up:

src/util/platform.ts
100% <100%> (ø)
:arrow_up:

src/tekton/taskrun.ts
100% <100%> (ø)
:arrow_up:

src/util/download.ts
100% <100%> (ø)
:arrow_up:

src/util/archive.ts
100% <100%> (ø)
:arrow_up:

src/commands.ts
100% <100%> (ø)
:arrow_up:

src/yaml-support/tkn-scheme-storage.ts
100% <100%> (ø)
:arrow_up:

src/util/progress.ts
100% <100%> (ø)
:arrow_up:

src/tekton/pipelinerun.ts
100% <100%> (ø)
:arrow_up:

src/util/windowUtils.ts
100% <100%> (ø)
:arrow_up:

... and 20 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 0ab37c1...b9e057f. Read the comment docs.

authors: codecov-io evidolob

repo: redhat-developer/vscode-tekton


text:
@themr0c - Can you confirm whether the WARNING is still needed? It was still in the content that was recently updated. Thanks.

About the following warning in modules/user-guide/pages/using-vale-github-action.adoc
WARNING: Because of GitHub token permissions, the Vale GitHub Action cannot post annotations to pull requests from a forked repository.

@themr0c - Can you confirm whether the WARNING is still needed? It was still in the content that was recently updated. Thanks.
Originally posted by @michelle-purcell in https://github.com/redhat-documentation/vale-at-red-hat/pull/303#discussion_r970006058

Not relevant any more

Implemented, closing

repo: redhat-documentation/vale-at-red-hat


text:
File Server: handle 204

handles a 204 returned from the file server

see #246 for latest


text:
operator ibm-block-csi-operator-community (1.11.2)

Thanks for submitting your Operator. Please check the below list before you create your Pull Request.
New Submissions

[x] Are you familiar with our contribution guidelines?
[x] Are you familiar with our operator pipeline?
[x] Have you tested your Operator with all Custom Resource Definitions packaging?
[x] Have you tested your Operator in all supported installation modes?
[x] Have you considered whether you want to use semantic versioning order?
[x] Is your submission signed?
[x] Is operator icon set?

Your submission should not

[x] Add more than one operator bundle per PR
[x] Modify any operator
[x] Rename an operator
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description of the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/lgtm

/lgtm

this is failing for an unclear reason, and also not allowing to mark LGTM flag
opened issue 4543

@jfrancin please note that @ArbelNathan already attempted to mark with /lgtm flag last week
https://github.com/redhat-openshift-ecosystem/community-operators-prod/pull/4542#issuecomment-2112051959

1.11.1 does indeed exist. But only in the v4.13 bundle index. It doesn't exist in the OperatorHubs for RHOCP 4.14 and 4.15.
The 1.11.1 version's annotations.yaml file contains: com.redhat.openshift.versions="v4.10-v4.13".
The 1.11.2 version's annotations.yaml file contains: com.redhat.openshift.versions="v4.13-v4.15".
Since the 1.11.1, the version being replaced, does not exist in 4.14 or 4.15, the replaces directive cannot be satisfied, as the precursor version must exist in every version's operatorHub/bundle index.  You cannot use replaces in an operator bundle which supports a wider range of versions than the one being replaced.
Instead, change the replaces: ibm-block-csi-operator.v1.11.1 to: a skips block:
spec:
  [...]
  version: 1.11.2
  skips:
     - ibm-block-csi-operator.v1.11.1
  [...]

authors: csiblock jfrancin kasserater


text:
PPC64LE image tags should have prefix

Otherwise they overwrite our, existing images.
ISV-1495

cc: @mayurwaghmode


text:
Fix 'Scenario' typo in the docs

Description

Just a simple typo fix.
Secnario -> Scenario

Checklist

[ ] Does npm test pass with this change (including linting)?
[ ] Is the new or changed code fully tested?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Hello! Do I need to complete the checklist for this simple doc change?
I honestly don't wanna clone the repo locally just to run npm test, specially since it should never fail (unless it checks the docs somehow).

Thanks!

@HeCorr Thank you!

authors: HeCorr leibale

repo: redis/node-redis


text:
How to remove z in Datetime when deserializing

Hello
I had this issue that datetime have the offset +02.00 when deserializing but I upgraded to 0.4.2 The offset removed but it had been replaced with z in the datetime which the front also converts it to local time from utc so he add offset to the time.
how can I remove the z when deserializing to prevent this issue ?

Something like DateTime.SpecifyKind(item.DateTime, DateTimeKind.Unspecified)
fixs the problem but I can't foreach for every record in a large table when getting it
that would be a headache for  the server

So like @zulander1 points out the date-times are converted to/from longs when being stored in Redis - be curious to see what you mean (we provide the ability to set a timezone now for you application - may be something that needs to get more granular).

Going to close this out for now (I think 0.4.2's timezone setting capabilities should have resolved this) if not feel free to reopen with something that I can reproduce with :) - thanks!

repo: redis/redis-om-dotnet


text:
Implement a dry run mode for the migrator that outputs what it would do.

Implement a dry run mode for the migrator that logs what it would do but doesn't do it.  See TODO notes in the code.

it would be a good. I had an issue with it, see this

#110

authors: gam-phon simonprickett

repo: redis/redis-om-python


text:
Fix primary update condition for cluster mode.

I'm using Redisson to interact with my Elasticache Redis cluster. The configuration is 15 shards with only primary nodes.
If I create a replica for one shard and failover the primary node to the replica, the new connection won't be established.

Thanks for contribution!

authors: JerryWzc mrniko

repo: redisson/redisson


text:
feat: simplify interface for team secrets

As a result of last conversation during the demo, I simplified team secrets interface. Thanks for pointing this out 👍

@Morriz your change request is blocking now. Let me know if there is anything else that need a change.

Just that last thing

authors: Morriz j-zimnowoda

repo: redkubes/otomi-core


text:
Feature requests project public for editing?

Describe the bug
I read the text on the "Projects" tab above, it says:

Hello, Bonjour, Hola, Hej, Ni Hao, Ciao! Feel free to contribute and share your ideas and feature requests here.
We will frequently go through all requests and make a choice of feature that are possible candidates for the next release.
Please use the existing categories or create a new one if needed.

I get the impression we're meant to be able to make changes to it, but I can't find any way to do that. I can just read the existing feature requests.
This is just to let you know about this issue, I don't have an opinion on if having a public project for making feature requests in is a good idea or not, but the text should match the intention.

We plan to use agil project management methods to speed up and better organize the development of OrbTk. But not everything is ready. This text is in artifact of this plan.

I will close this now and add more details later.

authors: FloVanGH lukors

repo: redox-os/orbtk


text:
Win condition "TIME" possibly assigned to wrong team

Describe the bug
When the win condition is TIME, sometimes, the wrong team gets the the won=true boolean set.
See this simple example of an exported JSON (exported with r6-dissect commit b593acf):
  [...]
  "teams": [
    {
      "name": "YOUR TEAM",
      "score": 0,
      "won": true,
      "winCondition": "TIME"
    },
    {
      "name": "OPPONENTS",
      "score": 1,
      "won": false
    }
  ],
  [...]

This is the first round of the match.
Since the score after this round is 0:1, won=true should be on the team with score=1, not score=0.
To Reproduce
Steps to reproduce the behavior:

Export a round replay that ended on win condition "TIME" (I can't attach match replay files due to GitHub file type restrictions, if you need a file, I will PM it to you)
See how the won=true value is sometimes* wrong

*note on "sometimes":
I don't have enough match replay files to find out under what conditions it is wrong, I suspect it does not always happen.
It might be related to whether the observing player starts as defender or attacker.
Expected behavior
won=true should be set on the winning team.
Additional context
All other win conditions seem correct, thus, I assume that the issue is related to this section:
https://github.com/redraskal/r6-dissect/blob/b593acfca32cb2929be6164f81a3ed23c9db007f/dissect/time.go#L102-L107
This in turn relies on alliances which is set here:
https://github.com/redraskal/r6-dissect/blob/b593acfca32cb2929be6164f81a3ed23c9db007f/dissect/time.go#L56-L60
I think this all boils down to the TeamIndex field being wrong as already partly discussed in #20, but I wanted to put this out there just to be sure in case it's actually related to a different issue.
Let me know if I can help with anything. I'm confident in Go, but not with the match replay file format, but I'll try my best.

Can you send a replay to me @ ben@ryben.dev?

Can you send a replay to me @ ben@ryben.dev?

Done.

Looks resolved to me, at least based on the replays available to me.
Love that you included a non-integer team role too, lovely stuff.
Feel free to close the issue if you feel it is resolved.

authors: redraskal stnokott


text:
doc not match in setup with nextjs

What docs page needs to be fixed?

Section:
Page:
https://redux.js.org/usage/nextjs#loading-initial-data

What is the problem?
What should be changed to fix the problem?
app/ProductName.tsx
need to fix lib/providers.tsx
in doc

real in npx create-next-app --example with-redux my-app

how can i Contributing this issue?

we actually have an ongoing PR to get the template updated to match the docs, rather than the other way around :)
https://github.com/vercel/next.js/pull/59864/

repo: reduxjs/redux


text:
Issue with opening Content Slots

Overview:
When clicking on a Content Slot comment, the correct label will show, but the button is missing.
I'm wondering if it's because of the siteID missing from the comment, so the code below doesn't build the slotURL correctly.
But it should still build the button, so I'm at a loss of what it could be exactly.
if (props.type === 'slot' && props.id && domain && siteID)

@nancyhstorey I was able to sort out a solution after talking this over with @evechey.  Seems the actual issue here was that the siteId I was trying to fetch from the URL could possibly not actually be the correct siteId needed.
The correct solution here was to check for the siteID in a pre-existing javascript variable that should exist on all SFCC sites CQuotient.siteId.
There is a new release that just went out today, that should be approved and available for download soon for both Chrome and Firefox.
If you would like early access to this new version, you can download the extension manually using the Manual Install notes in the README.

authors: manifestinteractive nathanhemesath

repo: redvanworkshop/sfcc-devtools


text:
[Bug] font color all turned black

Describe the bug
Font color of all screen icons turned black and become unreadable.
To Reproduce
Steps to reproduce the behavior:

Selected a gif
stretch to full screen scale
no matter what color to pick in color picker, all font colors on screens changed to black, even though the gif background is black, so texts are illegible.

Expected behavior
Allow text colors to be changed to white or some other light color.
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: Mi 11 Ultra
Android version: Android 13
App version [latest 2.2.0]

Additional context
Add any other context about the problem here.

Hey! Is it happening with a gif in particular, or with any gif?

If so, can you share that gif, and maybe a screenshot of the issue? I have never seen such behaviour

Before applying gif wallpaper

After:

Do you see the same behavior with another gif that would have a dark background like this one? https://jn3008.tumblr.com/image/621813774846951424

Yes:

authors: redwarp sdssrr

repo: redwarp/gif-wallpaper


text:
Update Login.tsx

Added required, so that it will not submit the form with empty value.

Good one!

authors: hkjena reeversedev

repo: reeversedev/devdevdev


text:
releases-tab: Cannot read properties of undefined (reading 'append')

Description
TypeError: Cannot read properties of undefined (reading 'append')
    at appendBefore (chrome-extension://hlepfoohegkhhmjieoechaddaejaokhf/refined-github.js:5711:60)
    at chrome-extension://hlepfoohegkhhmjieoechaddaejaokhf/refined-github.js:5765:54
    at async init (chrome-extension://hlepfoohegkhhmjieoechaddaejaokhf/refined-github.js:5744:55)
    at async runFeature (chrome-extension://hlepfoohegkhhmjieoechaddaejaokhf/refined-github.js:3955:20)
    at async setupPageLoad (chrome-extension://hlepfoohegkhhmjieoechaddaejaokhf/refined-github.js:3961:34)

How to replicate the issue
Error appears, if developers tools are enabled and a mobile device is selected.
Extension version
22.7.20
Browser(s) used
Chrome 103

Tracked in https://github.com/refined-github/refined-github/issues/5895

authors: fregante jkroepke

repo: refined-github/refined-github


text:
define length of database columns

The database tables are created automatically from Java sources by Hibernate using default column lengths. The column lengths should be defined to sensible values.

We don't need this anymore because of the switch to Neo4j.

authors: rzpio thombergs

repo: reflectoring/coderadar


text:
[REF-3448] 404 not rendering

Describe the bug
entering a random url give a 404 with text in the page
going to localhost:3000/404 updates the title but nothing is rendered in the page. Making a custom 404 has the same issue where no components are rendered in the page
To Reproduce
Steps to reproduce the behavior:

enter URL as above

Expected behavior
correct 404 page be returned at localhost:3000/404
Screenshots
If applicable, add screenshots to help explain your problem.
Specifics (please complete the following information):

Python Version:
Reflex Version:
OS:
Browser (Optional):

Additional context
Add any other context about the problem here.
REF-3448

I'm able to reproduce the issue, even with our main website https://reflex.dev/404/ renders as a blank page.
Not sure what the problem is.
Is there a use case? Or are you just wondering for consistency sake why /404 doesn't render? (I'm also curious)

If only for consistency we should figure out what happens and fix it I think.

No probs. My two use cases are:

Custom 404. You cant make a custom 404 now as no components will be rendered
I am using chakra.box as an iframe so i don't see the title change. All i can see in the iframe is blank space and i have no idea what is going on.

Custom 404

does it not work when you use app.add_custom_404_page? reflex.dev uses custom 404 and it seems to work fine (although accessing it directly via /404 does not work)

Custom or built in 404s do not render at /404. I am trying to handle and return /404 under certain cases that i check for. Automatically returning 404 on random routes doesnt help my use case.

Problems with dynamic routes return the broken /404 automatically as well.

@charlestondance can you share a small example snippet ? Just want to test if its fixed with the mentioned PR !

"""Welcome to Reflex! This file outlines the steps to create a basic app."""

import reflex as rx

from rxconfig import config

class State(rx.State):
    """The app state."""

    @rx.var
    def test_id(self) -> str:
        return self.router.page.params.get("id", "No Test ID")

@rx.page(route="/test/[id]")
def test():
    """A page that updates based on the route."""
    return rx.container(
        rx.heading(f"Testing ID: {State.test_id}", size="9"),
        spacing="5",
        align="center",
        justify="center",
        min_height="85vh",
    )

app = rx.App()
app.add_custom_404_page(
    rx.container(
        rx.color_mode.button(position="top-right"),
        rx.vstack(
            rx.heading("Oops! Page Not Found", size="9"),
            rx.text(
                "It looks like the page you're looking for doesn't exist. You can start exploring by editing ",
                rx.code(f"{config.app_name}/{config.app_name}.py"),
                size="5",
            ),
            rx.link(
                rx.button("Visit our Documentation"),
                href="https://reflex.dev/docs/getting-started/introduction/",
                is_external=True,
            ),
            spacing="5",
            justify="center",
            min_height="85vh",
        ),
        rx.logo(),
    )
)

I have verified my code using an example, as shown in the screenshots after the updated code mentioned in my PR.

@charlestondance Can you confirm once if this is the expected outcome !

This should be resolved in the latest code (please re-open if needed!)

authors: Lendemor Manas1820 charlestondance masenf picklelo

repo: reflex-dev/reflex


text:
feat: Explorer panel design for multiple projects

This PR updated the UI to the latest design with menu triggers for the project and files.

As part of this change can we create a top-level folder called "documents", put it above exports and uploads, and put the documents under there?

Or if that breaks the data model, we should at least have the documents above the exports and uploads, I think.

That change of having a top-level documents folder is a bit complex in terms of data model and UI logic to make it expanded (when opening a project, creating a new file, ...).
But showing the files before folders (i.e. above exports and uploads) should be very simple. We will do that in a follow-up PR.

authors: cguedes hammer

repo: refstudio/refstudio


text:
error in insert

hello
when i insert SnappingSliderDelegate error  Use of undeclared type 'SnappingSliderDelegate'
and when that  import SnappingSlider error  :  No such module 'SnappingSlider'

Are you able to share more information on how you've added the files to your project?

authors: J-Arji rehatkathuria

repo: rehatkathuria/SnappingSlider


text:
🛑 BINUSMAYA (Academic Services) is down

In 87d3a85, BINUSMAYA (Academic Services) (https://binusmaya.binus.ac.id/LoginAD.php) was down:

HTTP code: 0
Response time: 0 ms

Resolved: BINUSMAYA (Academic Services) is back up in 091ad03.


text:
🛑 GreatNusa.com is down

In 15a64b5, GreatNusa.com (https://greatnusa.com) was down:

HTTP code: 403
Response time: 138 ms

Resolved: GreatNusa.com is back up in 78a1abc.


text:
Error java.lang.IncompatibleClassChangeError: Found interface com.google.android.gms.location.FusedLocationProviderClient

I just found an error like this when running my App
D/AndroidRuntime(16579): Shutting down VM
E/AndroidRuntime(16579): FATAL EXCEPTION: main
E/AndroidRuntime(16579): java.lang.IncompatibleClassChangeError: Found interface com.google.android.gms.location.FusedLocationProviderClient, but class was expected (declaration of 'com.google.android.gms.location.FusedLocationProviderClient' appears in /data/app/~~KvMgk1xXU1Va-RNufrd5Fg==/com.my.ientry.mypersonal.system-xXn3QY3cUGgQp0Z-wJ0ZFw==/base.apk)
E/AndroidRuntime(16579):        at com.almoullim.background_location.LocationUpdatesService.getLastLocation(LocationUpdatesService.kt:188)
E/AndroidRuntime(16579):        at com.almoullim.background_location.LocationUpdatesService.onCreate(LocationUpdatesService.kt:125)
E/AndroidRuntime(16579):        at android.app.ActivityThread.handleCreateService(ActivityThread.java:4648)
E/AndroidRuntime(16579):        at android.app.ActivityThread.access$1900(ActivityThread.java:275)
E/AndroidRuntime(16579):        at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2187)
E/AndroidRuntime(16579):        at android.os.Handler.dispatchMessage(Handler.java:106)
E/AndroidRuntime(16579):        at android.os.Looper.loopOnce(Looper.java:210)
E/AndroidRuntime(16579):        at android.os.Looper.loop(Looper.java:299)
E/AndroidRuntime(16579):        at android.app.ActivityThread.main(ActivityThread.java:8280)
E/AndroidRuntime(16579):        at java.lang.reflect.Method.invoke(Native Method)
E/AndroidRuntime(16579):        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:576)
E/AndroidRuntime(16579):        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1073)
W/personal.system(16579): type=1400 audit(0.0:31782): avc: denied { search } for name="mqsas" dev="mmcblk0p46" ino=1294337 scontext=u:r:untrusted_app:s0:c46,c257,c512,c768 tcontext=u:object_r:mqsas_data_file:s0 tclass=dir permissive=0 app=com.my.ientry.mypersonal.system
W/personal.system(16579): type=1400 audit(0.0:31783): avc: denied { search } for name="mqsas" dev="mmcblk0p46" ino=1294337 scontext=u:r:untrusted_app:s0:c46,c257,c512,c768 tcontext=u:object_r:mqsas_data_file:s0 tclass=dir permissive=0 app=com.my.ientry.mypersonal.system
W/personal.system(16579): type=1400 audit(0.0:31784): avc: denied { search } for name="mqsas" dev="mmcblk0p46" ino=1294337 scontext=u:r:untrusted_app:s0:c46,c257,c512,c768 tcontext=u:object_r:mqsas_data_file:s0 tclass=dir permissive=0 app=com.my.ientry.mypersonal.system
W/OOMEventManagerFK(16579): Failed to mkdir /data/mqsas/hprof/
W/personal.system(16579): type=1400 audit(0.0:31785): avc: denied { search } for name="mqsas" dev="mmcblk0p46" ino=1294337 scontext=u:r:untrusted_app:s0:c46,c257,c512,c768 tcontext=u:object_r:mqsas_data_file:s0 tclass=dir permissive=0 app=com.my.ientry.mypersonal.system
W/personal.system(16579): type=1400 audit(0.0:31786): avc: denied { getattr } for path="/data/mqsas" dev="mmcblk0p46" ino=1294337 scontext=u:r:untrusted_app:s0:c46,c257,c512,c768 tcontext=u:object_r:mqsas_data_file:s0 tclass=dir permissive=0 app=com.my.ientry.mypersonal.system
W/personal.system(16579): type=1400 audit(0.0:31787): avc: denied { search } for name="mqsas" dev="mmcblk0p46" ino=1294337 scontext=u:r:untrusted_app:s0:c46,c257,c512,c768 tcontext=u:object_r:mqsas_data_file:s0 tclass=dir permissive=0 app=com.my.ientry.mypersonal.system
I/Process (16579): Sending signal. PID: 16579 SIG: 9
Lost connection to device.

How to fix it yea?

I think this package conflicts with the geolocator package. When I comment on the geolocator in the .yml file then this app works well.


text:
Renewal dashboard frontend

What is this PR for?
Frontend renewal for the backend renewal. -> https://github.com/rekcurd/dashboard/pull/75
This PR includes

Add UI for data_server
Add UI for Project
Add UI for Project Access Right
Replace redux-form to formik
Use Yup

What type of PR is it?
Feature
What is the issue?
https://github.com/rekcurd/dashboard/issues/66
https://github.com/rekcurd/dashboard/issues/61
How should this be tested?
No frontend test available...

Codecov Report

Merging #83 into master will increase coverage by 0.12%.
The diff coverage is 88.05%.

@@            Coverage Diff             @@
##           master      #83      +/-   ##
==========================================
+ Coverage   83.57%   83.69%   +0.12%
==========================================
  Files          42       42
  Lines        2094     2128      +34
==========================================
+ Hits         1750     1781      +31
- Misses        344      347       +3

Impacted Files
Coverage Δ

rekcurd_dashboard/apis/api_project.py
95.58% <100%> (+0.5%)
:arrow_up:

rekcurd_dashboard/apis/api_service_routing.py
100% <100%> (ø)
:arrow_up:

rekcurd_dashboard/apis/api_service.py
98.73% <100%> (+0.03%)
:arrow_up:

rekcurd_dashboard/apis/kubernetes_handler.py
82.95% <50%> (-0.32%)
:arrow_down:

rekcurd_dashboard/apis/__init__.py
86.66% <50%> (+0.22%)
:arrow_up:

rekcurd_dashboard/apis/api_admin.py
54.41% <66.66%> (-0.07%)
:arrow_down:

rekcurd_dashboard/apis/api_service_deployment.py
96.4% <85.71%> (+0.3%)
:arrow_up:

rekcurd_dashboard/apis/api_model.py
97.36% <88.88%> (+0.07%)
:arrow_up:

rekcurd_dashboard/apis/api_kubernetes.py
88.63% <88.88%> (-0.6%)
:arrow_down:

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 2d034bb...946c2e3. Read the comment docs.

@yuki-mt Thank you for your review.

The current dispatchers do no use thunk, so we can remove react-thunk, I guess.

I am not familiar with redux, so could you send us the PR for this?

@keigohtr
OK! I will send a new PR to remove redux-thunk .

@yuki-mt

typo: APIRequestStatusList.failue -> failure
Now you can access to /projects/{non_existing_project_id}/data_servers , and submit. This can be confusing although this is a minor issue.

I have updated!

https://github.com/rekcurd/dashboard/pull/83/commits/6725f62ad6682e9f7353573ab6d319ec0043f23b https://github.com/rekcurd/dashboard/pull/83/commits/88b93ce5d4ab041f6dad324b3648ed9c7343c5d7 https://github.com/rekcurd/dashboard/pull/83/commits/2c38cb095a37bccabc7b365b8139f2ff43c0d1f3
https://github.com/rekcurd/dashboard/pull/83/commits/1a1853dfee037e13813bf9412b3605afe1e07721

@yuki-mt
I have updated!

OK! I will send a new PR to remove redux-thunk .

After this PR is merged, all issues will be solved! Thank you!

authors: codecov-io keigohtr yuki-mt

repo: rekcurd/dashboard


text:
Remove 'Warning Key' thrown by running the application and switching …

…to stable version of react

Thanks for the PR! I left a comment inline, so if you can address that we'll be able to merge this in.

authors: redcom wincent

repo: relayjs/relay-starter-kit


text:
GH-91: Fix java.lang.ArithmeticException in findBuilds

Closes #91.

@dwalluck  Can we add a test case please to verify this doesn't happen again ? Why does Main catch and print the IOException rather than propagating it out ?

@dwalluck Can we add a test case please to verify this doesn't happen again ? Why does Main catch and print the IOException rather than propagating it out ?

Unless this particular code occurs offline entirely, it will have to be one of these mocked online tests which takes a lot of code and requires access to the Koji server to create the test I/O.
In main(), there is nowhere left to propagate to except the JVM itself. I think at that point is should be logging the exception and exiting the JVM.
Do you want the signature to be
public static void main(String[] args) throws IOException
instead of
public static void main(String[] args)
and what are the advantages?

@bxf12315 retest this please

@dwalluck If we catch an IOException but don't handle it (i.e. by either propagating it or calling System.exit(1)) then the program has an incorrect return code?

@dwalluck Re testing this .... something hacked together like
+
 +    @Test
 +    public void verifyArithmeticException() throws Exception {
 +        final String checksum = "2e7e85f0ee97afde716231a6c792492a";
 +        MockKojiClientSession session = new MockKojiClientSession("skip-import-test");
 +        BuildConfig config = new BuildConfig();
 +        BuildFinder finder = new BuildFinder(session, config);
 +        Map<String, Collection<String>> checksumTable = new HashMap<>();
 +        checksumTable.put(checksum, new HashSet<>());
 +        finder.findBuilds(checksumTable);
 +    }

would verify that ArithmeticException is not thrown ?

@dwalluck Re testing this .... something hacked together like

This is already added in https://github.com/release-engineering/koji-build-finder/pull/92/commits/ec16448395b02a530539d43b958d8ce177eafd7a. It simulates a call to a non-existing checkum "abc" which returns an "empty" response. This causes the builds to be empty which triggers the exception in the original code.

@dwalluck If we catch an IOException but don't handle it (i.e. by either propagating it or calling System.exit(1)) then the program has an incorrect return code?

How about adding System.exit(1) after https://github.com/release-engineering/koji-build-finder/blob/ab9034167e3d46447ad276d8d84c5d7978d20988/src/main/java/com/redhat/red/build/finder/Main.java#L364-L365
The usage() method https://github.com/release-engineering/koji-build-finder/blob/ab9034167e3d46447ad276d8d84c5d7978d20988/src/main/java/com/redhat/red/build/finder/Main.java#L366-L369
already calls System.exit(1)
https://github.com/release-engineering/koji-build-finder/blob/ab9034167e3d46447ad276d8d84c5d7978d20988/src/main/java/com/redhat/red/build/finder/Main.java#L80-L82
so this change would make it consistent with that.
This should be a separate pull request, I think, though.
In the next micro release, we can improve the handling in BuildFinder which is just logging and continuing on Koji client errors right now, but should probably exit on the first error instead.

@rnc Given the above, is this ready to be merged?

Issue #98 filed concerning exceptions.

@dwalluck  +1 , especially given you have created #98.

authors: dwalluck rnc

repo: release-engineering/koji-build-finder


text:
Update peerDependencies to support Grunt 1.0

Hello,
This is an automated issue request to update the peerDependencies for your Grunt plugin.
We ask you to merge this and publish a new release on npm to get your plugin working in Grunt 1.0!
Read more here: http://gruntjs.com/blog/2016-02-11-grunt-1.0.0-rc1-released#peer-dependencies
Also on Twitter: https://twitter.com/gruntjs/status/700819604155707392
If you have any questions or you no longer have time to maintain this plugin, then please let us know in this thread.
Thanks for creating and working on this plugin!
(P.S. Close this PR if it is a mistake, sorry)

+Merge

authors: Rakhmanov gruntjs-updater


text:
Explicitly import React

Otherwise TSLint doesn't know what React.Component comes from, etc.

Was actually vim issue. Sorry.

repo: reminisceme/app


text:
🛑 Remix Dashboard is down

In 17b34c0, Remix Dashboard (https://remix.fairuse.org) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Remix Dashboard is back up in 344fbaa after 43 minutes.

repo: remix-bot/status


text:
Error: Invariant failed - Cypress Build Fails -

Have you experienced this bug with the latest version of the template?
First time using this template
Steps to Reproduce

Forked repo.
Followed egghead.io tutorial.
Pushed to github.
Added SESSION_SECRET and DATABASE_URL  to Fly.io and Github.

Works locally but building fails in Cypress on invariant:
node_modules/tiny-invariant/dist/tiny-invariant.cjs.js:10
        throw new Error(prefix);
              ^
Error: Invariant failed
    at invariant (/home/runner/work/remixed/remixed/node_modules/tiny-invariant/dist/tiny-invariant.cjs.js:10:15)
    at getEnv (/home/runner/work/remixed/remixed/build/index.js:42:44)
   //...etc

Expected Behavior
That it builds and works
Actual Behavior
Fails on Cypress build in github.
> start:mocks
> binode --require ./mocks -- @remix-run/serve:remix-serve build binode ./mocks ./node_modules/dotenv/config @remix-run/serve:remix-serve build

🔶 Mock server running

/home/runner/work/remixed/remixed/node_modules/tiny-invariant/dist/tiny-invariant.cjs.js:10
        throw new Error(prefix);
              ^
Error: Invariant failed
    at invariant (/home/runner/work/remixed/remixed/node_modules/tiny-invariant/dist/tiny-invariant.cjs.js:10:15)
    at getEnv (/home/runner/work/remixed/remixed/build/index.js:42:44)
    at Object.<anonymous> (/home/runner/work/remixed/remixed/build/index.js:[49](https://github.com/Andrehatlo/remixed/actions/runs/3696328499/jobs/6259880586#step:9:50):14)
    at Module._compile (node:internal/modules/cjs/loader:11[55](https://github.com/Andrehatlo/remixed/actions/runs/3696328499/jobs/6259880586#step:9:56):14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1209:10)
    at Module.load (node:internal/modules/cjs/loader:1033:32)
    at Function.Module._load (node:internal/modules/cjs/loader:868:12)
    at Module.require (node:internal/modules/cjs/loader:10[57](https://github.com/Andrehatlo/remixed/actions/runs/3696328499/jobs/6259880586#step:9:58):19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Object.<anonymous> (/home/runner/work/remixed/remixed/node_modules/@remix-run/serve/dist/cli.js:42:13)
http://localhost:8811 timed out on retry 91 of 3, elapsed 90307ms, limit 90000ms
Error: connect ECONNREFUSED 127.0.0.1:8811

I have checked out the other closed issues regarding `` this issue and have edited the deploy.yml file with the following:
github/workflows/deploy.yml
- name: 🌳 Cypress run
        uses: cypress-io/github-action@v5
        with:
          start: npm run start:mocks binode --require ./mocks --require ./node_modules/dotenv/config -- @remix-run/serve:remix-serve build
          wait-on: "http://localhost:8811"
        env:
          PORT: "8811"

Hey @Andrehatlo, if you don't mind commenting back with how you fixed this issue, this will be very helpful for future travelers 🙏

Indeed, I'm having the same exact issue
Run cypress-io/github-action@v5
start server "npm run start:mocks command "npm run start:mocks"
current working directory "/home/runner/work/remix-project-lab/remix-project-lab"
waiting on "http://localhost:8811" with timeout of 60 seconds
/opt/hostedtoolcache/node/16.18.1/x64/bin/npm run start:mocks

> start:mocks
> binode --require ./mocks -- @remix-run/serve:remix-serve build

🔶 Mock server running

/home/runner/work/remix-project-lab/remix-project-lab/node_modules/tiny-invariant/dist/tiny-invariant.cjs.js:10
        throw new Error(prefix);
              ^
Error: Invariant failed
    at invariant (/home/runner/work/remix-project-lab/remix-project-lab/node_modules/tiny-invariant/dist/tiny-invariant.cjs.js:10:15)
    at Object.<anonymous> (/home/runner/work/remix-project-lab/remix-project-lab/build/index.js:155:35)
    at Module._compile (node:internal/modules/cjs/loader:1155:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1209:10)
    at Module.load (node:internal/modules/cjs/loader:1033:32)
    at Function.Module._load (node:internal/modules/cjs/loader:868:12)
    at Module.require (node:internal/modules/cjs/loader:1057:19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Object.<anonymous> (/home/runner/work/remix-project-lab/remix-project-lab/node_modules/@remix-run/serve/dist/cli.js:42:13)
    at Module._compile (node:internal/modules/cjs/loader:1155:14)
http://localhost:8811 timed out on retry 91 of 3, elapsed 90236ms, limit 90000ms
Error: connect ECONNREFUSED 127.0.0.1:8811

Locally npm run test:e2e:run and npm run test:e2e:dev run fine. Also npm run start:mocks and it changes port from 3000 to 8811 when setting the env variable locally.

Found other similar issues like https://github.com/alexreardon/tiny-invariant/issues/146 and https://github.com/remix-run/grunge-stack/issues/71, and the most likely culprit mentioned was missing environment variables. And indeed, that was my case as well. Make sure you are defining all the same variables as in your local environment, you can also change the CI command to use npm run dev instead of npm run start:mocks or other production like commands, because run dev gives you better error messages.
In my case I was missing DATABASE_URL, SESSION_SECRET, and other project specific env variables.

authors: Andrehatlo jackbravo machour

repo: remix-run/indie-stack


text:
Framer motion's full page transitions breaks with remix while using loader in route file

What version of Remix are you using?
1.6.3
Steps to Reproduce

Make simple remix app with framer-motion
Set up page transtions in root
make route that uses loader data,

Expected Behavior
I expected animation to be compled (exiting from page with loader data).
Actual Behavior
While exiting from page with loader data, it gives an error TypeError: Cannot destructure property 'data' of 'useLoaderData(...)' as it is undefined.. I noticed that loader data delets first off, not allowing animation to complete, so error shows.
All other routes with no loader works completly fine!
made clean and simple new remix app with framer motion to show

Could you provide a sample repository reproducing this issue? This would help the team getting on track quickly to see if it can be fixed

I made one with loader bug. U can try yourself - change
<AnimatePresence exitBeforeEnter>      <motion.main key={location.key}>         {outlet}      </motion.main> </AnimatePresence>
to
<AnimatePresence exitBeforeEnter>             {outlet} </AnimatePresence>
to remove animations and It'll work fine

Idk, how to name this, but data from loader unmounts before animation is compled, so then data is undefinied, maybe this helps :)

I was having a similar problem after following the https://github.com/remix-run/remix/tree/main/examples/framer-route-animation example. What I figured out was that with framer motion, the route component that is exiting renders 1 extra time after changing to the next route. If you log useLocation in the route that's exiting you'll see that.

So, what seems to be happening is that the component is not in the React tree (isPresent is from the useIsPresent()  hook provided  by Framer Motion) but it is still executing and trying to access useLoaderData() but the routes don't match? I'm a bit over my knowledge here so I'm guessing.
The way I fixed it for now was moving the <motion.main key=useLocation().key>  to each of the route files I want animated instead of the root route:
root.tsx:
(...)
 <AnimatePresence exitBeforeEnter={true}>
      {outlet}
 </AnimatePresence>
(...)

index.tsx:
<motion.main
      key={location.key}
      initial={{ opacity: 0 }}
      animate={{ opacity: 1 }}
      exit={{ opacity: 0 }}
>
(...)
<motion.main/>

Thanks for helping @jamesilva, but I did it like you said (moved motion.div to route files), but now there are not exit animation :( Did it on my test repo

@jamesilva if you feel that the two examples in the repository can be enhanced, don't hesitate to open pull requests 🙏🏼

i have same isusse so i check up example project but same not working exit
https://codesandbox.io/s/github/remix-run/examples/tree/main/framer-route-animation
how to solve this problem ????

Closing this out because this is not a bug in Remix, which will update stat as you navigate.  You will need to capture that into local state to preserve during the animation - as shown in https://github.com/remix-run/remix/issues/3625#issuecomment-1179178857.

authors: ImExoOdeex brophdawg11 jamesilva machour usback

repo: remix-run/remix


text:
x-remix-redirect responses are not ignored when route has changed

What version of Remix are you using?
1.18.0
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
https://stackblitz.com/edit/remix-run-remix-yjcpcy?file=app%2Froutes%2F_index.tsx
Note that this is not reproducible on stackblitz, as the loader response times seem to be too fast. Download and run locally with Fast3G network speed enabled

Download the reproduction remix app in the link
Load /
Set network speed to Fast3G
click the link that says "link to e/123/456"
After the /e/123/456 route loads, click the "Action" button, then quickly click the link below it that says "home"
💥

If you hit the error you should see something like this:

Expected Behavior
The core of the issue seems to be around timing of POST requests that respond with an x-remix-redirect header.

If the POST request finishes,
and simultaneously the user navigates to a new route,
the POST request cannot be canceled (it's finished).
Remix / Router cannot cancel it, so they respect the x-remix-redirect header, even though the url and route have changed.
Remix dutifully triggers the loaders for the redirected route
One of the routes (e.tsx in my repro) has a shouldRevalidate function which returns false
This causes undefined to be returned and 💥

I'd expect the finished POST request's response which includes the x-remix-redirect header to be ignored.
Actual Behavior
What actually happens is the POST request's x-remix-redirect header is not ignored.

@brophdawg11 I was able to create a reproduction case for the issue I mentioned here: https://github.com/remix-run/react-router/issues/10473#issuecomment-1601996335
Stackblitz seems to be running the loaders in something like a wasm nodejs runtime so adjusting the network speed doesn't have any effect and the repro is not possible in their env. It is trivial to reproduce locally on: 1.17.0, 1.17.1, 1.18.0 and remix/dev branch

🙌 Thank you for the reproduction - that was super helpful in identifying the underlying issue!  This should be fixed by https://github.com/remix-run/react-router/pull/10674 and available in the next Remix release

@brophdawg11 thank you! This was a huge reliability issue for us. Having it resolved will be a major relief.


text:
[Bug] Path parameter inside url not working as expected

Reproduction
See this reproduction repo: https://github.com/AndrewOt/remix-bug-repro

Clone, install, and start the example site with npm run dev
Navigate to /my-page/123 (or any number or "theId")
Notice the form on the page. Input some text and click submit
Notice that the url changes to /my-page/123/success but the content on the page does not change.

Alternatively, you can click the "Link test" button an show that the same occurs.
If you look at the code for the route /my-page/123, you can see that the action handling the form POST request returns a redirect to the success url. The url in fact changes but the content of the page does not. Notice also that this is defining the route with the folder name (with the route.tsx in the directory); this behavior also occurs with the typescript file itself containing the route name.
Behavior is reproducible on both chromium based browsers and firefox.
System Info
System:
    OS: Windows 10 10.0.19045
    CPU: (16) x64 AMD Ryzen 7 2700X Eight-Core Processor
    Memory: 4.41 GB / 15.93 GB
  Binaries:
    Node: 18.8.0 - C:\Program Files\nodejs\node.EXE
    npm: 8.19.1 - C:\Program Files\nodejs\npm.CMD
    pnpm: 8.6.12 - C:\Program Files\nodejs\pnpm.CMD
  Browsers:
    Chrome: 119.0.6045.200
    Edge: Spartan (44.19041.3636.0), Chromium (119.0.2151.97)
    Internet Explorer: 11.0.19041.3636
  npmPackages:
    @remix-run/css-bundle: ^2.3.1 => 2.3.1
    @remix-run/dev: ^2.3.1 => 2.3.1
    @remix-run/eslint-config: ^2.3.1 => 2.3.1
    @remix-run/node: ^2.3.1 => 2.3.1
    @remix-run/react: ^2.3.1 => 2.3.1
    @remix-run/serve: ^2.3.1 => 2.3.1

Used Package Manager
npm
Expected Behavior
The action should properly redirect the user to the new page (i.e. the url should change and the content of the page should change).
Actual Behavior
Only the url changes and the content remains the same.

This is expected behavior with your current example. I believe you want to change your my-page.$theId folder to be my-page.$theId._index and then you'll get the behavior you're expecting.
Right now when you navigate to your success page, (whether via redirect in an action, link or just typing the URL. Remix thinks that your route my-page.$theId is a layout route and is expecting there to be an <Outlet /> in order to render the success page.
I believe you don't want the my-page.$theId to be a layout route, and just a normal page, so renaming the folder to my-page.$theId._index will fix that.
Hope this helps

Ah ok, so I'm just a noob. I will do that! Thank you!


text:
fix(dev): skip CSS build on missing css-bundle dep

Since you can only access the output of the CSS bundle build via the @remix-run/css-bundle package, we can detect the lack of this package and use this to avoid running an esbuild pass for no reason.
To make this easier to test (since our monorepo unavoidably has this package available), and also to give consumers more explicit control over this behaviour, I've added a cssBundle option to the Remix config, but I'm leaving it undocumented for now.

Going to attempt a delayed initialization of the css bundle based on the conditional include of @remix-run/css-bundle.

Closing for now since we're going to try a different approach.

authors: jacob-ebey markdalgleish


text:
Support an optional Layout export from the root route

RFC https://github.com/remix-run/remix/discussions/8702
Closes https://github.com/remix-run/remix/discussions/8703

Does this require the Vite dev server?
In my usage so far it works fine with remix vite:dev, but with remix dev the hydrated document is not wrapped with the Layout component.
I'm not sure if you want a bug report, or if you would rather just document this limitation. It probably depends on how aggressively you want to drop non-Vite support.

I currently use the <html className /> for setting the user preferred dark class. It should also match the rendered state, so it is synchronized between default browser preference and server with a cookie. Even setting this in <body className /> still needs additional context to be rendered from the Layout component.
I can't set it at any lower of a level than body and still have headless-portal-root elements pick up the dark class.
Is there a possibility to pass more than just children to the Layout component?

Is there a possibility to pass more than just children to the Layout component?

No, Layout is the top-most component so there's nowhere to determine prop values and pass them through.  Layout is really just syntactic sugar for the simple case where the app shell is shared between ahppy and error-path UIs.  If you need more complex differences than you may be better off not using Layout and using your own component that accepts props.  However, chances are you're going to be determining those prop values from useLoaderData and useRouteError and could just as easily grab that info from useRouteLoaderData("root") and useRouteError in Layout

grab that info from useRouteLoaderData("root") and useRouteError in Layout

Thanks, useRouteLoaderData("root") was the missing link for me. It gave me the added benefit of having the root ErrorBoundary have more of the configuration from the root loader (if the root loader isn't what crashed).

authors: aaronadamsCA brophdawg11 ngbrown


text:
Forgotten class InetAddress in the description of enum.

There was forgotten class InetAddress in the description of enums in autocomplete documentation.

Codecov Report

Merging #177 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #177   +/-   ##
=========================================
  Coverage     91.11%   91.11%
  Complexity      100      100
=========================================
  Files             2        2
  Lines          1609     1609
  Branches        404      404
=========================================
  Hits           1466     1466
  Misses           46       46
  Partials         97       97

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 6dcfa5f...804b1d4. Read the comment docs.

Oops! Good catch! Thank you very much!

authors: FilipJirsak codecov-io remkop

repo: remkop/picocli


text:
docs: Add docs (build)

This library was part of @remotecom internal source code.
As we are open-sourcing it, this is the docs (build version).
The library code itself is copied at #1 .
Why not open-sourcing the docs' source?
Although the library code is open-sourced with unit tests, the docs cannot be moved yet. Our docs are still coupled to Remote's internal Design System and integration tests. The effort to decouple it at the moment is too high.
So our strategy is to at each new PR, to update the docs internally and manually copy its build to this repo.
How to review this

Check the website where the docs are hosted (Vercel) is working well

Docs site looks great to me 👍 Clicked around and played with several pieces. Love that it will show the updates at the bottom of the PR for testing.

Cool! Which "updates at the bottom" are you talking about?

Docs site looks great to me 👍 Clicked around and played with several pieces. Love that it will show the updates at the bottom of the PR for testing.

Cool! Which "updates at the bottom" are you talking about?

repo: remoteoss/json-schema-form


text:
:tada: NEW: CSR admin ui (entities metadata)

After this PR, you can "reload entities" and it will use the session bearer.

Local storage key for Auth will also be back to work, as request will happen on client side.

To add tomorrow : don't return anything when the admin flag is not allowed!

repo: remult/remult


text:
cocoapod for mac

pls update cocoapod for osx
Thank

Same bug as #21

Same bug as #20 not #21

repo: renebigot/XlsxReaderWriter


text:
Impossible to flush cache without tags

For me it seems that flushQueryCache() doesn't work without tags at all: https://github.com/rennokki/laravel-eloquent-query-cache/blob/master/src/Traits/QueryCacheModule.php#L152
Am I right? There doesn't seem to be easy way to empty whole cache per model?
Originally posted by @apeisa in https://github.com/rennokki/laravel-eloquent-query-cache/issues/12#issuecomment-612664909

You can set up an additional tag on each query and flush it by that tag:
Post::cacheFor(60)
    ->cacheTags(['query', 'posts'])
    ->get();

Category::cacheFor(60)
    ->cacheTags(['query', 'categories'])
    ->get();

Post::flushQueryCache(['query']); // flushes both

Query Cache relies on the tags to invalidate the cache, making a distinction between the cached queries and your normal cache; otherwise you can call cache:clear to invalidate all the cache.

Check this tag: https://github.com/rennokki/laravel-eloquent-query-cache/releases/tag/1.4.0

Thanks, looks great! Does that implementation mean that this doesn't work at all with cache drivers that doesn't support tags?

You have to "mark" each query. It would be a pain to invalidate queries giving the query string.

authors: apeisa rennokki

repo: rennokki/laravel-eloquent-query-cache


text:
🛑 Intranet is down

In 008ca44, Intranet (https://oceba.gba.gov.ar/intranet) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Intranet is back up in 3cb4bc9.

repo: renzoalis/upptime


text:
Make updating compatibility a semi-manual process, or just add new versions

Something I concluded last time I was deep in the weeds of the compatibility matrices is that we probably want this to be a semi-manual process.
There are a few problems with the current completely automatic approach:

There are a few entries that are incorrect on PyTorch/Tensorflow's website.
Some are just missing.
PyTorch and Tensorflow's websites keep on changing and we keep on having to update the regexes and stuff, but the entries for previous versions never change so there's no need to keep on editing past versions.

This all implies that we might want a semi-manual process for updating compatibility matrices instead of rewriting the whole file, or maybe it just adds new versions and leaves old ones alone.

Oh this is a dupe: https://github.com/replicate/cog/issues/820

repo: replicate/cog


text:
Llama 2 outputs disjointed text

Hi @michalkubizna. Can you share some more information to help figure out what's happening? Which model / version of LLaMA are you using. Can you share the code that produced this? Can you share the ID of the prediction on Replicate?

same here. here is the sample code:
import replicate
api_token = "my token"
client = replicate.Client(api_token=api_token)
while True:
prompt = input("Enter your prompt (or 'exit' to quit): ")
if prompt == "exit":
break
output = client.run(
    "replicate/llama-2-70b-chat:2c1608e18606fad2812020dc541930f2d0495ce32eee50074220b87300bc16e1",
    input={"prompt": prompt}
)
for item in output:
print(item)

@monshizadeh Thanks. Can you please share the prediction ID so that I can see the raw output?

@monshizadeh @michalkubizna LLaMA and other LLMs generate models by token. You can get a better idea about how words generally split into tokens in this interactive demo.
The output of running LLaMA is an iterator or list of tokens. The reason why you're getting the results you are is that the Python print statement appends a newline by default. You can disable that by passing end="":
for token in output:
    print(token, end="")

The problem is that it divides the individual words. For instance (refer to the attached photo), instead of printing "factually", it prints "fact" and "ually" separately. Similarly, for "Coherent", it displays "coh", "er", and "ent" as distinct segments. Please check the attached photo for reference.

@michalkubizna Yes, this is expected. LLaMA outputs by token, not word. It's hard to tell from a screenshot of text rather than text itself, but notice that "ually" doesn't have a leading space, like " fact", so if you were join with an empty string or print without a newline, everything would work as expected.

Thanks. That makes sense.

authors: mattt michaecube michalkubizna monshizadeh

repo: replicate/replicate-python


text:
Use XDG variables

Today HelmVM stores its data under $HOME/.helmvm (.helmvm is replaced by the binary name). We need to start taking into account the XDG configuration instead.

Moved to Shortcut.

repo: replicatedhq/helmvm


text:
Support v2 async controllers for posting launch/test/log

Hidden stones:

Finish test request requires specifying of LaunchUuid
...

Still need to figure out why tests for async logs are failing.

Still need to figure out why tests for async logs are failing.

repo: reportportal/client-net