Skip to content
This repository has been archived by the owner on Jul 8, 2022. It is now read-only.

Set persistent entity map #178

Closed
wants to merge 36 commits into from
Closed

Set persistent entity map #178

wants to merge 36 commits into from

Conversation

froboy
Copy link
Collaborator

@froboy froboy commented Jan 28, 2022

Do not merge PR, only review it and approve/request changes.
Merge is allowed by QA Engineer only.

Related Issue/Ticket:

PLEASE CHECK BASE BRANCH FOR YOUR PR
ONLY urgent and approved fixes for point release should go to master branch

#177

Steps to test:

This is challenging to test without first breaking your db by editing one of the entity.definitions.bundle_field_map rows in the key_value table.

Quality checks:

Please check these boxes to confirm this PR covers the following cases:

  • Maintaining our upgrade path is essential. Check one or the other:
    • This PR provides updates via hook_update_N or other means.
    • No updates are necessary for this change.
  • Front end fixes should be tested against all of the Open Y Themes.
    • Tested against Carnation
    • Tested against Lily
    • Tested against Rose
    • This change does not contain front-end fixes.
  • I have flagged this PR "Needs Review" or pinged the VY devs/QA
    team in Slack

froboy and others added 30 commits December 21, 2021 08:47
See fivejars@def1e0d for all of the mappings that are happening.
Will need deduping at some point but it's a start.
@froboy froboy requested a review from anpolimus January 28, 2022 17:54
@froboy froboy closed this Jan 28, 2022
@froboy
Copy link
Collaborator Author

froboy commented Jan 28, 2022

Made against the wrong fork.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants