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.

Sign up for GitHub

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

Jump to bottom

Ci test #4447

Closed
wants to merge 33 commits into from
Closed

Ci test #4447

wants to merge 33 commits into from

Conversation

codewithvk
Copy link
Collaborator

  • Resolves: #
  • Target version: main

Summary

TODO

  • ...

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Documentation (manuals or wiki) has been updated or is not required

@codewithvk @juliusknorr
Initial cool admin setting iframe setup
6c519e1 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
WIP auth
0a1dccf 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
Temp: Auth handling stuff
62c41a0 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
WIP: wopi setting upload
58f567b 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
Created an AppData-based directory for system settings and user settings
73e2f70 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
Create a Settings controller API endpoint for handling AppData-based …
f5320d2 
…directories

Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
Temporary Commit: Created a temporary UI to validate the functionalit…
e993dde 
…y of user and system settings.

Note: We will delete this commit/code after the entire prototype is ready. Reviewers can ignore this commit during the review process.
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
Change the admin settings iframe URL to adminIntegratorSettings.
35f45ba 
TODO: We should try to centralize the path everywhere so that a change in one place updates it everywhere!
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
WOPI: Update the wopi setting upload route to accept a file and store…
3a39075 
… it to system-settings dir

Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
add delete button and create wopi/setting route to handle wopi file r…
c800205 
…equest

Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
Manage setting configs files with dynamic routes
c0e65e4 
- Implement dynamic routing for settings files, enabling URLs structured as /settings/{type}/{category}/{filename}.
- Support various setting types (e.g. userconfigs, sharedconfigs) and categories (e.g. autotext, wordbook) so that multiple files can be stored for each category.
- Ensure proper URL parsing and directory handling for uploading and retrieving files via the WOPI interface.

Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
Code cleanup: Remove POC helper functions
1660bca 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
Send WOPI setting base URL to integrator
dce7f65 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
fix: fetch config url
786f1cb 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
wopi: add delete setting file route
388b2b9 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
refactor: token generation for iframe
5704fdc 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
feat(user-settings): introduce iframe for user settings
fc0915a 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
fix: linting issue for CI
bdfa8b9 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
fix: generate token for user shared config url
6686315 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
fix: accept document and setting url token for file upload
dbf9d3a 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
fix: set proper WOPI response and remove unnecessary WOPI callback fr…
82bb7a8 
…om iframe

Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
fix(settings): remove iframe title and section
0e79c20 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
fix(settings): fetch directory from root folder
59a2192 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
fix(wopi): share SharedSettings to wopi checkfileInfo
747c528 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
fix(wopi): Generating setting token for guest users
5defde2 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
fix(settings): generate user config per userId & handle guest users
969a4f9 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
fix: composer psalm error via pointing correct folder
495a02c 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
fix: code cleanups
9554d4f 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
fix: move admin setting iframe below server-config
7c7c779 
Signed-off-by: codewithvk <[email protected]>
@codewithvk @juliusknorr
refactor: some code cleanups
bf69966 
Signed-off-by: codewithvk <[email protected]>
@codewithvk codewithvk closed this Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@elzody elzody Awaiting requested review from elzody elzody will be requested when the pull request is marked ready for review elzody is a code owner

@juliusknorr juliusknorr Awaiting requested review from juliusknorr juliusknorr will be requested when the pull request is marked ready for review juliusknorr is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@codewithvk @juliusknorr