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

Update action buttons on details views #4362

Merged
merged 18 commits into from
Sep 21, 2024
Merged

Conversation

pa-lem
Copy link
Contributor

@pa-lem pa-lem commented Sep 17, 2024

  • Adds component for actions in details view depending on the tab
  • Refactor delete modal

@pa-lem pa-lem added the group/frontend Issue related to the frontend (React) label Sep 17, 2024
@pa-lem pa-lem requested a review from bilalabbad September 18, 2024 07:29
@pa-lem pa-lem added the type/feature New feature or request label Sep 18, 2024
@pa-lem
Copy link
Contributor Author

pa-lem commented Sep 21, 2024

will merge, only nats is timing out

@pa-lem pa-lem merged commit fd811a8 into develop Sep 21, 2024
27 of 28 checks passed
@pa-lem pa-lem deleted the ple-IFC-76-IFC-269-action-button branch September 21, 2024 14:17
pa-lem added a commit that referenced this pull request Sep 23, 2024
* start extract action buttons and modals for object details

* add relationship action button

* fix test id

* fix test id

* fix refetch

* udpate confirm label in modal delete

* fix refresh

* revert rename

* fix test

* fix refetch

* improve on delete

* improve delete label

* add fragment

* wait for ondelete

* mark tests as slow
pa-lem added a commit that referenced this pull request Sep 24, 2024
* start extract action buttons and modals for object details

* add relationship action button

* fix test id

* fix test id

* fix refetch

* udpate confirm label in modal delete

* fix refresh

* revert rename

* fix test

* fix refetch

* improve on delete

* improve delete label

* add fragment

* wait for ondelete

* mark tests as slow
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
group/frontend Issue related to the frontend (React) type/feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant