Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ml_pathfind and ml_jwt #3

Closed
wants to merge 4 commits into from
Closed

Add ml_pathfind and ml_jwt #3

wants to merge 4 commits into from

Conversation

StiviiK
Copy link

@StiviiK StiviiK commented Jul 25, 2019

@StiviiK StiviiK changed the title [WIP] Add ml_pathfind and ml_jwt Add ml_pathfind and ml_jwt Jul 25, 2019
@StiviiK
Copy link
Author

StiviiK commented Jul 25, 2019

Modules are for this state feature complete.

@qaisjp
Copy link

qaisjp commented Jul 26, 2019

These look like really fantastic modules, thank you very much for the pull request!

We should probably figure out what we want to do with this (mta-modules) repo. I'm not sure if this repo is actually used for anything.

I can't figure out why new modules would be added (even as submodules) to this repo instead of their own standalone repo, other than for publicity.

If publicity is what you're looking for, that's fine: I think it would be a good idea to showcase some exemplary community modules. But in that case I think adding a README.md with links to your projects would be sufficient enough.

/cc @Jusonex since you've worked on ml_pathfind?

@jushar
Copy link
Contributor

jushar commented Jul 26, 2019

I'm not sure either.
I'd probably prefer an integration for modules in Community 2.0.

The only advantage of this repository is that the modules here are compiled by us, thus trustworthy. We could achieve the same with some form of verified state within the new community also though.

@StiviiK
Copy link
Author

StiviiK commented Jul 26, 2019

A (better) integration in the Community sounds like a reasonable Idea.
Currently it is pretty hard to find modules or even know which exist.

@StiviiK StiviiK closed this by deleting the head repository Sep 23, 2022
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.

3 participants