Skip to content

Adjust form upload label changes#13836

Merged
andreslucena merged 20 commits intodecidim:developfrom
i-need-another-coffee:change/upload-form
Jan 16, 2025
Merged

Adjust form upload label changes#13836
andreslucena merged 20 commits intodecidim:developfrom
i-need-another-coffee:change/upload-form

Conversation

@alecslupu
Copy link
Copy Markdown
Contributor

🎩 What? Why?

Back in #13735 @andreslucena requested some changes in the UX via #13735 (review) . This PR aims to add those changes

📌 Related Issues

Link your PR to an issue

Testing

  1. Pipeline should be green
  2. Visit component / space creating form in frontend and admin area
  3. See the wording
  4. Apply patch
  5. See the new wording.

♥️ Thank you!

Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This pull request does not contain a valid label. Please add one of the following labels: ['type: feature', 'type: change', 'type: fix', 'type: removal', 'target: developer-experience', 'type: internal']

@alecslupu alecslupu self-assigned this Jan 14, 2025
@alecslupu alecslupu marked this pull request as ready for review January 14, 2025 08:27
@alecslupu alecslupu added the type: change PRs that implement a change for an existing feature label Jan 14, 2025
github-actions[bot]
github-actions bot previously approved these changes Jan 14, 2025
Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing one thing from my comment:

  • Change "(Optional) Add an attachment" to "Add a document or an image"

@alecslupu
Copy link
Copy Markdown
Contributor Author

@andreslucena Can we merge this ?

@andreslucena
Copy link
Copy Markdown
Member

@andreslucena Can we merge this ?

Yeah, of course, I missed it

@andreslucena andreslucena merged commit 825d981 into decidim:develop Jan 16, 2025
@alecslupu alecslupu deleted the change/upload-form branch January 16, 2025 13:39
greenwoodt pushed a commit to greenwoodt/decidim that referenced this pull request Jan 21, 2025
* Merged upload field on proposal admin form

* Fixed tests

* Run linter

* Fixed tests

* Removed unused translations

* Apply suggestions from code review

* Change text

* Form upload change labels

* Fix pipelines

* Fix failing specs

* Apply review recommendations

---------

Co-authored-by: andra-panaite <andra.panaite@tremend.com>
entantoencuanto added a commit that referenced this pull request Jan 24, 2025
* develop: (30 commits)
  Change hardcoded english in seeds (#13917)
  Allow importing accountability results from a Proposals component (#13817)
  Remove user interests (#13910)
  Move remaining categories from some resources to taxonomies (#13838)
  Fix translation issue on import projects mailer (#13894)
  Update DownloadYourData exports for decidim-debates (#13895)
  Allow admins to publish the questions' answers in surveys (#13786)
  Fix flaky spec in autocomplete (#13901)
  Refine taxonomy filters (part 2) (#13725)
  Fix single attachment with title (#13795)
  Fix incorrect breadcrumb encoding on mobile and tablet (#13891)
  Add string for unpublish survey admin log message (#13890)
  Fix ux meeting date location (#13872)
  New Crowdin updates (#13767)
  Fix deleted user error in schema (#13681)
  Remove assembly types from user interaction (#13881)
  Remove participatory process types from user interaction (#13880)
  Lock concurrent-ruby to 1.3.4 (#13879)
  Fix taxonomy serialization for export/imports (#13857)
  Adjust form upload label changes (#13836)
  ...
antopalidi pushed a commit to openpoke/decidim that referenced this pull request Feb 12, 2025
* Merged upload field on proposal admin form

* Fixed tests

* Run linter

* Fixed tests

* Removed unused translations

* Apply suggestions from code review

* Change text

* Form upload change labels

* Fix pipelines

* Fix failing specs

* Apply review recommendations

---------

Co-authored-by: andra-panaite <andra.panaite@tremend.com>
andreslucena pushed a commit that referenced this pull request Mar 3, 2025
* Merged upload field on proposal admin form

* Fixed tests

* Run linter

* Fixed tests

* Removed unused translations

* Apply suggestions from code review

* Change text

* Form upload change labels

* Fix pipelines

* Fix failing specs

* Apply review recommendations

---------

Co-authored-by: andra-panaite <andra.panaite@tremend.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants