Skip to content

docs(compensation): assign transfer fees to contractor#7

Merged
markholdex merged 1 commit into
mainfrom
resolve-hr-issue-847-terms
Jun 9, 2026
Merged

docs(compensation): assign transfer fees to contractor#7
markholdex merged 1 commit into
mainfrom
resolve-hr-issue-847-terms

Conversation

@zolotokrylin

@zolotokrylin zolotokrylin commented Jun 9, 2026

Copy link
Copy Markdown
Member

Summary

  • New Transfer Fees clause (§ Compensation 8) states the Company pays the exact invoice amount and any fees charged by the contractor's bank, receiving payment network, correspondent banks, or intermediaries in transit are the contractor's responsibility.

  • Renumbered Compensation clauses 8–10 to 9–11 accordingly.

  • CHANGELOG updated with v1.4.0 entry.

  • Closes https://github.com/holdex/hr-internal/issues/847

Summary by CodeRabbit

  • Documentation
    • Added v1.4.0 changelog entry (effective 2026-06-09).
    • Introduced new "Transfer Fees" compensation clause specifying that the Company pays the exact invoice amount while the Contractor assumes responsibility for any fees charged by their bank or payment intermediaries.

@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 782ba3fb-ca0b-4125-a438-cc64c8e9fe67

📥 Commits

Reviewing files that changed from the base of the PR and between c90567b and 35cf057.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • STANDARD_TERMS.md

Walkthrough

This PR adds a new "Transfer Fees" clause to the Compensation section of the standard terms, establishing that the Company pays the exact invoice amount and the Contractor assumes responsibility for any fees charged by banks or payment intermediaries. The change is documented in the changelog and shifts downstream clause numbering.

Changes

Transfer Fees Clause Addition

Layer / File(s) Summary
Add Transfer Fees clause and renumber subsequent clauses
STANDARD_TERMS.md
New Compensation clause specifies Company pays exact valid invoice amount; Contractor bears intermediary/bank transfer fees. Downstream clauses renumbered: "Taxes" becomes clause 10, "Finality of Payment" becomes clause 11.
Document Transfer Fees in changelog
CHANGELOG.md
New v1.4.0 entry documents the Transfer Fees clause and clarifies fee responsibility for bank/payment network charges during payment transfer.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Suggested reviewers

  • markholdex
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding a Transfer Fees clause to the Compensation section that assigns these fees to the contractor.
Linked Issues check ✅ Passed The PR fully addresses the objective in #847 by adding a Transfer Fees clause clarifying that Holdex pays the exact invoice amount and contractor bears network/intermediary fees.
Out of Scope Changes check ✅ Passed All changes are directly scoped to adding the Transfer Fees clause and updating documentation; no unrelated modifications are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch resolve-hr-issue-847-terms

Warning

Review ran into problems

🔥 Problems

Stopped waiting for pipeline failures after 30000ms. One of your pipelines takes longer than our 30000ms fetch window to run, so review may not consider pipeline-failure results for inline comments if any failures occurred after the fetch window. Increase the timeout if you want to wait longer or run a @coderabbit review after the pipeline has finished.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@zolotokrylin
zolotokrylin marked this pull request as ready for review June 9, 2026 13:00
@zolotokrylin
zolotokrylin requested a review from markholdex June 9, 2026 13:00
@zolotokrylin

Copy link
Copy Markdown
Member Author

@holdex pr submit-time 15m

@holdex

holdex Bot commented Jun 9, 2026

Copy link
Copy Markdown

Time Submission Status

Member # Time Running Total Status Last Update
zolotokrylin 15min ✅ Submitted Jun 9, 2026, 1:02 PM
markholdex 10min ✅ Submitted Jun 9, 2026, 1:08 PM

Submit or update total time with:

@holdex pr submit-time 2h

Add time on top of previous submission with:

@holdex pr add-time 1h30m

See available commands to help comply with our Guidelines.

@markholdex

Copy link
Copy Markdown

@holdex pr submit-time 10m

@markholdex
markholdex merged commit 00906b8 into main Jun 9, 2026
5 checks passed
@markholdex
markholdex deleted the resolve-hr-issue-847-terms branch June 9, 2026 13:08
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.

2 participants