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.

Sign up for GitHub

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

Jump to bottom

[RELEASE] Educated Eucalyptus #148

Merged
merged 52 commits into from
Apr 18, 2024
Merged

[RELEASE] Educated Eucalyptus #148

merged 52 commits into from
Apr 18, 2024

Conversation

roguenet
Copy link
Collaborator

roguenet added 30 commits April 3, 2024 10:47
@roguenet
[TM-785] Move constants to avoid a problem in PHPUnit.
4adf29c
@roguenet
[TM-785] Fix old migration that was throwing an error in PHPUnit
57900ba
@roguenet
[TM-785] Fail gracefully if SNS is not enabled.
0d5d2ca
@roguenet
Merge pull request #130 from wri/main
1afda1c 
[MERGE] main -> staging post DD release
@roguenet
[TM-785] Remove the legacy unit tests.
1c18d99
@roguenet
[TM-785] Get the linter passing.
081c5cb
@roguenet
[TM-785] Get the UpdateRequests tests passing.
22ac48d
@roguenet
[TM-785] Stages tests passing.
1b39e58
@roguenet
[TM-785] Don't let faker create random statuses; it leads to flaky te…
8a254d8 
…sts.
@roguenet
[TM-785] Avoid generating multiple forms for the same framework/entit…
446244b 
…y in a single test run.
@roguenet
[TM-785] Sites tests all passing.
9d60eb5
@roguenet
[TM-785] Projects tests all passing.
da09048
@roguenet
[TM-785] Nursery tests all passing.
be83f11
@roguenet
[TM-785] Task tests passing.
fc602d4
@roguenet
[TM-785] Report tests passing.
b51534b
@roguenet
[TM-785] Get use of the roles migration consistent.
999eabe 
I'm torn on this approach for setting up the roles, but it's what we're using and leaving it commented out in tests is making the tests fail when run in isolation. The --fresh flag should never be needed, and without it the artisan command is quick and idempotent.
@roguenet
[TM-785] Clean up some warnings in tests found by PhpStorm
81db145
@roguenet
[TM-785] All V2 tests passing.
973d987
@roguenet
[TM-785] Another lint pass after fixing the tests.
5a8deaa
@roguenet
[TM-785] make test now runs locally without overwriting the local d…
3af2e8a 
…ev db.
@roguenet
[TM-785] Experiment with initial workflow structure.
d78ba05
@roguenet
[TM-785] Try a different command.
5a4cb69
@roguenet
[TM-785] See if the simplest workflow will work
7e8a968
@roguenet
[TM-785] See if I can get a default workflow from Github to work
f1db14c
@roguenet
[TM-785] Delete .github/workflows directory
6dca35d
@roguenet
[TM-785] Github workflow to run tests.
5818748
@roguenet
Merge remote-tracking branch 'origin/staging' into feat/TM-785-unit-t…
ea861e0 
…ests
@roguenet
Fix linting problem with new code from staging
2c17192
@roguenet
Merge pull request #135 from wri/feat/TM-785-unit-tests
fc77911 
[TM-785] Unit testing and linting for the BE repo
@roguenet
[TM-812] Streamline the process of updating from form values.
b7e704e
roguenet and others added 21 commits April 11, 2024 08:50
@roguenet
[TM-812] Include the form title on the form data so the entity edit p…
f2fb2b6 
…age doesn't need to load the entity too.
@roguenet
[TM-812] Include update request data in the entity schema response fo…
337974d 
…r FE efficiency.
@roguenet
[TM-812] Update docs
792eef9
@roguenet
[TM-812] Implement tree species sync in form updates.
cf92c77
@roguenet
[TM-812] Remove the tree species mutation endpoints.
4223664
@roguenet
[TM-812] Include the full update request resource on the EntityWithSc…
c58925a 
…hemaResource.
@roguenet
Merge pull request #142 from wri/feat/TM-812-tree-species
1d50364 
[TM-812] Tree Species refactor
@cesarLima1
TM-782 fix: change framework reference for exporting
09ca621
@cesarLima1
Merge pull request #143 from wri/TM-782_export_landscapes
3789e2b 
TM-782 fix: change framework reference for exporting
@roguenet
[TM-812] Remove project establishment and seed collection from workda…
e11ad5a 
…y collections.
@roguenet
Fix lint
9d365bf
@roguenet
[TM-812] Create permissions for the new terrafund-enterprises framework.
3a466c3
@roguenet
[TM-812] Use can instead of hasPermissionTo, as it doesn't throw an e…
c4dbdef 
…rror when the permission isn't found.
@roguenet
[TM-782] Fix lint and failing unit test from staging.
58b023a
@roguenet
[TM-782] Fix lint and failing unit test from staging.
93bd4c3
@roguenet
[TM-515] Only assign spatie roles if the user doesn't already have them.
bbbc874
@roguenet
Merge pull request #144 from wri/feat/TM-515-workday-categories-removal
788fbb5 
[TM-812] Remove project establishment and seed collection from workday collections
@roguenet
Merge pull request #146 from wri/fix/TM-833-new-framework
be521e9 
[TM-833] Allow creation of new frameworks without breaking existing API access.
@roguenet
[TM-714] Update workdays as an entity relation.
4aef243
@roguenet
[TM-714] Remove the workdays mutation endpoints.
8292b99
@roguenet
Merge pull request #147 from wri/feat/TM-741-workdays-be
1c39e14 
[TM-741] Turns Workdays into a BE concern
@roguenet roguenet mentioned this pull request Apr 18, 2024
Merged
@roguenet roguenet requested a review from pachonjcl April 18, 2024 20:51
@roguenet roguenet merged commit 73a21ca into main Apr 18, 2024
1 check passed
@roguenet roguenet deleted the release/educated-eucalyptus branch April 18, 2024 22:30
egrojMonroy pushed a commit that referenced this pull request Nov 1, 2024
@roguenet
Merge pull request #148 from wri/release/educated-eucalyptus
04dc334 
[RELEASE] Educated Eucalyptus
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pachonjcl pachonjcl pachonjcl approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@roguenet @pachonjcl @cesarLima1 @LimberHope