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

Replace build your own Xero management with connections SDK #1172

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ajacksoncodat
Copy link
Collaborator

Removed steps to build your own "Integration settings" (connection management) page.
Replaced with link to new connections management SDK.

Happy to have this change revoked if further discussion needed.

Removed steps to build your own "Integration settings" (connection management) page. Replaced with link to new connections management SDK.
Copy link

vercel bot commented May 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
codat-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 13, 2024 4:27pm

@ajacksoncodat ajacksoncodat changed the title Replace build your own Xero management page with connections SDK Replace build your own Xero management with connections SDK May 13, 2024
Copy link

github-actions bot commented May 13, 2024

Overall readability score: 63.41 (🔴 -0.01)

File Readability
checkpoints-lending.md 52.96 (🔴 -3.31)
View detailed metrics

🟢 - Shows an increase in readability
🔴 - Shows a decrease in readability

File Readability FRE GF ARI CLI DCRS
checkpoints-lending.md 52.96 46.67 10.98 12.5 13.45 7.93
  🔴 -3.31 🔴 -1.02 🔴 -0.88 🔴 -0.5 🔴 -0.23 🔴 -0.18

Averages:

  Readability FRE GF ARI CLI DCRS
Average 63.41 54.44 9.77 10.69 11.27 7.5
  🔴 -0.01 🟢 +0 🟢 +0 🟢 +0 🟢 +0 🟢 +0
View metric targets
Metric Range Ideal score
Flesch Reading Ease 100 (very easy read) to 0 (extremely difficult read) 60
Gunning Fog 6 (very easy read) to 17 (extremely difficult read) 8 or less
Auto. Read. Index 6 (very easy read) to 14 (extremely difficult read) 8 or less
Coleman Liau Index 6 (very easy read) to 17 (extremely difficult read) 8 or less
Dale-Chall Readability 4.9 (very easy read) to 9.9 (extremely difficult read) 6.9 or less

@pzaichkina
Copy link
Collaborator

hi @ajacksoncodat , two things to note here:

  1. please can you check with Jasper if he is happy with this change, because it's his team that uses these pages the most
  2. there are a few more pages with this requirement that also need to be updated - this one on Bank Feeds, this one on AP payments, and this one on general partnerships.

@pzaichkina pzaichkina added DO NOT MERGE WIP or a spike comments Comments left on the PR labels May 13, 2024
Copy link

Link check results (filtered):

[
  "[404] https://docs.codat.io/bank-feeds/integrations/qbo/bank-feeds-api",
  "[404] https://docs.codat.io/usecases/bill-pay/accounting-api"
]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comments Comments left on the PR DO NOT MERGE WIP or a spike
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants