ui-predicate icon indicating copy to clipboard operation
ui-predicate copied to clipboard

Vue 3 support

Open andonov85 opened this issue 4 years ago • 27 comments

Hello, François-Guillaume Ribreau and thank you for the great library!

We use the library in a project that we plan to migrate to Vue 3 and the question is whether support for Vue 3 is planned in the near future?

Thanks!

andonov85 avatar Oct 27 '21 18:10 andonov85

@FGRibreau What is complexity / level of work required here? Is this something a junior developer (in my team) could help with?

Thanks!

marclaporte avatar Nov 06 '22 16:11 marclaporte

Hello @marclaporte,

Since there is already an exemple facade for vue2, I think creating a facade for vue3 should not be a big deal yes :)

I will be able to guide him once a PR is open :)

FGRibreau avatar Nov 06 '22 16:11 FGRibreau

@johnsantosDev Show us what you got :-)

marclaporte avatar Nov 06 '22 18:11 marclaporte

Thanks @marclaporte, on it!

johnsantosDev avatar Nov 06 '22 19:11 johnsantosDev

Change of plan (sorry)

@alvinBM is now in charge.

marclaporte avatar Jun 23 '23 03:06 marclaporte

Closed as done! https://github.com/FGRibreau/ui-predicate/pull/192

Congrats @alvinBM and thank you to @FGRibreau

marclaporte avatar Aug 12 '23 17:08 marclaporte

@alvinBM @marclaporte @johnsantosDev Hello! Just a little update, I've indeed merged the PR but lots of things are not working (storybooks, tests, docs) on the vuejs3 package, I've started a refactor to make these parts rights and then I will create a new release.

Thanks a lot for your work!

FGRibreau avatar Aug 14 '23 09:08 FGRibreau

@FGRibreau I am sending you positive thoughts on this topic :-)

marclaporte avatar Feb 22 '24 04:02 marclaporte

@marclaporte I'm deeply sorry Marc, I don't have the time right now to work on fixing all the missing things in this PR to make it mergeable in ui-predicate, maybe you have available developer bandwidth to do it (fix storybooks, tests & docs generation)?

(I can guide one of your dev asynchronously though!)

FGRibreau avatar Feb 26 '24 06:02 FGRibreau

@marclaporte I'm deeply sorry Marc, I don't have the time right now to work on fixing all the missing things in this PR to make it mergeable in ui-predicate, maybe you have available developer bandwidth to do it (fix storybooks, tests & docs generation)?

(I can guide one of your dev asynchronously though!)

@FGRibreau , I am available to fix the various blocking points, so I'll always come back to you if necessary to move forward more quickly. THANKS

alvinBM avatar Feb 27 '24 01:02 alvinBM

@FGRibreau Please re-open this task and assign to @alvinBM

marclaporte avatar Mar 15 '24 13:03 marclaporte

done @marclaporte !

FGRibreau avatar Mar 15 '24 13:03 FGRibreau

Hello @FGRibreau,

I am in the process of fixing the problems related to the doc and the storybooks, but I am currently facing a lot of bugs and perhaps incompatibility especially with the storybooks.

I built the ui-predicate-vue3 package with vite 4.3.2 and vue 3.2.13, so I got a lot of errors related to parcelRequire used in ui-predicate-core.

I wonder if the current ui-predicate-core package is compatible with vite? Maybe you can have additional information that can help me move forward faster.

NB: For storybooks I use version 8.0.0

alvinBM avatar Mar 18 '24 15:03 alvinBM

I wonder if the current ui-predicate-core package is compatible with vite? Maybe you can have additional information that can help me move forward faster.

It might be indeed, it's a vanilla js library (using commonjs) that needs to be compiled for the browser so "any" builder will do :)

You can — of course — please update the current dependencies, I will release ui-predicate-core to a new major if I detect an interface change :)

On Mon, Mar 18, 2024 at 4:36 PM Alvin BAUMA @.***> wrote:

Hello @FGRibreau https://github.com/FGRibreau,

I am in the process of fixing the problems related to the doc and the storybooks, but I am currently facing a lot of bugs and perhaps incompatibility especially with the storybooks.

I built the ui-predicate-vue3 package with vite 4.3.2 and vue 3.2.13, so I got a lot of errors related to parcelRequire used in ui-predicate-core.

I wonder if the current ui-predicate-core package is compatible with vite? Maybe you can have additional information that can help me move forward faster.

NB: For storybooks I use version 8.0.0

— Reply to this email directly, view it on GitHub https://github.com/FGRibreau/ui-predicate/issues/152#issuecomment-2004238113, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABBWQWTCFMWH35JTMSSZK3YY4CXNAVCNFSM5G2643Q2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMBQGQZDGOBRGEZQ . You are receiving this because you were mentioned.Message ID: @.***>

FGRibreau avatar Mar 18 '24 15:03 FGRibreau

Hello @FGRibreau,

Here is the PR that fixes the points mentioned below (fixes some bugs, updates Vue3 docs, and adds related Storybooks) https://github.com/FGRibreau/ui-predicate/pull/204

Yannick243 avatar May 08 '24 18:05 Yannick243

We can close via https://github.com/FGRibreau/ui-predicate/pull/204 and wait for a new version! 2.0.0?

marclaporte avatar May 17 '24 20:05 marclaporte

@FGRibreau How can we help for the next step? Thanks!

marclaporte avatar Aug 03 '24 14:08 marclaporte

@marclaporte sorry for the delay, I'm under heavy workload currently, the 2.0 release is on my todo-list be sure I will complete it in the next weeks :)

FGRibreau avatar Aug 31 '24 16:08 FGRibreau

@FGRibreau Is there anything we can do to help?

marclaporte avatar Jan 26 '25 13:01 marclaporte

@marclaporte I'm really sorry, I've cofounded multiple new product in the past months and can't find time to go back to ui-predicate. Would you like to be a co-maintainer ?

FGRibreau avatar Jan 29 '25 08:01 FGRibreau

Yes, with my colleagues @alvinBM @Yannick243 @jonnybradley and @kroky

marclaporte avatar Jan 29 '25 17:01 marclaporte

Ok, I invited you for now, let's see how it goes first :)

FGRibreau avatar Jan 29 '25 21:01 FGRibreau

hehe, OK but I focus on my role as a CEO so I will encourage one of our devs to prepare what is necessary for a release.

Here is where/how we are using: https://gitlab.com/tikiwiki/tiki/-/merge_requests/6847

marclaporte avatar Jan 29 '25 22:01 marclaporte

Duly noted, invited your team then :)

FGRibreau avatar Feb 02 '25 21:02 FGRibreau

Thanks, accepted - just wish i had time to get back into Vue stuff! Soon soon... 🤞

jonnybradley avatar Feb 03 '25 12:02 jonnybradley

new PR with enhancements https://github.com/FGRibreau/ui-predicate/pull/209

Yannick243 avatar Feb 28 '25 18:02 Yannick243

@Yannick243 @FGRibreau Anything left here or we can close this?

Thanks!

marclaporte avatar Mar 19 '25 18:03 marclaporte

@FGRibreau

FYI, @Yannick243 and @andonov85 are no longer active with us.

marclaporte avatar Dec 04 '25 18:12 marclaporte