-
Notifications
You must be signed in to change notification settings - Fork 113
Adding support for sharepoint backed Hermes #701
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
Closed
Closed
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
92e787d
Remove migration/deprecation banners
sjaiswalh 0ad81d9
Phase 2: Merge SharePoint backend into hermes
sjaiswalh eb77b08
Phase 3: Upgrade frontend to Ember 5 + Word Add-in cleanup
sjaiswalh 4103ad1
Phase 4: Update CI/CD, Dockerfile, and configuration
sjaiswalh 146537c
Phase 5: Cleanup enterprise values and fix tests
sjaiswalh 1db7775
fix: restore missing dotfiles and config files in web/ and hermes-plu…
sjaiswalh 6e98368
fix: remove accidentally tracked hermes-plugin/dist and clean config …
sjaiswalh 0a2d8b7
feat: implement dual-backend SharePoint/Google support across all API…
sjaiswalh 1d5cb83
Fix FileID/GoogleFileID dual-field lookups across all handlers
sjaiswalh 299d849
Remove MERGE_PLAN.md from branch
sjaiswalh b3c788a
Fix formatting/indentation (gofmt)
sjaiswalh 89f4ec1
Restore hcd.IsLocked checks in v2 approvals handler (lost in Phase 2 …
sjaiswalh a85eec3
Guard IsLocked checks with !srv.IsSharePoint() (Google-only concern)
sjaiswalh 1621ed7
Restore ReplaceHeader calls lost in Phase 2 merge (guarded for Google…
sjaiswalh fa76076
Restore remaining Google-only functions lost in Phase 2 merge
sjaiswalh c20b6e1
fix: dual-backend support for Google and SharePoint
sjaiswalh 2cbd708
Restore dual-backend support and add comprehensive tests
sjaiswalh 2a220d0
fix: restore Google mode support across frontend and backend (18 files)
sjaiswalh b517742
fix: remove SharePoint default init that breaks Google mode, add nil …
sjaiswalh 240c7e7
fix: resolve TS errors in document route (null check, await async)
sjaiswalh 7518dac
Fix SharePoint auth flow and add-in reauth
sjaiswalh 59e14d5
Retry published PATCH after draft 404
sjaiswalh 10ed77d
Fix published ownership transfer checks
sjaiswalh fbe699d
Fix dual-backend document and auth flows
sjaiswalh e432a23
Refine auth probes and startup routing
sjaiswalh ad0e52d
We don't index published folder in sharepoint version
sjaiswalh 3a7cd07
Update add-in manifest
sjaiswalh d5ed5c6
Handle paginated Graph memberOf lookups
sjaiswalh 43154be
Make draft archive owner check case-insensitive
sjaiswalh e2f51e3
Update readme and sample config
sjaiswalh File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| * @hashicorp-forge/labs | ||
| * @hashicorp-forge/labs @hashicorp/team-scale-performance-eng |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,48 @@ | ||
| name: Docker Build & Push | ||
|
|
||
| on: | ||
| push: | ||
| branches: [main] | ||
| workflow_dispatch: | ||
|
|
||
| jobs: | ||
| build-and-push: | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| contents: read | ||
| packages: write | ||
| steps: | ||
| - uses: actions/checkout@v3 | ||
|
|
||
| - name: Setup Node | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: "18" | ||
|
|
||
| - name: Enable Corepack | ||
| run: | | ||
| corepack enable | ||
| cd web | ||
| corepack prepare yarn@$(node -p "require('./package.json').packageManager.split('@')[1]") --activate | ||
|
|
||
| - name: Set up Go | ||
| uses: actions/setup-go@v4 | ||
| with: | ||
| go-version: "^1.24" | ||
|
|
||
| - name: Build | ||
| run: make build/linux | ||
|
|
||
| - name: Log in to GitHub Container Registry | ||
| uses: docker/login-action@v3 | ||
| with: | ||
| registry: ghcr.io | ||
| username: ${{ github.actor }} | ||
| password: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| - name: Build and push Docker image | ||
| uses: docker/build-push-action@v5 | ||
| with: | ||
| context: . | ||
| push: true | ||
| tags: ghcr.io/${{ github.repository }}:${{ github.sha }} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -9,6 +9,9 @@ | |
| .DS_Store | ||
| .env | ||
|
|
||
| # TLS certs (local dev) | ||
| /certs | ||
|
|
||
| # Web application | ||
| node_modules | ||
| /web/.pnp.* | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| FROM alpine:3.21.5 | ||
|
|
||
| # Update the package repository and install dependencies | ||
| RUN apk --no-cache update && \ | ||
| apk --no-cache upgrade | ||
|
|
||
| # Set the working directory | ||
| WORKDIR /app | ||
|
|
||
| # Copy the application binary and configuration | ||
| COPY hermes /app/hermes | ||
| COPY configs /app/configs | ||
|
|
||
| # Set the entrypoint to run the hermes application | ||
| ENTRYPOINT ["/app/hermes"] | ||
|
|
||
| # Default command when container starts | ||
| CMD ["server"] | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.