-
Notifications
You must be signed in to change notification settings - Fork 206
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
Implementgroup #959
Merged
Merged
Implementgroup #959
Changes from all commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
58bc692
h
sashadev-sky 8dcff6a
h
sashadev-sky c075966
Merge branch 'main' of https://github.com/publiclab/mapknitter into main
sashadev-sky 986b49a
Merge branch 'main' of https://github.com/publiclab/mapknitter into main
sashadev-sky 34cf99d
Merge branch 'main' of https://github.com/publiclab/mapknitter into main
sashadev-sky 11e11be
Merge branch 'main' of https://github.com/publiclab/mapknitter into main
sashadev-sky ba41fb4
Merge branch 'main' of https://github.com/publiclab/mapknitter into main
sashadev-sky d398f00
Merge branch 'main' of https://github.com/publiclab/mapknitter into main
sashadev-sky 1828645
Merge branch 'main' of https://github.com/publiclab/mapknitter into main
sashadev-sky 95759ad
Merge branch 'main' of https://github.com/publiclab/mapknitter into main
sashadev-sky 8238c6f
Merge branch 'main' of https://github.com/publiclab/mapknitter into main
sashadev-sky bf84e93
Merge branch 'main' of https://github.com/publiclab/mapknitter into main
sashadev-sky fc9d5ce
Merge branch 'main' of https://github.com/publiclab/mapknitter into main
sashadev-sky ae6a192
Merge branch 'main' of https://github.com/publiclab/mapknitter into main
sashadev-sky 596f150
updates
sashadev-sky 13db549
Fix featuregroup
sashadev-sky 19671b1
checkout randomly modified file
sashadev-sky eae111b
Implementation
sashadev-sky a24cd58
setupToolbar working
sashadev-sky a827d7c
readOnly mode working
sashadev-sky 6fd342d
remove googleMutant
sashadev-sky c3755ff
finalize
sashadev-sky 6924249
group.editing call
sashadev-sky 63fcd73
fix puma pidfile path, set it for development
sashadev-sky ba52e3d
review rebase
sashadev-sky 5155f9d
update schema.rb to fix DB setup and commands like migrate, setup, etc
sashadev-sky ba147f1
delete obsolete config file
sashadev-sky ca49d2e
update example file
sashadev-sky 0dab975
readd new line at end of file
sashadev-sky 46d1e2e
remove left behind comment
sashadev-sky 821eb7f
add new line to file
sashadev-sky 54c3a49
bump ldi to 0.7.6
sashadev-sky 79fcc5b
commit data dump
sashadev-sky 0b6fa5e
remove db dump
sashadev-sky b784b50
update prod
sashadev-sky 32aa001
return puma back to production
sashadev-sky b218335
add new migration to avoid potential corruption
sashadev-sky 9a2b1a1
delete extra whitespace
sashadev-sky 419b4b9
re-run all edited migrations
sashadev-sky d44a40b
update routes
sashadev-sky aba021d
remove defaults and set before_validations
sashadev-sky 94e8aa7
update u to self
sashadev-sky 3502a20
fix ||= space
sashadev-sky c1c0e1b
reverse order reversal
sashadev-sky bbf4f51
bump LDI to 0.7.7
sashadev-sky 2b40f59
remove warpable update hook
sashadev-sky ce0f2d2
try now
sashadev-sky 446cc25
before
sashadev-sky 974a996
modifications
sashadev-sky b21a192
update
sashadev-sky f7d890a
remove before_validation hook for map
sashadev-sky 3410ce6
update incorrect syntax in tests
sashadev-sky 73125e8
Merge branch 'main' into implementgroup
sashadev-sky File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should add forego as a dependency. With this we should start MK with
forego start
to run both Puma and Passenger.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@alaxalves can we handle that in a follow up pr? is there an issue open for it? Everything works fine with it so if its not critical id rather not add it on here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Of course! Also it could be a FTO issue. 😄 😄 😄