Skip to content

Added functionality so the GD menu item and select folder are not enabled until oauth token is loaded #2571

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

Merged

Conversation

bp-cos
Copy link
Contributor

@bp-cos bp-cos commented Jun 6, 2025

Added functionality so the GD menu item and select folder are not enabled until oauth token is loaded

  • Ticket: []
  • Feature flag: n/a

Purpose

Added functionality so the GD menu item and select folder are not enabled until oauth token is loaded

Summary of Changes

Add a boolean to the GPF
Add disabled to the menu button and select folder button

Screenshot(s)

N/A

Side Effects

Prevents errors from users clicking the button before there is a token.

QA Notes

Now

@bp-cos bp-cos requested a review from adlius June 6, 2025 16:59
@adlius adlius merged commit e8a9108 into CenterForOpenScience:feature/pbs-25-10 Jun 7, 2025
9 checks passed
@bp-cos bp-cos deleted the feature/wait-to-load branch June 9, 2025 14:14
adlius added a commit that referenced this pull request Jun 13, 2025
* fix: use compact IRIs to match the api

(yet todo: json-ld parsing that would handle either)

* fix: IRIs represented as {"@id": "..."}

* (partly) fix types

* [ENG-7876] Feature/file picker select root (#2556)

* Added the files

* Removed tokens

* Updates to display the selected root folder

* Added GFP to the files page

* Updates to add the root folder

* Added the manager to reload the wb once a new file or folder is selected

* Enabled the disabled button featuree again

* Fixed a test

* Updates to the flow for a token from the BE

* Added the final touches and removed pruned instructions

* Updates for the PR

* Updates to move all the google file picker code into the component

* Cleaned up comments and models

* Update for PR request

* do not send initiate_oauth after oauth flow is successful

* fix

* Removed a call to a function

* Added functionality so the GD menu item and select folder are not enabled until oauth token is loaded (#2571)

* Removed a call to a function

* Updates to disable the menu and select folder button until the token is loaded

* remove unnecessary group embed

* Fixed a css issue (#2577)

---------

Co-authored-by: abram axel booth <boothaa@gmail.com>
Co-authored-by: abram axel booth <aaxelb@users.noreply.github.com>
Co-authored-by: Lord Business <113387478+bp-cos@users.noreply.github.com>
Co-authored-by: Brian Pilati <brianpilati@cos.io>
Co-authored-by: John Tordoff <Johnetordoff@users.noreply.github.com>
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