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

Update FMTM Field Mapping Guidelines.md #2064

Merged
merged 7 commits into from
Jan 8, 2025

Conversation

manjitapandey
Copy link
Contributor

@manjitapandey manjitapandey commented Jan 8, 2025

Added FMTM guidelines for field mappers

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation
  • πŸ§‘β€πŸ’» Refactor
  • βœ… Test
  • πŸ€– Build or CI
  • ❓ Other (please specify)

Related Issue

Example: Related #921

Describe this PR

This PR adds the FMTM guidelines for field mappers including constraints of FMTM for field mappers.

Screenshots

Please provide screenshots of the change.

Alternative Approaches Considered

Did you attempt any other approaches that are not documented in code?

Review Guide

Notes for the reviewer. How to test this change?

Checklist before requesting a review

[optional] What gif best describes this PR or how it makes you feel?

@github-actions github-actions bot added the docs Improvements or additions to documentation label Jan 8, 2025
@spwoodcock spwoodcock force-pushed the manjitapandey-patch-4 branch from 97c759e to 323d7af Compare January 8, 2025 16:38
@spwoodcock
Copy link
Member

spwoodcock commented Jan 8, 2025

I merged some of the docs together a bit to simplify things.
(there was a little bit of duplicated info as a result that I removed)
Hope that's ok! πŸ˜„

Ideally:

  • One mapper guide
  • One manager guide

@spwoodcock spwoodcock merged commit 4147549 into development Jan 8, 2025
2 checks passed
@spwoodcock spwoodcock deleted the manjitapandey-patch-4 branch January 8, 2025 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants