Skip to content

Improved imports. - #144

Open
KsaR99 wants to merge 2 commits into
brufdev:devfrom
KsaR99:dev
Open

Improved imports.#144
KsaR99 wants to merge 2 commits into
brufdev:devfrom
KsaR99:dev

Conversation

@KsaR99

@KsaR99 KsaR99 commented Sep 3, 2026

Copy link
Copy Markdown

Hello, I can't test if it works, but I tried... (after like 100's of ssh commands, fixing disabled php extensions on my side: sqlite3, pdo_sqlite, sockets, intl, zip - were not mentioned in contributing :D), 1 test failing on windows thus I changed "/$file" to $file etc etc, still can't figure out, how to properly setup dev. (ended up with some reverb? errors). or in console app-CofSMmhW.js:90 You must pass your app key when you instantiate Pusher.. Hours & hours of debugging.

But anyway, I have refactored some of the import(s) to use index.ts, seems cleaner/shorter, and I just hope it works. @brufdev. 🙏

@@ -1,5 +1,4 @@
import { update } from '@/routes/vaults';
import { children } from '@/routes/vaults/nodes';

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

btw, I haven't done this so must've been some from composer* commands?

$attributes['name'] = $pathInfo['filename'];
$attributes['extension'] = $pathInfo['extension'] ?? 'md';
$attributes['content'] = (string) file_get_contents("/$file");
$attributes['content'] = (string) file_get_contents($file);

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Windows failed on test without it

$replacementLinkPath = $node->ancestorsAndSelf()->get()->last()->full_path;
}
/** @var string $replacementLinkPath */
$replacementLinkPath ??= $node->ancestorsAndSelf()->get()->last()->full_path;

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rector suggested this change.

@brufdev

brufdev commented Sep 3, 2026

Copy link
Copy Markdown
Owner

Cześć my polish friend! Hope you're doing well.

I have to document the dev environment for sure. So many things to do and so little free time.

I'm on vacations. I will try to review this next week ok?

Thank you!

@KsaR99

KsaR99 commented Sep 3, 2026

Copy link
Copy Markdown
Author

Cześć my polish friend! Hope you're doing well.

Hola :D yes, thank you.

I have to document the dev environment for sure. So many things to do and so little free time.

Would be helpful, I'm not too much into laravel etc, so I don't know how to test it properly :D (there's some missing parts, from REVERB_* in .ini.example and probably some other too, so I can't build it. :\

I'm on vacations. I will try to review this next week ok?

No problem. take your time & enjoy vacation! 🥂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants