Skip to content

Changed frontend to be able to enrich only 1 member#937

Merged
joanreyero merged 1 commit intomainfrom
improvement/enrich-one
Jun 2, 2023
Merged

Changed frontend to be able to enrich only 1 member#937
joanreyero merged 1 commit intomainfrom
improvement/enrich-one

Conversation

@joanreyero
Copy link
Copy Markdown
Contributor

@joanreyero joanreyero commented Jun 2, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at bef1d58

This pull request improves the code style and adds a new feature to the member store module. It formats the code in actions.js to follow consistent conventions and handle edge cases. It also enables the doEnrichMemberBulk action to enrich a single member if the input array has only one element.

🤖 Generated by Copilot at bef1d58

Sing, O Muse, of the code that was polished and refined
By the skillful developer, who sought to align
The style and the syntax of actions.js
And make it more pleasing to human and god.

Why

How

🤖 Generated by Copilot at bef1d58

  • Add conditional logic to handle single member enrichment (link)
  • Format import statement and object properties for readability and consistency (link, link, link, link, link, link, link)
  • Remove extra line breaks after function calls for readability and consistency (link, link, link, link, link, link, link)
  • Break long filter condition into multiple lines for readability and consistency (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@joanreyero joanreyero merged commit 8efbcdc into main Jun 2, 2023
@joanreyero joanreyero deleted the improvement/enrich-one branch June 2, 2023 14:03
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.

1 participant