Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: 🍰 Refine Group Creation And Group Edit #5418

Merged
merged 26 commits into from
Oct 10, 2022

Conversation

ogerly
Copy link
Member

@ogerly ogerly commented Sep 28, 2022

🍰 Pull Request

  • add tiptap editor in FormGroup.vue
  • location Name wurd gelöscht
  • validation of formSchema
  • localisation of selects

Issues

tip-tap editor

image

clear location input

image

localisation

ezgif.com-gif-maker.mp4

@ogerly ogerly requested a review from Tirokk September 28, 2022 08:20
@ogerly ogerly self-assigned this Sep 28, 2022
@ogerly ogerly changed the base branch from master to 5344-add-group-members-management September 28, 2022 08:20
@Tirokk Tirokk changed the title 5397 refine group creation and group edit feat: [WIP] 🍰 Refine Group Creation And Group Edit Sep 28, 2022
@ogerly
Copy link
Member Author

ogerly commented Sep 29, 2022

FireShot Capture 1246 - ocelot social - ocelot social - localhost

FireShot Capture 1248 - ocelot social - ocelot social - localhost

@ogerly ogerly requested a review from Tirokk September 29, 2022 13:09
Base automatically changed from 5344-add-group-members-management to 5059-epic-groups September 30, 2022 06:13
Copy link
Member

@Tirokk Tirokk left a comment

Choose a reason for hiding this comment

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

I give a prereview and then go ahead with reviewing …

Copy link
Member

@Tirokk Tirokk left a comment

Choose a reason for hiding this comment

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

Cool that you have found out the validation problem with the selects. 😄

I would change some things …

The error message name must be between 3 and 50 characters is not translated.

Can we get rid of this grey strokes under the select if all is okay?
Or put 1 / 1 in the grey chip?

Bildschirmfoto 2022-10-02 um 12 48 29

We have different space between several things:

Bildschirmfoto 2022-10-02 um 12 56 29

@ogerly ogerly changed the title feat: [WIP] 🍰 Refine Group Creation And Group Edit feat: 🍰 Refine Group Creation And Group Edit Oct 5, 2022
@ogerly ogerly requested a review from Tirokk October 5, 2022 09:03
Copy link
Member

@Tirokk Tirokk left a comment

Choose a reason for hiding this comment

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

Very cool @ogerly ! 😍
Nice that you cared for the validations !!! 🚀🚀💫💫💫

I refined some things to make it even better, I hope.

Please test and have a look what I have changed before you merge:
350d59b

@ogerly
Copy link
Member Author

ogerly commented Oct 7, 2022

@Tirokk i put another ds-space in the form when editing group data. there was no clean space between slug and groupType.

@ogerly ogerly merged commit 356627a into 5059-epic-groups Oct 10, 2022
@ogerly ogerly deleted the 5397-Refine-Group-Creation-And-Group-Edit branch October 10, 2022 06:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🚀 [Feature] Refine Group Creation And Group Edit
2 participants